Fixing Code Sniffer errors

cakephp · Dec 28, 2015 · c2449c9 · c2449c9
1 parent 33f2f46
commit c2449c9
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 4 deletions.
diff --git a/lib/Cake/Cache/Cache.php b/lib/Cake/Cache/Cache.php
@@ -275,7 +275,7 @@ public static function set($settings = array(), $value = null, $config = 'defaul
  *
  * @param string $config [optional] The config name you wish to have garbage collected. Defaults to 'default'
  * @param int $expires [optional] An expires timestamp. Defaults to NULL
- * @return boolean
+ * @return bool
  */
 	public static function gc($config = 'default', $expires = null) {
 		return static::$_engines[$config]->gc($expires);

diff --git a/lib/Cake/Cache/Engine/FileEngine.php b/lib/Cake/Cache/Engine/FileEngine.php
@@ -108,7 +108,7 @@ public function gc($expires = null) {
  * @param int $duration How long to cache the data, in seconds
  * @return bool True if the data was successfully cached, false on failure
  */
-	public function write($key, $data = null, $duration) {
+	public function write($key, $data, $duration) {
 		if (!$this->_init) {
 			return false;
 		}

diff --git a/lib/Cake/Model/Datasource/Session/CacheSession.php b/lib/Cake/Model/Datasource/Session/CacheSession.php
@@ -53,7 +53,7 @@ public function close() {
  */
 	public function read($id) {
 		$data = Cache::read($id, Configure::read('Session.handler.config'));
-		if(empty($data)){
+		if (empty($data)){
 			return '';
 		}
 		return $data;

diff --git a/lib/Cake/Test/Case/Model/Datasource/Session/DatabaseSessionTest.php b/lib/Cake/Test/Case/Model/Datasource/Session/DatabaseSessionTest.php
@@ -157,7 +157,6 @@ public function testDestroy() {
 
 		$this->assertTrue($this->storage->destroy('foo'), 'Destroy failed');
 		$this->assertSame($this->storage->read('foo'), '');
-
 	}
 
 /**