Resolve deprecation warnings and backwards compat issues.

Author: beporter

README.md

@@ -105,19 +105,19 @@ Array
 
 CakePHP 3 by default "consumes" some of its configs so as not to confuse developers. [`Configure::consume()`](http://book.cakephp.org/3.0/en/development/configuration.html#Cake\Core\Configure::consume) removes the configuration key from Configure, making it unavailable to the rest of the app. At the [time of this writing](https://github.com/cakephp/app/blob/a0f2c4/config/bootstrap.php#L136,L141), it does this for the following keys/classes:
 
-| _`[Configure.Key]`_  | _`Class::configEnumerationMethod()`_  | _`Class::configFetchMethod()`_  |
-|----------------------|---------------------------------------|---------------------------------|
-| `[Cache]`            | `Cache::configured()`                 | `Cache::config()`               |
-| `[Datasources]`      | `ConnectionManager::configured()`     | `ConnectionManager::config()`   |
-| `[EmailTransport]`   | `Email::configuredTransport()`        | `Email::configTransport()`      |
-| `[Email]`            | `Email::configured()`                 | `Email::config()`               |
-| `[Log]`              | `Log::configured()`                   | `Log::config()`                 |
-| `[Security.salt]`    | _(none)_                              | `Security::salt()`              |
+| _`[Configure.Key]`_  | _`Class::configEnumerationMethod()`_  | _`Class::configFetchMethod()`_    |
+|----------------------|---------------------------------------|-----------------------------------|
+| `[Cache]`            | `Cache::configured()`                 | `Cache::getConfig()`              |
+| `[Datasources]`      | `ConnectionManager::configured()`     | `ConnectionManager::getConfig()`  |
+| `[EmailTransport]`   | `Email::configuredTransport()`        | `Email::getConfigTransport()`     |
+| `[Email]`            | `Email::configured()`                 | `Email::getConfig()`              |
+| `[Log]`              | `Log::configured()`                   | `Log::getConfig()`                |
+| `[Security.salt]`    | _(none)_                              | `Security::getSalt()`             |
 
 
-The ConfigReadShell devotes about half of its codebase dealing with this for you, allowing you to continue to fetch values using the Configure path (`Datasources.default.host` -> `localhost`) while in the background it is actually querying `ConnectionManager::config('default')['host']`. (This is particularly helpful if you are using [Environment-Aware Configs](https://github.com/beporter/CakePHP-EnvAwareness/tree/master/slides).)
+The ConfigReadShell devotes about half of its codebase dealing with this for you, allowing you to continue to fetch values using the Configure path (`Datasources.default.host` -> `localhost`) while in the background it is actually querying `ConnectionManager::getConfig('default')['host']`. (This is particularly helpful if you are using [Environment-Aware Configs](https://github.com/beporter/CakePHP-EnvAwareness/tree/master/slides).)
 
-The "gotcha" here is that ConfigReadShell has to maintain a hard-coded list of Configure keys that are normally consumed, and how to access them in their new container. This is further complicated by the fact that not all consumed configs are loaded into or retrieved from their containers the same way, although the base assumption is that the container implements the [`StaticConfigTrait`](http://api.cakephp.org/3.0/class-Cake.Core.StaticConfigTrait.html) and so will have `::config()` and `::configured()` available.
+The "gotcha" here is that ConfigReadShell has to maintain a hard-coded list of Configure keys that are normally consumed, and how to access them in their new container. This is further complicated by the fact that not all consumed configs are loaded into or retrieved from their containers the same way, although the base assumption is that the container implements the [`StaticConfigTrait`](http://api.cakephp.org/3.0/class-Cake.Core.StaticConfigTrait.html) and so will have `::getConfig()` and `::configured()` available.
 
 :warning: **If your app uses `Configure::consume()` on any non-standard Configure key during bootstrapping, you will not be able to obtain any child values of those keys from the ConfigReadShell.**
 

src/Shell/ConfigReadShell.php

@@ -47,29 +47,29 @@ class ConfigReadShell extends Shell {
 	 * Cake 3 uses Configure::consume() on a number of Configure keys to
 	 * prime different Cake modules in `config/boostrap.php`.
 	 *
-	 * Cache::config(Configure::consume('Cache'));
-	 * ConnectionManager::config(Configure::consume('Datasources'));
-	 * Email::configTransport(Configure::consume('EmailTransport'));
-	 * Email::config(Configure::consume('Email'));
-	 * Log::config(Configure::consume('Log'));
-	 * Security::salt(Configure::consume('Security.salt'));
+	 * Cache::setConfig(Configure::consume('Cache'));
+	 * ConnectionManager::setConfig(Configure::consume('Datasources'));
+	 * Email::setConfigTransport(Configure::consume('EmailTransport'));
+	 * Email::setConfig(Configure::consume('Email'));
+	 * Log::setConfig(Configure::consume('Log'));
+	 * Security::setSalt(Configure::consume('Security.salt'));
 	 *
 	 * This removes the configs from Configure, making them inaccessible
 	 * through standard means in this Shell.
 	 *
 	 * This property tracks specific key names and the class::method they
 	 * map to so we can perform the correct logic to fetch the values. For
 	 * example, a request for `Cache._cake_core_.className` would result
-	 * in a call like `\Cake\Cache\Cache::config('_cake_core_')['className']`.
+	 * in a call like `\Cake\Cache\Cache::getConfig('_cake_core_')['className']`.
 	 *
 	 * @var array
 	 */
 	public $specialKeys = [
-		'Cache' => '\Cake\Cache\Cache::config',
-		'Datasources' => '\Cake\Datasource\ConnectionManager::config',
-		'EmailTransport' => '\Cake\Mailer\Email::configTransport',
-		'Email' => '\Cake\Mailer\Email::config',
-		'Log' => '\Cake\Log\Log::config',
+		'Cache' => '\Cake\Cache\Cache::getConfig',
+		'Datasources' => '\Cake\Datasource\ConnectionManager::getConfig',
+		'EmailTransport' => '\Cake\Mailer\Email::getConfigTransport',
+		'Email' => '\Cake\Mailer\Email::getConfig',
+		'Log' => '\Cake\Log\Log::getConfig',
 		'Security.salt' => 'self::securitySaltHelper',
 	];
 
@@ -100,7 +100,7 @@ public function startup() {
 		}
 
 		// All other output should not be processed by the Shell.
-		$this->_io->outputAs(ConsoleOutput::RAW);
+		$this->_io->setOutputAs(ConsoleOutput::RAW);
 	}
 
 	/**
@@ -262,16 +262,16 @@ protected function specialKey($search) {
 	 * There are three cases to handle:
 	 *
 	 *   1. A "deep" subkey like `Cache.default.className`. In this case,
-	 *      we need to fetch `Cache::config('default')` and then return
+	 *      we need to fetch `Cache::getConfig('default')` and then return
 	 *      the ['className'] from the result.
 	 *
 	 *   2. A single config array like `Cache.default. We need to fetch
-	 *      `Cache::config('default')` and return the whole thing.
+	 *      `Cache::getConfig('default')` and return the whole thing.
 	 *
 	 *   3. All configs in a module like `Cache`. In this case we need to
 	 *      try calling `Cache::configured()` (if it exists), then looping
 	 *      over the results using each as a key name for a separate call
-	 *      to `Cache::config($name)` and accumulating all of the results
+	 *      to `Cache::getConfig($name)` and accumulating all of the results
 	 *      together to return.
 	 *
 	 * @param array $special An array containing at least a [callable] key and possibly [arg] and [subkey]s.
@@ -282,7 +282,7 @@ protected function fetchSpecial($special) {
 			$set = call_user_func($special['callable'], $special['arg']);
 		} else {
 			$allConfigCallable = str_replace(
-				'::config',
+				'::getConfig',
 				'::configured',
 				$special['callable'],
 				$replaceCount
@@ -362,20 +362,20 @@ protected function printVal($key, $val) {
 	/**
 	 * Provides a custom helper for fetching the App's Security.salt value.
 	 *
-	 * The call to Security::salt() method requires no arguments to get the
+	 * The call to Security::getSalt() method requires no arguments to get the
 	 * current value but we will have split the command line request for
 	 * `Security.salt` into
-	 * `call_user_func('\Cake\Utility\Security::salt', 'salt'). That will
+	 * `call_user_func('\Cake\Utility\Security::getSalt', 'salt'). That will
 	 * **set** the salt to `salt` and return "salt" as the new value, which
 	 * isn't what we want. This method exists to be the callable function,
-	 * which itself passes the proper `null` value as the argument, which
-	 * in turn will return the _actual_ salt value.
+	 * which itself passes no arguments to getSalt, which in turn will return
+	 * the _actual_ salt value.
 	 *
 	 * @param string $salt Because of how this is implemented, this will always be the literal string "salt" and will be ignored.
-	 @return string The result of calling `Security::salt(null);`
+	 * @return string The result of calling `Security::getSalt();`
 	 */
 	private function securitySaltHelper($salt) {
-		return call_user_func('\Cake\Utility\Security::salt', null);
+		return call_user_func('\Cake\Utility\Security::getSalt');
 	}
 
 	/**
@@ -402,10 +402,10 @@ public function getOptionParser() {
 				'default' => false,
 				'help' => __('Encode all output using PHP\'s `serialize()` method. Makes the Shell\'s output suitable for consumption by other PHP console scripts. Always overrides the --bash option. A single requested key will be serialized directly. Multiple requested keys will be combined into an associative array with the provided arguments as key names and then serialized.'),
 			])
-			->description(
+			->setDescription(
 				__('Provides CLI access to variables defined in the Configure class of the host CakePHP application. Will output the value of any keys passed as arguments. Equivelant to `Configure::read(\'Key.Name\')`. Unrecognized keys will produce empty string or `null` output.')
 			)
-			->epilog(
+			->setEpilog(
 				__('Provide the Key.name(s) to fetch from Configure::read() as arguments. Multiple keys may be specified, separated by spaces.')
 			);
 

tests/TestCase/Shell/ConfigReadShellTest.php

@@ -88,10 +88,10 @@ class ConfigReadShellTest extends TestCase {
 	 */
 	public static function setUpBeforeClass() {
 		// Prime the ConnectionManager with some configs.
-		ConnectionManager::config(self::$datasources);
+		ConnectionManager::setConfig(self::$datasources);
 
 		// Prime the security salt.
-		Security::salt(self::$salt);
+		Security::setSalt(self::$salt);
 	}
 
 	/**
@@ -104,7 +104,7 @@ public static function tearDownAfterClass() {
 			ConnectionManager::drop($ds);
 		}
 
-		Security::salt('');
+		Security::setSalt('');
 	}
 
 	/**
@@ -206,26 +206,29 @@ protected function initSUT($additionalMocks = []) {
 			'_displayHelp', 'error',
 		];
 
-		$this->io = $this->getMock('\Cake\Console\ConsoleIo', [], [], '', false);
+		$this->io = $this->getMockBuilder('\Cake\Console\ConsoleIo')
+			->disableOriginalConstructor()
+			->getMock();
 		$this->io->expects($this->any())
 			->method('out')
 			->with($this->anything())
 			->will($this->returnCallback([$this, 'outputCollector']));
 
-
 		$class = $this->getSUTClassName();
 		$mockedMethods = array_merge($defaultMocks, $additionalMocks);
-		$shell = $this->getMock(
-			$class,
-			$mockedMethods,
-			[$this->io]
-		);
+		$shell = $this->getMockBuilder($class)
+			->setMethods($mockedMethods)
+			->setConstructorArgs([$this->io])
+			->getMock();
 
 		$shell->expects($this->any())
 			->method('error')
 			->with($this->anything())
 			->will($this->returnCallback([$this, 'outputCollector']));
-		$shell->OptionParser = $this->getMock('\Cake\Console\ConsoleOptionParser', [], [null, false]);
+		$shell->OptionParser = $this->getMockBuilder('\Cake\Console\ConsoleOptionParser')
+			->setMethods([])
+			->setConstructorArgs([null, false])
+			->getMock();
 		$shell->params = [
 			'help' => false,
 			'verbose' => false,
@@ -253,7 +256,7 @@ public function testStartupHelp() {
 		$this->Shell->expects($this->once())
 			->method('_displayHelp');
 
-		Cache::config('_cake_core_', [
+		Cache::setConfig('_cake_core_', [
 			'className' => 'File',
 		]);
 		$this->Shell->startup();
@@ -482,7 +485,6 @@ public function provideSerializedArgs() {
 		];
 	}
 
-
 	/**
 	 * test main(), requesting "special" config keys.
 	 *

tests/bootstrap.php

@@ -52,7 +52,7 @@
 if (!getenv('db_dsn')) {
 	putenv('db_dsn=sqlite:///:memory:');
 }
-ConnectionManager::config('test', ['url' => getenv('db_dsn')]);
+ConnectionManager::setConfig('test', ['url' => getenv('db_dsn')]);
 
 Plugin::load('ConfigReadShell', [
 	'path' => dirname(dirname(__FILE__)) . DS,