PHPSocialNetwork / phpfastcache

src/phpFastCache/Core/Pool/CacheItemPoolTrait.php

Doc Comments +2 added lines, -2 removed lines

@@ -157,7 +157,7 @@ 
 
     /**
      * @param \Psr\Cache\CacheItemInterface $item
-     * @return mixed
+     * @return boolean
      * @throws \InvalidArgumentException
      */
     public function save(CacheItemInterface $item)
@@ -175,7 +175,7 @@ 
     }
 
     /**
-     * @return mixed|null
+     * @return boolean
      * @throws \InvalidArgumentException
      */
     public function commit()

Braces +2 added lines, -2 removed lines

@@ -85,7 +85,7 @@ 
                     } else {
                         $item->setHit(true);
                     }
-                }else{
+                } else{
                     $item->expiresAfter(abs((int) $this->getConfig()[ 'defaultTtl' ]));
                 }
 
@@ -229,7 +229,7 @@ 
     {
         if (!array_key_exists($item->getKey(), $this->itemInstances)) {
             $this->itemInstances[ $item->getKey() ] = $item;
-        }else if(spl_object_hash($item) !== spl_object_hash($this->itemInstances[ $item->getKey() ])){
+        } else if(spl_object_hash($item) !== spl_object_hash($this->itemInstances[ $item->getKey() ])){
             throw new \RuntimeException('Spl object hash mismatches ! You probably tried to save a detached item which has been already retrieved from cache.');
         }
 

Spacing +14 added lines, -14 removed lines

@@ -67,13 +67,13 @@ 
                 $driverArray = $this->driverRead($item);
 
                 if ($driverArray) {
-                    if(!is_array($driverArray)){
+                    if (!is_array($driverArray)) {
                         throw new phpFastCacheCoreException(sprintf('The driverRead method returned an unexpected variable type: %s', gettype($driverArray)));
                     }
                     $item->set($this->driverUnwrapData($driverArray));
                     $item->expiresAt($this->driverUnwrapEdate($driverArray));
 
-                    if($this->config['itemDetailedDate']){
+                    if ($this->config['itemDetailedDate']) {
 
                         /**
                          * If the itemDetailedDate has been
@@ -97,8 +97,8 @@ 
                     } else {
                         $item->setHit(true);
                     }
-                }else{
-                    $item->expiresAfter(abs((int) $this->getConfig()[ 'defaultTtl' ]));
+                } else {
+                    $item->expiresAfter(abs((int) $this->getConfig()['defaultTtl']));
                 }
 
             }
@@ -111,9 +111,9 @@ 
          * @param $this ExtendedCacheItemPoolInterface
          * @param $this ExtendedCacheItemInterface
          */
-        $this->eventManager->dispatch('CacheGetItem', $this, $this->itemInstances[ $key ]);
+        $this->eventManager->dispatch('CacheGetItem', $this, $this->itemInstances[$key]);
 
-        return $this->itemInstances[ $key ];
+        return $this->itemInstances[$key];
     }
 
     /**
@@ -124,7 +124,7 @@ 
     public function setItem(CacheItemInterface $item)
     {
         if ($this->getClassNamespace() . '\\Item' === get_class($item)) {
-            $this->itemInstances[ $item->getKey() ] = $item;
+            $this->itemInstances[$item->getKey()] = $item;
 
             return $this;
         } else {
@@ -141,7 +141,7 @@ 
     {
         $collection = [];
         foreach ($keys as $key) {
-            $collection[ $key ] = $this->getItem($key);
+            $collection[$key] = $this->getItem($key);
         }
 
         return $collection;
@@ -230,8 +230,8 @@ 
          * @var ExtendedCacheItemInterface $item
          */
         if (!array_key_exists($item->getKey(), $this->itemInstances)) {
-            $this->itemInstances[ $item->getKey() ] = $item;
-        } else if(spl_object_hash($item) !== spl_object_hash($this->itemInstances[ $item->getKey() ])){
+            $this->itemInstances[$item->getKey()] = $item;
+        } else if (spl_object_hash($item) !== spl_object_hash($this->itemInstances[$item->getKey()])) {
             throw new \RuntimeException('Spl object hash mismatches ! You probably tried to save a detached item which has been already retrieved from cache.');
         }
 
@@ -261,8 +261,8 @@ 
     public function saveDeferred(CacheItemInterface $item)
     {
         if (!array_key_exists($item->getKey(), $this->itemInstances)) {
-            $this->itemInstances[ $item->getKey() ] = $item;
-        }else if(spl_object_hash($item) !== spl_object_hash($this->itemInstances[ $item->getKey() ])){
+            $this->itemInstances[$item->getKey()] = $item;
+        } else if (spl_object_hash($item) !== spl_object_hash($this->itemInstances[$item->getKey()])) {
             throw new \RuntimeException('Spl object hash mismatches ! You probably tried to save a detached item which has been already retrieved from cache.');
         }
 
@@ -273,7 +273,7 @@ 
          */
         $this->eventManager->dispatch('CacheSaveDeferredItem', $this, $item);
 
-        return $this->deferredList[ $item->getKey() ] = $item;
+        return $this->deferredList[$item->getKey()] = $item;
     }
 
     /**
@@ -293,7 +293,7 @@ 
         foreach ($this->deferredList as $key => $item) {
             $result = $this->save($item);
             if ($return !== false) {
-                unset($this->deferredList[ $key ]);
+                unset($this->deferredList[$key]);
                 $return = $result;
             }
         }

src/phpFastCache/Drivers/Devfalse/Driver.php

Doc Comments +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
 
     /**
      * @param \Psr\Cache\CacheItemInterface $item
-     * @return mixed
+     * @return boolean
      * @throws \InvalidArgumentException
      */
     public function driverWrite(CacheItemInterface $item)

Indentation +6 added lines, -6 removed lines

@@ -79,9 +79,9 @@ 
     protected function driverRead(CacheItemInterface $item)
     {
         return [
-          self::DRIVER_DATA_WRAPPER_INDEX => false,
-          self::DRIVER_TAGS_WRAPPER_INDEX => [],
-          self::DRIVER_TIME_WRAPPER_INDEX => new \DateTime(),
+            self::DRIVER_DATA_WRAPPER_INDEX => false,
+            self::DRIVER_TAGS_WRAPPER_INDEX => [],
+            self::DRIVER_TIME_WRAPPER_INDEX => new \DateTime(),
         ];
     }
     /**
@@ -130,9 +130,9 @@ 
     {
         $stat = new driverStatistic();
         $stat->setInfo('[Devfalse] A void info string')
-          ->setSize(0)
-          ->setData(implode(', ', array_keys($this->itemInstances)))
-          ->setRawData(false);
+            ->setSize(0)
+            ->setData(implode(', ', array_keys($this->itemInstances)))
+            ->setRawData(false);
 
         return $stat;
     }

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;
             }

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)