PHPSocialNetwork / phpfastcache

src/phpFastCache/Drivers/Memcached/Item.php

Doc Comments +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
 
     /**
      * Item constructor.
-     * @param \phpFastCache\Drivers\Memcache\Driver $driver
+     * @param Driver $driver
      * @param $key
      * @throws \InvalidArgumentException
      */

src/phpFastCache/Drivers/Predis/Item.php

Doc Comments +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
 
     /**
      * Item constructor.
-     * @param \phpFastCache\Drivers\Apc\Driver $driver
+     * @param Driver $driver
      * @param $key
      * @throws \InvalidArgumentException
      */

src/phpFastCache/Core/Item/ItemExtendedTrait.php

Spacing +10 added lines, -10 removed lines

@@ -67,9 +67,9 @@ 
      */
     public function getCreationDate()
     {
-        if($this->driver->getConfig()['itemDetailedDate']){
+        if ($this->driver->getConfig()['itemDetailedDate']) {
             return $this->creationDate;
-        }else{
+        } else {
             throw new \LogicException('Cannot access to the creation date when the "itemDetailedDate" configuration is disabled.');
         }
     }
@@ -81,10 +81,10 @@ 
      */
     public function setCreationDate(\DateTimeInterface $date)
     {
-        if($this->driver->getConfig()['itemDetailedDate']){
+        if ($this->driver->getConfig()['itemDetailedDate']) {
             $this->creationDate = $date;
             return $this;
-        }else{
+        } else {
             throw new \LogicException('Cannot access to the creation date when the "itemDetailedDate" configuration is disabled.');
         }
     }
@@ -95,9 +95,9 @@ 
      */
     public function getModificationDate()
     {
-        if($this->driver->getConfig()['itemDetailedDate']){
+        if ($this->driver->getConfig()['itemDetailedDate']) {
             return $this->modificationDate;
-        }else{
+        } else {
             throw new \LogicException('Cannot access to the modification date when the "itemDetailedDate" configuration is disabled.');
         }
     }
@@ -109,10 +109,10 @@ 
      */
     public function setModificationDate(\DateTimeInterface $date)
     {
-        if($this->driver->getConfig()['itemDetailedDate']){
+        if ($this->driver->getConfig()['itemDetailedDate']) {
             $this->modificationDate = $date;
             return $this;
-        }else{
+        } else {
             throw new \LogicException('Cannot access to the creation date when the "itemDetailedDate" configuration is disabled.');
         }
     }
@@ -279,7 +279,7 @@ 
     public function removeTag($tagName)
     {
         if (($key = array_search($tagName, $this->tags)) !== false) {
-            unset($this->tags[ $key ]);
+            unset($this->tags[$key]);
             $this->removedTags[] = $tagName;
         }
 
@@ -345,7 +345,7 @@ 
     final public function __debugInfo()
     {
         $info = get_object_vars($this);
-        $info[ 'driver' ] = 'object(' . get_class($info[ 'driver' ]) . ')';
+        $info['driver'] = 'object(' . get_class($info['driver']) . ')';
 
         return (array) $info;
     }

Braces +4 added lines, -4 removed lines

@@ -69,7 +69,7 @@ 
     {
         if($this->driver->getConfig()['itemDetailedDate']){
             return $this->creationDate;
-        }else{
+        } else{
             throw new \LogicException('Cannot access to the creation date when the "itemDetailedDate" configuration is disabled.');
         }
     }
@@ -84,7 +84,7 @@ 
         if($this->driver->getConfig()['itemDetailedDate']){
             $this->creationDate = $date;
             return $this;
-        }else{
+        } else{
             throw new \LogicException('Cannot access to the creation date when the "itemDetailedDate" configuration is disabled.');
         }
     }
@@ -97,7 +97,7 @@ 
     {
         if($this->driver->getConfig()['itemDetailedDate']){
             return $this->modificationDate;
-        }else{
+        } else{
             throw new \LogicException('Cannot access to the modification date when the "itemDetailedDate" configuration is disabled.');
         }
     }
@@ -112,7 +112,7 @@ 
         if($this->driver->getConfig()['itemDetailedDate']){
             $this->modificationDate = $date;
             return $this;
-        }else{
+        } else{
             throw new \LogicException('Cannot access to the creation date when the "itemDetailedDate" configuration is disabled.');
         }
     }

src/phpFastCache/Core/Pool/DriverBaseTrait.php

Indentation +3 added lines, -3 removed lines

@@ -159,9 +159,9 @@
     public function driverPreWrap(ExtendedCacheItemInterface $item)
     {
         $wrap = [
-          self::DRIVER_DATA_WRAPPER_INDEX => $item->get(),
-          self::DRIVER_TAGS_WRAPPER_INDEX => $item->getTags(),
-          self::DRIVER_EDATE_WRAPPER_INDEX => $item->getExpirationDate(),
+            self::DRIVER_DATA_WRAPPER_INDEX => $item->get(),
+            self::DRIVER_TAGS_WRAPPER_INDEX => $item->getTags(),
+            self::DRIVER_EDATE_WRAPPER_INDEX => $item->getExpirationDate(),
         ];
 
         if($this->config['itemDetailedDate']){

Braces +1 added lines, -1 removed lines

@@ -273,7 +273,7 @@
         
         if (!is_dir($path)) {
             throw new phpFastCacheDriverException("Can't read PATH:" . $path, 94);
-        }else{
+        } else{
             $size = Directory::dirSize($path);
         }
 

Spacing +14 added lines, -14 removed lines

@@ -52,7 +52,7 @@ 
         if (is_array($config_name)) {
             $this->config = array_merge($this->config, $config_name);
         } else {
-            $this->config[ $config_name ] = $value;
+            $this->config[$config_name] = $value;
         }
     }
 
@@ -164,16 +164,16 @@ 
           self::DRIVER_EDATE_WRAPPER_INDEX => $item->getExpirationDate(),
         ];
 
-        if($this->config['itemDetailedDate']){
-            $wrap[ self::DRIVER_MDATE_WRAPPER_INDEX ] = new \DateTime();
+        if ($this->config['itemDetailedDate']) {
+            $wrap[self::DRIVER_MDATE_WRAPPER_INDEX] = new \DateTime();
             /**
              * If the creation date exists
              * reuse it else set a new Date
              */
-            $wrap[ self::DRIVER_CDATE_WRAPPER_INDEX ] = $item->getCreationDate() ?: new \DateTime();
-        }else{
-            $wrap[ self::DRIVER_MDATE_WRAPPER_INDEX ] = null;
-            $wrap[ self::DRIVER_CDATE_WRAPPER_INDEX ] = null;
+            $wrap[self::DRIVER_CDATE_WRAPPER_INDEX] = $item->getCreationDate() ?: new \DateTime();
+        } else {
+            $wrap[self::DRIVER_MDATE_WRAPPER_INDEX] = null;
+            $wrap[self::DRIVER_CDATE_WRAPPER_INDEX] = null;
         }
 
         return $wrap;
@@ -185,7 +185,7 @@ 
      */
     public function driverUnwrapData(array $wrapper)
     {
-        return $wrapper[ self::DRIVER_DATA_WRAPPER_INDEX ];
+        return $wrapper[self::DRIVER_DATA_WRAPPER_INDEX];
     }
 
     /**
@@ -194,7 +194,7 @@ 
      */
     public function driverUnwrapTags(array $wrapper)
     {
-        return $wrapper[ self::DRIVER_TAGS_WRAPPER_INDEX ];
+        return $wrapper[self::DRIVER_TAGS_WRAPPER_INDEX];
     }
 
 
@@ -204,7 +204,7 @@ 
      */
     public function driverUnwrapEdate(array $wrapper)
     {
-        return $wrapper[ self::DRIVER_EDATE_WRAPPER_INDEX ];
+        return $wrapper[self::DRIVER_EDATE_WRAPPER_INDEX];
     }
 
     /**
@@ -213,7 +213,7 @@ 
      */
     public function driverUnwrapCdate(array $wrapper)
     {
-        return $wrapper[ self::DRIVER_CDATE_WRAPPER_INDEX ];
+        return $wrapper[self::DRIVER_CDATE_WRAPPER_INDEX];
     }
 
 
@@ -223,7 +223,7 @@ 
      */
     public function driverUnwrapMdate(array $wrapper)
     {
-        return $wrapper[ self::DRIVER_MDATE_WRAPPER_INDEX ];
+        return $wrapper[self::DRIVER_MDATE_WRAPPER_INDEX];
     }
 
     /**
@@ -248,7 +248,7 @@ 
          * on tags item, it can leads
          * to an infinite recursive calls
          */
-        if(strpos($item->getKey(), self::DRIVER_TAGS_KEY_PREFIX ) === 0){
+        if (strpos($item->getKey(), self::DRIVER_TAGS_KEY_PREFIX) === 0) {
             throw new \LogicException('Trying to set tag(s) to an Tag item index: ' . $item->getKey());
         }
 
@@ -291,7 +291,7 @@ 
         foreach ($tagsItems as $tagsItem) {
             $data = (array) $tagsItem->get();
 
-            unset($data[ $item->getKey() ]);
+            unset($data[$item->getKey()]);
             $tagsItem->set($data);
 
             /**

Unused Use Statements -2 removed lines

@@ -13,9 +13,7 @@
  */
 namespace phpFastCache\Core\Pool;
 
-use phpFastCache\Exceptions\phpFastCacheDriverException;
 use phpFastCache\Core\Item\ExtendedCacheItemInterface;
-use phpFastCache\Util\Directory;
 
 
 /**

src/phpFastCache/Util/MemcacheDriverCollisionDetectorTrait.php

Indentation +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
                 return true;
             } else if (constant($CONSTANT_NAME) !== $driverName) {
                 trigger_error('Memcache collision detected, you used both Memcache and Memcached driver in your script, this may leads to unexpected behaviours',
-                  E_USER_WARNING);
+                    E_USER_WARNING);
 
                 return false;
             }

src/phpFastCache/Drivers/Sqlite/Driver.php

Spacing +11 added lines, -11 removed lines

@@ -162,12 +162,12 @@ 
             $stm = $this->indexing->prepare("SELECT MAX(`db`) as `db` FROM `balancing`");
             $stm->execute();
             $row = $stm->fetch(PDO::FETCH_ASSOC);
-            if (!isset($row[ 'db' ])) {
+            if (!isset($row['db'])) {
                 $db = 1;
-            } elseif ($row[ 'db' ] <= 1) {
+            } elseif ($row['db'] <= 1) {
                 $db = 1;
             } else {
-                $db = $row[ 'db' ];
+                $db = $row['db'];
             }
 
             // check file size
@@ -189,8 +189,8 @@ 
           ':keyword' => $keyword,
         ]);
         $row = $stm->fetch(PDO::FETCH_ASSOC);
-        if (isset($row[ 'db' ]) && $row[ 'db' ] != '') {
-            $db = $row[ 'db' ];
+        if (isset($row['db']) && $row['db'] != '') {
+            $db = $row['db'];
         } else {
             /*
              * Insert new to Indexing
@@ -221,7 +221,7 @@ 
         /**
          * init instant
          */
-        if (!isset($this->instance[ $instant ])) {
+        if (!isset($this->instance[$instant])) {
             // check DB Files ready or not
             $createTable = false;
             if (!file_exists($this->SqliteDir . '/db' . $instant) || $reset == true) {
@@ -234,12 +234,12 @@ 
                 $this->initDB($PDO);
             }
 
-            $this->instance[ $instant ] = $PDO;
+            $this->instance[$instant] = $PDO;
             unset($PDO);
 
         }
 
-        return $this->instance[ $instant ];
+        return $this->instance[$instant];
     }
 
     /**
@@ -253,7 +253,7 @@ 
          * Check for Cross-Driver type confusion
          */
         if ($item instanceof Item) {
-            $skipExisting = isset($this->config[ 'skipExisting' ]) ? $this->config[ 'skipExisting' ] : false;
+            $skipExisting = isset($this->config['skipExisting']) ? $this->config['skipExisting'] : false;
             $toWrite = true;
 
             // check in cache first
@@ -329,8 +329,8 @@ 
             }
         }
 
-        if (isset($row[ 'object' ])) {
-            return $this->decode($row[ 'object' ]);
+        if (isset($row['object'])) {
+            return $this->decode($row['object']);
         }
 
         return null;

Doc Comments +2 added lines, -2 removed lines

@@ -209,7 +209,7 @@ 
     }
 
     /**
-     * @param $keyword
+     * @param string $keyword
      * @param bool $reset
      * @return PDO
      */
@@ -246,7 +246,7 @@ 
 
     /**
      * @param \Psr\Cache\CacheItemInterface $item
-     * @return mixed
+     * @return boolean
      * @throws \InvalidArgumentException
      */
     protected function driverWrite(CacheItemInterface $item)

Indentation +22 added lines, -22 removed lines

@@ -153,7 +153,7 @@ 
 
             $PDO = new PDO("sqlite:" . $this->SqliteDir . '/' . self::INDEXING_FILE);
             $PDO->setAttribute(PDO::ATTR_ERRMODE,
-              PDO::ERRMODE_EXCEPTION);
+                PDO::ERRMODE_EXCEPTION);
 
             if ($createTable == true) {
                 $this->initIndexing($PDO);
@@ -188,7 +188,7 @@ 
         // look for keyword
         $stm = $this->indexing->prepare("SELECT * FROM `balancing` WHERE `keyword`=:keyword LIMIT 1");
         $stm->execute([
-          ':keyword' => $keyword,
+            ':keyword' => $keyword,
         ]);
         $row = $stm->fetch(PDO::FETCH_ASSOC);
         if (isset($row[ 'db' ]) && $row[ 'db' ] != '') {
@@ -200,8 +200,8 @@ 
             $db = $this->currentDB;
             $stm = $this->indexing->prepare("INSERT INTO `balancing` (`keyword`,`db`) VALUES(:keyword, :db)");
             $stm->execute([
-              ':keyword' => $keyword,
-              ':db' => $db,
+                ':keyword' => $keyword,
+                ':db' => $db,
             ]);
         }
 
@@ -272,11 +272,11 @@ 
             if ($toWrite == true) {
                 try {
                     $stm = $this->getDb($item->getKey())
-                      ->prepare("INSERT OR REPLACE INTO `caching` (`keyword`,`object`,`exp`) values(:keyword,:object,:exp)");
+                        ->prepare("INSERT OR REPLACE INTO `caching` (`keyword`,`object`,`exp`) values(:keyword,:object,:exp)");
                     $stm->execute([
-                      ':keyword' => $item->getKey(),
-                      ':object' => $this->encode($this->driverPreWrap($item)),
-                      ':exp' => time() + $item->getTtl(),
+                        ':keyword' => $item->getKey(),
+                        ':object' => $this->encode($this->driverPreWrap($item)),
+                        ':exp' => time() + $item->getTtl(),
                     ]);
 
                     return true;
@@ -284,11 +284,11 @@ 
 
                     try {
                         $stm = $this->getDb($item->getKey(), true)
-                          ->prepare("INSERT OR REPLACE INTO `caching` (`keyword`,`object`,`exp`) values(:keyword,:object,:exp)");
+                            ->prepare("INSERT OR REPLACE INTO `caching` (`keyword`,`object`,`exp`) values(:keyword,:object,:exp)");
                         $stm->execute([
-                          ':keyword' => $item->getKey(),
-                          ':object' => $this->encode($this->driverPreWrap($item)),
-                          ':exp' => time() + $item->getTtl(),
+                            ':keyword' => $item->getKey(),
+                            ':object' => $this->encode($this->driverPreWrap($item)),
+                            ':exp' => time() + $item->getTtl(),
                         ]);
                     } catch (PDOException $e) {
                         return false;
@@ -310,20 +310,20 @@ 
     {
         try {
             $stm = $this->getDb($item->getKey())
-              ->prepare("SELECT * FROM `caching` WHERE `keyword`=:keyword AND (`exp` >= :U)  LIMIT 1");
+                ->prepare("SELECT * FROM `caching` WHERE `keyword`=:keyword AND (`exp` >= :U)  LIMIT 1");
             $stm->execute([
-              ':keyword' => $item->getKey(),
-              ':U' => time(),
+                ':keyword' => $item->getKey(),
+                ':U' => time(),
             ]);
             $row = $stm->fetch(PDO::FETCH_ASSOC);
 
         } catch (PDOException $e) {
             try {
                 $stm = $this->getDb($item->getKey(), true)
-                  ->prepare("SELECT * FROM `caching` WHERE `keyword`=:keyword AND (`exp` >= :U)  LIMIT 1");
+                    ->prepare("SELECT * FROM `caching` WHERE `keyword`=:keyword AND (`exp` >= :U)  LIMIT 1");
                 $stm->execute([
-                  ':keyword' => $item->getKey(),
-                  ':U' => time(),
+                    ':keyword' => $item->getKey(),
+                    ':U' => time(),
                 ]);
                 $row = $stm->fetch(PDO::FETCH_ASSOC);
             } catch (PDOException $e) {
@@ -351,13 +351,13 @@ 
         if ($item instanceof Item) {
             try {
                 $stm = $this->getDb($item->getKey())
-                  //->prepare("DELETE FROM `caching` WHERE (`id`=:id) OR (`exp` <= :U) ");
-                  ->prepare("DELETE FROM `caching` WHERE (`exp` <= :U) OR (`keyword`=:keyword) ");
+                    //->prepare("DELETE FROM `caching` WHERE (`id`=:id) OR (`exp` <= :U) ");
+                    ->prepare("DELETE FROM `caching` WHERE (`exp` <= :U) OR (`keyword`=:keyword) ");
 
                 return $stm->execute([
                     // ':id' => $row[ 'id' ],
-                  ':keyword' => $item->getKey(),
-                  ':U' => time(),
+                    ':keyword' => $item->getKey(),
+                    ':U' => time(),
                 ]);
             } catch (PDOException $e) {
                 return false;

tests/AbstractProxy.test.php

Spacing +2 added lines, -2 removed lines

@@ -42,10 +42,10 @@
 if (!is_object($driverInstance->getItem('test'))) {
     echo '[FAIL] $driverInstance->getItem() returned an invalid var type:' . gettype($driverInstance) . "\n";
     $status = 1;
-}else if(!($driverInstance->getItem('test') instanceof ExtendedCacheItemInterface)){
+} else if (!($driverInstance->getItem('test') instanceof ExtendedCacheItemInterface)) {
     echo '[FAIL] $driverInstance->getItem() returned an invalid class that does not implements ExtendedCacheItemInterface: ' . get_class($driverInstance) . "\n";
     $status = 1;
-}else{
+} else {
     echo '[PASS] $driverInstance->getItem() returned a valid class that implements ExtendedCacheItemInterface: ' . get_class($driverInstance) . "\n";
 }
 

Braces +2 added lines, -2 removed lines

@@ -42,10 +42,10 @@
 if (!is_object($driverInstance->getItem('test'))) {
     echo '[FAIL] $driverInstance->getItem() returned an invalid var type:' . gettype($driverInstance) . "\n";
     $status = 1;
-}else if(!($driverInstance->getItem('test') instanceof ExtendedCacheItemInterface)){
+} else if(!($driverInstance->getItem('test') instanceof ExtendedCacheItemInterface)){
     echo '[FAIL] $driverInstance->getItem() returned an invalid class that does not implements ExtendedCacheItemInterface: ' . get_class($driverInstance) . "\n";
     $status = 1;
-}else{
+} else{
     echo '[PASS] $driverInstance->getItem() returned a valid class that implements ExtendedCacheItemInterface: ' . get_class($driverInstance) . "\n";
 }
 

src/phpFastCache/Entities/driverStatistic.php

Doc Comments +3 added lines, -3 removed lines

@@ -52,7 +52,7 @@ 
     }
 
     /**
-     * @return int|bool Return size in octet or false if no information available
+     * @return string Return size in octet or false if no information available
      */
     public function getSize()
     {
@@ -60,7 +60,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     public function getData()
     {
@@ -68,7 +68,7 @@ 
     }
 
     /**
-     * @param $info
+     * @param string $info
      * @return $this
      */
     public function setInfo($info)

tests/AttachingDetachingMethods.test.php

Braces +4 added lines, -5 removed lines

@@ -24,10 +24,10 @@ 
 if (!is_object($driverInstance)) {
     echo '[FAIL] CacheManager::getInstance() returned an invalid variable type:' . gettype($driverInstance) . "\n";
     $status = 1;
-}else if(!($driverInstance instanceof CacheItemPoolInterface)){
+} else if(!($driverInstance instanceof CacheItemPoolInterface)){
     echo '[FAIL] CacheManager::getInstance() returned an invalid class:' . get_class($driverInstance) . "\n";
     $status = 1;
-}else{
+} else{
     $key = 'test_attaching_detaching';
 
     $itemDetached = $driverInstance->getItem($key);
@@ -37,8 +37,7 @@ 
     if(!$driverInstance->isAttached($itemDetached))
     {
         echo '[PASS] ExtendedCacheItemPoolInterface::isAttached() identified $itemDetached as being detached.' . "\n";
-    }
-    else
+    } else
     {
         echo '[FAIL] ExtendedCacheItemPoolInterface::isAttached() failed to identify $itemDetached as to be detached.' . "\n";
         $status = 1;
@@ -48,7 +47,7 @@ 
         $driverInstance->attachItem($itemDetached);
         echo '[FAIL] ExtendedCacheItemPoolInterface::attachItem() attached $itemDetached without trowing an error.' . "\n";
         $status = 1;
-    }catch(\LogicException $e){
+    } catch(\LogicException $e){
         echo '[PASS] ExtendedCacheItemPoolInterface::attachItem() failed to attach $itemDetached by trowing a LogicException exception.' . "\n";
     }
 

Spacing +5 added lines, -5 removed lines

@@ -24,17 +24,17 @@ 
 if (!is_object($driverInstance)) {
     echo '[FAIL] CacheManager::getInstance() returned an invalid variable type:' . gettype($driverInstance) . "\n";
     $status = 1;
-}else if(!($driverInstance instanceof CacheItemPoolInterface)){
+} else if (!($driverInstance instanceof CacheItemPoolInterface)) {
     echo '[FAIL] CacheManager::getInstance() returned an invalid class:' . get_class($driverInstance) . "\n";
     $status = 1;
-}else{
+} else {
     $key = 'test_attaching_detaching';
 
     $itemDetached = $driverInstance->getItem($key);
     $driverInstance->detachItem($itemDetached);
     $itemAttached = $driverInstance->getItem($key);
 
-    if($driverInstance->isAttached($itemDetached) !== true)
+    if ($driverInstance->isAttached($itemDetached) !== true)
     {
         echo '[PASS] ExtendedCacheItemPoolInterface::isAttached() identified $itemDetached as being detached.' . "\n";
     }
@@ -44,11 +44,11 @@ 
         $status = 1;
     }
 
-    try{
+    try {
         $driverInstance->attachItem($itemDetached);
         echo '[FAIL] ExtendedCacheItemPoolInterface::attachItem() attached $itemDetached without trowing an error.' . "\n";
         $status = 1;
-    }catch(\LogicException $e){
+    } catch (\LogicException $e) {
         echo '[PASS] ExtendedCacheItemPoolInterface::attachItem() failed to attach $itemDetached by trowing a LogicException exception.' . "\n";
     }