Correct return types.

src/Controller/Component/CookieComponent.php

@@ -171,7 +171,7 @@ public function initialize(array $config) {
  * @param null|string|array $option Either the option name to set, or an array of options to set,
  *   or null to read config options for a given key.
  * @param string|null $value Either the value to set, or empty when $option is an array.
- * @return void
+ * @return array|null
  */
 	public function configKey($keyname, $option = null, $value = null) {
 		if ($option === null) {
@@ -183,6 +183,7 @@ public function configKey($keyname, $option = null, $value = null) {
 			$option = [$option => $value];
 		}
 		$this->_keyConfig[$keyname] = $option;
+		return null;
 	}
 
 /**

src/Controller/Component/RequestHandlerComponent.php

@@ -625,7 +625,7 @@ public function responseType() {
  * Maps a content type alias back to its mime-type(s)
  *
  * @param string|array $alias String alias to convert back into a content type. Or an array of aliases to map.
- * @return string Null on an undefined alias. String value of the mapped alias type. If an
+ * @return string|null Null on an undefined alias. String value of the mapped alias type. If an
  *   alias maps to more than one content type, the first one will be returned.
  */
 	public function mapAlias($alias) {

src/Core/functions.php

@@ -167,7 +167,7 @@ function pj($var) {
  * environment information.
  *
  * @param string $key Environment variable name.
- * @return string Environment variable setting.
+ * @return string|null Environment variable setting.
  * @link http://book.cakephp.org/3.0/en/core-libraries/global-constants-and-functions.html#env
  */
 	function env($key) {

src/Database/Connection.php

@@ -134,7 +134,7 @@ public function config() {
  */
 	public function configName() {
 		if (empty($this->_config['name'])) {
-			return null;
+			return '';
 		}
 		return $this->_config['name'];
 	}

src/Database/Schema/PostgresSchema.php

@@ -151,7 +151,7 @@ public function convertColumnDescription(Table $table, $row) {
  * We need to remove those.
  *
  * @param string $default The default value.
- * @return string
+ * @return string|null
  */
 	protected function _defaultValue($default) {
 		if (is_numeric($default) || $default === null) {

src/Database/Type/BinaryType.php

@@ -44,7 +44,7 @@ public function toDatabase($value, Driver $driver) {
  *
  * @param null|string|resource $value The value to convert.
  * @param Driver $driver The driver instance to convert with.
- * @return resource
+ * @return resource|null
  * @throws \Cake\Core\Exception\Exception
  */
 	public function toPHP($value, Driver $driver) {

src/Filesystem/Folder.php

@@ -596,7 +596,7 @@ public function delete($path = null) {
 			$path = $this->pwd();
 		}
 		if (!$path) {
-			return null;
+			return false;
 		}
 		$path = Folder::slashTerm($path);
 		if (is_dir($path)) {

src/Network/Response.php

@@ -723,7 +723,7 @@ public function httpCodes($code = null) {
  * e.g `type(array('jpg' => 'text/plain'));`
  *
  * @param string $contentType Content type key.
- * @return mixed current content type or false if supplied an invalid content type
+ * @return mixed Current content type or false if supplied an invalid content type
  */
 	public function type($contentType = null) {
 		if ($contentType === null) {
@@ -751,7 +751,7 @@ public function type($contentType = null) {
  * e.g `getMimeType('pdf'); // returns 'application/pdf'`
  *
  * @param string $alias the content type alias to map
- * @return mixed string mapped mime type or false if $alias is not mapped
+ * @return mixed String mapped mime type or false if $alias is not mapped
  */
 	public function getMimeType($alias) {
 		if (isset($this->_mimeTypes[$alias])) {
@@ -837,7 +837,7 @@ public function cache($since, $time = '+1 day') {
  *   if set to false, the response will be set to private
  *   if no value is provided, it will return whether the response is sharable or not
  * @param int $time time in seconds after which the response should no longer be considered fresh
- * @return bool
+ * @return bool|null
  */
 	public function sharable($public = null, $time = null) {
 		if ($public === null) {
@@ -872,7 +872,7 @@ public function sharable($public = null, $time = null) {
  * If called with no parameters, this function will return the current max-age value if any
  *
  * @param int $seconds if null, the method will return the current s-maxage value
- * @return int
+ * @return int|null
  */
 	public function sharedMaxAge($seconds = null) {
 		if ($seconds !== null) {
@@ -892,7 +892,7 @@ public function sharedMaxAge($seconds = null) {
  * If called with no parameters, this function will return the current max-age value if any
  *
  * @param int $seconds if null, the method will return the current max-age value
- * @return int
+ * @return int|null
  */
 	public function maxAge($seconds = null) {
 		if ($seconds !== null) {
@@ -955,7 +955,7 @@ protected function _setCacheControl() {
  * `$response->expires()` Will return the current expiration header value
  *
  * @param string|\DateTime $time Valid time string or \DateTime instance.
- * @return string
+ * @return string|null
  */
 	public function expires($time = null) {
 		if ($time !== null) {
@@ -979,7 +979,7 @@ public function expires($time = null) {
  * `$response->modified()` Will return the current Last-Modified header value
  *
  * @param string|\DateTime $time Valid time string or \DateTime instance.
- * @return string
+ * @return string|null
  */
 	public function modified($time = null) {
 		if ($time !== null) {
@@ -1024,7 +1024,7 @@ public function notModified() {
  *
  * @param string|array $cacheVariances a single Vary string or an array
  *   containing the list for variances.
- * @return array
+ * @return array|null
  */
 	public function vary($cacheVariances = null) {
 		if ($cacheVariances !== null) {
@@ -1056,7 +1056,7 @@ public function vary($cacheVariances = null) {
  * @param string $hash the unique hash that identifies this response
  * @param bool $weak whether the response is semantically the same as
  *   other with the same hash or not
- * @return string
+ * @return string|null
  */
 	public function etag($hash = null, $weak = false) {
 		if ($hash !== null) {

src/Network/Session.php

@@ -352,7 +352,7 @@ public function check($name = null) {
 		}
 
 		if (!isset($_SESSION)) {
-			return null;
+			return false;
 		}
 
 		return Hash::get($_SESSION, $name) !== null;

src/Network/Socket.php

@@ -188,7 +188,7 @@ protected function _connectionErrorHandler($code, $message) {
  */
 	public function context() {
 		if (!$this->connection) {
-			return;
+			return null;
 		}
 		return stream_context_get_options($this->connection);
 	}
@@ -232,7 +232,7 @@ public function addresses() {
 /**
  * Get the last error as a string.
  *
- * @return string Last error
+ * @return string|null Last error
  */
 	public function lastError() {
 		if (!empty($this->lastError)) {

src/ORM/ResultSet.php

@@ -255,7 +255,7 @@ public function unserialize($serialized) {
  * end of the results simulating the behavior when the result set is backed
  * by a statement.
  *
- * @return array|object
+ * @return array|object|null
  */
 	public function first() {
 		if (isset($this->_results[0])) {

src/Routing/Filter/AssetFilter.php

@@ -48,7 +48,7 @@ public function beforeDispatch(Event $event) {
 
 		$url = urldecode($request->url);
 		if (strpos($url, '..') !== false || strpos($url, '.') === false) {
-			return;
+			return null;
 		}
 
 		$assetFile = $this->_getAssetFile($url);

src/Validation/Validator.php

@@ -149,7 +149,7 @@ public function hasField($name) {
  *
  * @param string $name  The name under which the provider should be set.
  * @param null|object|string $object Provider object or class name.
- * @return Validator|object|string
+ * @return Validator|object|string|null
  */
 	public function provider($name, $object = null) {
 		if ($object === null) {

src/View/Helper/PaginatorHelper.php

@@ -175,7 +175,7 @@ public function current($model = null) {
  *
  * @param string $model Optional model name. Uses the default if none is specified.
  * @param array $options Options for pagination links. See #options for list of keys.
- * @return string The name of the key by which the recordset is being sorted, or
+ * @return string|null The name of the key by which the recordset is being sorted, or
  *  null if the results are not currently sorted.
  * @link http://book.cakephp.org/2.0/en/core-libraries/helpers/paginator.html#PaginatorHelper::sortKey
  */
@@ -491,7 +491,7 @@ protected function _hasPage($model, $page) {
 /**
  * Gets the default model of the paged sets
  *
- * @return string Model name or null if the pagination isn't initialized.
+ * @return string|null Model name or null if the pagination isn't initialized.
  * @link http://book.cakephp.org/2.0/en/core-libraries/helpers/paginator.html#PaginatorHelper::defaultModel
  */
 	public function defaultModel() {