PHPSocialNetwork / phpfastcache

src/phpFastCache/Core/Pool/ExtendedCacheItemPoolTrait.php

Doc Comments +2 added lines, -1 removed lines

@@ -46,6 +46,7 @@ 
 
     /**
      * @inheritdoc
+     * @param string $tagName
      */
     public function getItemsByTag($tagName)
     {
@@ -416,7 +417,7 @@ 
 
     /**
      * @internal This method de-register an item from $this->itemInstances
-     * @param CacheItemInterface|string $item
+     * @param CacheItemInterface $item
      * @throws \InvalidArgumentException
      */
     protected function deregisterItem($item)

Spacing +16 added lines, -16 removed lines

@@ -38,7 +38,7 @@ 
      */
     public function getItemsAsJsonString(array $keys = [], $option = 0, $depth = 512)
     {
-        $callback = function (CacheItemInterface $item) {
+        $callback = function(CacheItemInterface $item) {
             return $item->get();
         };
         return json_encode(array_map($callback, array_values($this->getItems($keys))), $option, $depth);
@@ -52,7 +52,7 @@ 
         if (is_string($tagName)) {
             $driverResponse = $this->getItem($this->getTagKey($tagName));
             if ($driverResponse->isHit()) {
-                $items = (array)$driverResponse->get();
+                $items = (array) $driverResponse->get();
 
                 /**
                  * getItems() may provides expired item(s)
@@ -64,7 +64,7 @@ 
                  *
                  * #headache
                  */
-                return array_filter($this->getItems(array_unique(array_keys($items))), function (ExtendedCacheItemInterface $item) {
+                return array_filter($this->getItems(array_unique(array_keys($items))), function(ExtendedCacheItemInterface $item) {
                     return $item->isHit();
                 });
             } else {
@@ -82,9 +82,9 @@ 
     {
         $items = [];
         foreach (array_unique($tagNames) as $tagName) {
-            if(is_string($tagName)){
+            if (is_string($tagName)) {
                 $items = array_merge($items, $this->getItemsByTag($tagName));
-            }else{
+            } else {
                 throw new InvalidArgumentException('$tagName must be a a string');
             }
         }
@@ -102,7 +102,7 @@ 
 
         foreach ($items as $key => $item) {
             if (array_diff($tagNames, $item->getTags())) {
-                unset($items[ $key ]);
+                unset($items[$key]);
             }
         }
 
@@ -115,7 +115,7 @@ 
      */
     public function getItemsByTagsAsJsonString(array $tagNames, $option = 0, $depth = 512)
     {
-        $callback = function (CacheItemInterface $item) {
+        $callback = function(CacheItemInterface $item) {
             return $item->get();
         };
 
@@ -386,7 +386,7 @@ 
      */
     public function detachItem(CacheItemInterface $item)
     {
-        if (isset($this->itemInstances[ $item->getKey() ])) {
+        if (isset($this->itemInstances[$item->getKey()])) {
             $this->deregisterItem($item);
         }
     }
@@ -406,10 +406,10 @@ 
      */
     public function attachItem(CacheItemInterface $item)
     {
-        if (isset($this->itemInstances[ $item->getKey() ]) && spl_object_hash($item) !== spl_object_hash($this->itemInstances[ $item->getKey() ])) {
+        if (isset($this->itemInstances[$item->getKey()]) && spl_object_hash($item) !== spl_object_hash($this->itemInstances[$item->getKey()])) {
             throw new \LogicException('The item already exists and cannot be overwritten because the Spl object hash mismatches ! You probably tried to re-attach a detached item which has been already retrieved from cache.');
         } else {
-            $this->itemInstances[ $item->getKey() ] = $item;
+            $this->itemInstances[$item->getKey()] = $item;
         }
     }
 
@@ -422,10 +422,10 @@ 
     protected function deregisterItem($item)
     {
         if ($item instanceof CacheItemInterface) {
-            unset($this->itemInstances[ $item->getKey() ]);
+            unset($this->itemInstances[$item->getKey()]);
 
         } else if (is_string($item)) {
-            unset($this->itemInstances[ $item ]);
+            unset($this->itemInstances[$item]);
         } else {
             throw new \InvalidArgumentException('Invalid type for $item variable');
         }
@@ -445,8 +445,8 @@ 
      */
     public function isAttached(CacheItemInterface $item)
     {
-        if (isset($this->itemInstances[ $item->getKey() ])) {
-            return spl_object_hash($item) === spl_object_hash($this->itemInstances[ $item->getKey() ]);
+        if (isset($this->itemInstances[$item->getKey()])) {
+            return spl_object_hash($item) === spl_object_hash($this->itemInstances[$item->getKey()]);
         }
         return null;
     }
@@ -466,8 +466,8 @@ 
      */
     public function saveMultiple(...$items)
     {
-        if (isset($items[ 0 ]) && is_array($items[ 0 ])) {
-            foreach ($items[ 0 ] as $item) {
+        if (isset($items[0]) && is_array($items[0])) {
+            foreach ($items[0] as $item) {
                 $this->save($item);
             }
             return true;

Braces +1 added lines, -1 removed lines

@@ -84,7 +84,7 @@
         foreach (array_unique($tagNames) as $tagName) {
             if(is_string($tagName)){
                 $items = array_merge($items, $this->getItemsByTag($tagName));
-            }else{
+            } else{
                 throw new InvalidArgumentException('$tagName must be a a string');
             }
         }

tests/ItemTags.test.php

Indentation +22 added lines, -22 removed lines

@@ -28,36 +28,36 @@
     $item3 = $driverInstance->getItem('tag-test3');
 
     $item1->set('item-test_1')
-      ->expiresAfter(600)
-      ->addTag('tag-test_1')
-      ->addTag('tag-test_all')
-      ->addTag('tag-test_all2')
-      ->addTag('tag-test_all3');
+        ->expiresAfter(600)
+        ->addTag('tag-test_1')
+        ->addTag('tag-test_all')
+        ->addTag('tag-test_all2')
+        ->addTag('tag-test_all3');
 
     $item2->set('item-test_2')
-      ->expiresAfter(600)
-      ->addTag('tag-test_1')
-      ->addTag('tag-test_2')
-      ->addTag('tag-test_all')
-      ->addTag('tag-test_all2')
-      ->addTag('tag-test_all3');
+        ->expiresAfter(600)
+        ->addTag('tag-test_1')
+        ->addTag('tag-test_2')
+        ->addTag('tag-test_all')
+        ->addTag('tag-test_all2')
+        ->addTag('tag-test_all3');
 
     $item3->set('item-test_3')
-      ->expiresAfter(600)
-      ->addTag('tag-test_1')
-      ->addTag('tag-test_2')
-      ->addTag('tag-test_3')
-      ->addTag('tag-test_all')
-      ->addTag('tag-test_all2')
-      ->addTag('tag-test_all3')
-      ->addTag('tag-test_all4');
+        ->expiresAfter(600)
+        ->addTag('tag-test_1')
+        ->addTag('tag-test_2')
+        ->addTag('tag-test_3')
+        ->addTag('tag-test_all')
+        ->addTag('tag-test_all2')
+        ->addTag('tag-test_all3')
+        ->addTag('tag-test_all4');
 
     $driverInstance->saveMultiple($item1, $item2, $item3);
 
     return [
-      'item1' => $item1,
-      'item2' => $item2,
-      'item3' => $item3
+        'item1' => $item1,
+        'item2' => $item2,
+        'item3' => $item3
     ];
 };
 

Spacing +34 added lines, -34 removed lines

@@ -11,7 +11,7 @@ 
 require_once __DIR__ . '/../src/autoload.php';
 echo '[PhpFastCache API v' . Api::getVersion() . "]\n\n";
 
-$defaultDriver = (!empty($argv[ 1 ]) ? ucfirst($argv[ 1 ]) : 'Files');
+$defaultDriver = (!empty($argv[1]) ? ucfirst($argv[1]) : 'Files');
 $status = 0;
 echo "Testing items tags feature\n";
 
@@ -69,14 +69,14 @@ 
 echo "\n#1 Testing getter: getItemsByTag() // Expecting 3 results\n";
 
 $tagsItems = $driverInstance->getItemsByTag('tag-test_all');
-if(is_array($tagsItems))
+if (is_array($tagsItems))
 {
-    if(count($tagsItems) === 3)
+    if (count($tagsItems) === 3)
     {
-        foreach($tagsItems as $tagsItem)
+        foreach ($tagsItems as $tagsItem)
         {
-            if(!in_array($tagsItem->getKey(), ['tag-test1', 'tag-test2', 'tag-test3'])){
-                echo '[FAIL] STEP#1 // Got unexpected tagged item key:' . $tagsItem->getKey()  .  "\n";
+            if (!in_array($tagsItem->getKey(), ['tag-test1', 'tag-test2', 'tag-test3'])) {
+                echo '[FAIL] STEP#1 // Got unexpected tagged item key:' . $tagsItem->getKey() . "\n";
                 $status = 1;
                 goto itemTagTest2;
             }
@@ -85,14 +85,14 @@ 
     }
     else
     {
-        echo '[FAIL] STEP#1 //Got wrong count of item:' . count($tagsItems) .  "\n";
+        echo '[FAIL] STEP#1 //Got wrong count of item:' . count($tagsItems) . "\n";
         $status = 1;
         goto itemTagTest2;
     }
 }
 else
 {
-    echo '[FAIL] STEP#1 // Expected $tagsItems to be an array, got: ' . gettype($tagsItems) .  "\n";
+    echo '[FAIL] STEP#1 // Expected $tagsItems to be an array, got: ' . gettype($tagsItems) . "\n";
     $status = 1;
     goto itemTagTest2;
 }
@@ -107,14 +107,14 @@ 
 
 $tagsItems = $driverInstance->getItemsByTagsAll(['tag-test_all', 'tag-test_all2', 'tag-test_all3']);
 
-if(is_array($tagsItems))
+if (is_array($tagsItems))
 {
-    if(count($tagsItems) === 3)
+    if (count($tagsItems) === 3)
     {
-        foreach($tagsItems as $tagsItem)
+        foreach ($tagsItems as $tagsItem)
         {
-            if(!in_array($tagsItem->getKey(), ['tag-test1', 'tag-test2', 'tag-test3'])){
-                echo '[FAIL] STEP#2 // Got unexpected tagged item key:' . $tagsItem->getKey()  .  "\n";
+            if (!in_array($tagsItem->getKey(), ['tag-test1', 'tag-test2', 'tag-test3'])) {
+                echo '[FAIL] STEP#2 // Got unexpected tagged item key:' . $tagsItem->getKey() . "\n";
                 $status = 1;
                 goto itemTagTest3;
             }
@@ -123,14 +123,14 @@ 
     }
     else
     {
-        echo '[FAIL] STEP#2 // Got wrong count of item:' . count($tagsItems) .  "\n";
+        echo '[FAIL] STEP#2 // Got wrong count of item:' . count($tagsItems) . "\n";
         $status = 1;
         goto itemTagTest3;
     }
 }
 else
 {
-    echo '[FAIL] STEP#2 // Expected $tagsItems to be an array, got: ' . gettype($tagsItems) .  "\n";
+    echo '[FAIL] STEP#2 // Expected $tagsItems to be an array, got: ' . gettype($tagsItems) . "\n";
     $status = 1;
     goto itemTagTest3;
 }
@@ -145,14 +145,14 @@ 
 
 $tagsItems = $driverInstance->getItemsByTagsAll(['tag-test_all', 'tag-test_all2', 'tag-test_all3', 'tag-test_all4']);
 
-if(is_array($tagsItems))
+if (is_array($tagsItems))
 {
-    if(count($tagsItems) === 1)
+    if (count($tagsItems) === 1)
     {
-        if(isset($tagsItems['tag-test3']))
+        if (isset($tagsItems['tag-test3']))
         {
-            if($tagsItems['tag-test3']->getKey() !== 'tag-test3'){
-                echo '[FAIL] STEP#3 // Got unexpected tagged item key:' . $tagsItems['tag-test3']->getKey()  .  "\n";
+            if ($tagsItems['tag-test3']->getKey() !== 'tag-test3') {
+                echo '[FAIL] STEP#3 // Got unexpected tagged item key:' . $tagsItems['tag-test3']->getKey() . "\n";
                 $status = 1;
                 goto itemTagTest4;
             }
@@ -160,21 +160,21 @@ 
         }
         else
         {
-            echo '[FAIL] STEP#3 // Got wrong array key, expected "tag-test3", got "' . key($tagsItems) .  "\"\n";
+            echo '[FAIL] STEP#3 // Got wrong array key, expected "tag-test3", got "' . key($tagsItems) . "\"\n";
             $status = 1;
             goto itemTagTest4;
         }
     }
     else
     {
-        echo '[FAIL] STEP#3 // Got wrong count of item:' . count($tagsItems) .  "\n";
+        echo '[FAIL] STEP#3 // Got wrong count of item:' . count($tagsItems) . "\n";
         $status = 1;
         goto itemTagTest4;
     }
 }
 else
 {
-    echo '[FAIL] STEP#3 // Expected $tagsItems to be an array, got: ' . gettype($tagsItems) .  "\n";
+    echo '[FAIL] STEP#3 // Expected $tagsItems to be an array, got: ' . gettype($tagsItems) . "\n";
     $status = 1;
     goto itemTagTest4;
 }
@@ -188,7 +188,7 @@ 
 $createItemsCallback();
 $driverInstance->deleteItemsByTag('tag-test_all');
 
-if(count($driverInstance->getItemsByTag('tag-test_all')) > 0)
+if (count($driverInstance->getItemsByTag('tag-test_all')) > 0)
 {
     echo "[FAIL] STEP#4 // Getter getItemsByTag() found item(s), possible memory leak \n";
     $status = 1;
@@ -199,9 +199,9 @@ 
 }
 
 $i = 0;
-while(++$i <= 3)
+while (++$i <= 3)
 {
-    if($driverInstance->getItem("tag-test{$i}")->isHit())
+    if ($driverInstance->getItem("tag-test{$i}")->isHit())
     {
         echo "[FAIL] STEP#4 // Item 'tag-test{$i}' should've been deleted and is still in cache storage \n";
         $status = 1;
@@ -222,7 +222,7 @@ 
 
 $driverInstance->deleteItemsByTagsAll(['tag-test_all', 'tag-test_all2', 'tag-test_all3']);
 
-if(count($driverInstance->getItemsByTag('tag-test_all')) > 0)
+if (count($driverInstance->getItemsByTag('tag-test_all')) > 0)
 {
     echo "[FAIL] STEP#5 // Getter getItemsByTag() found item(s), possible memory leak \n";
     $status = 1;
@@ -233,9 +233,9 @@ 
 }
 
 $i = 0;
-while(++$i <= 3)
+while (++$i <= 3)
 {
-    if($driverInstance->getItem("tag-test{$i}")->isHit())
+    if ($driverInstance->getItem("tag-test{$i}")->isHit())
     {
         echo "[FAIL] STEP#5 // Item 'tag-test{$i}' should've been deleted and is still in cache storage \n";
         $status = 1;
@@ -259,7 +259,7 @@ 
  */
 $driverInstance->deleteItemsByTagsAll(['tag-test_all', 'tag-test_all2', 'tag-test_all3', 'tag-test_all4']);
 
-if($driverInstance->getItem('item-test_3')->isHit())
+if ($driverInstance->getItem('item-test_3')->isHit())
 {
     echo "[FAIL] STEP#6 // Getter getItem() found item 'item-test_3', possible memory leak \n";
     $status = 1;
@@ -279,9 +279,9 @@ 
 $appendStr = '$*#*$';
 $driverInstance->appendItemsByTagsAll(['tag-test_all', 'tag-test_all2', 'tag-test_all3'], $appendStr);
 
-foreach($driverInstance->getItems(['tag-test1', 'tag-test2', 'tag-test3']) as $item)
+foreach ($driverInstance->getItems(['tag-test1', 'tag-test2', 'tag-test3']) as $item)
 {
-    if(strpos($item->get(), $appendStr) === false)
+    if (strpos($item->get(), $appendStr) === false)
     {
         echo "[FAIL] STEP#7 // Item '{$item->getKey()}' does not have the string part '{$appendStr}' in it's value \n";
         $status = 1;
@@ -302,9 +302,9 @@ 
 $prependStr = '&+_+&';
 $driverInstance->prependItemsByTagsAll(['tag-test_all', 'tag-test_all2', 'tag-test_all3'], $prependStr);
 
-foreach($driverInstance->getItems(['tag-test1', 'tag-test2', 'tag-test3']) as $item)
+foreach ($driverInstance->getItems(['tag-test1', 'tag-test2', 'tag-test3']) as $item)
 {
-    if(strpos($item->get(), $prependStr) === false)
+    if (strpos($item->get(), $prependStr) === false)
     {
         echo "[FAIL] STEP#7 // Item '{$item->getKey()}' does not have the string part '{$prependStr}' in it's value \n";
         $status = 1;

Braces +14 added lines, -28 removed lines

@@ -82,15 +82,13 @@ 
             }
         }
         echo "[PASS] STEP#1 // Successfully retrieved 3 tagged item keys\n";
-    }
-    else
+    } else
     {
         echo '[FAIL] STEP#1 //Got wrong count of item:' . count($tagsItems) .  "\n";
         $status = 1;
         goto itemTagTest2;
     }
-}
-else
+} else
 {
     echo '[FAIL] STEP#1 // Expected $tagsItems to be an array, got: ' . gettype($tagsItems) .  "\n";
     $status = 1;
@@ -120,15 +118,13 @@ 
             }
         }
         echo "[PASS] STEP#2 // Successfully retrieved 3 tagged item key\n";
-    }
-    else
+    } else
     {
         echo '[FAIL] STEP#2 // Got wrong count of item:' . count($tagsItems) .  "\n";
         $status = 1;
         goto itemTagTest3;
     }
-}
-else
+} else
 {
     echo '[FAIL] STEP#2 // Expected $tagsItems to be an array, got: ' . gettype($tagsItems) .  "\n";
     $status = 1;
@@ -157,22 +153,19 @@ 
                 goto itemTagTest4;
             }
             echo "[PASS] STEP#3 // Successfully retrieved 1 tagged item keys\n";
-        }
-        else
+        } else
         {
             echo '[FAIL] STEP#3 // Got wrong array key, expected "tag-test3", got "' . key($tagsItems) .  "\"\n";
             $status = 1;
             goto itemTagTest4;
         }
-    }
-    else
+    } else
     {
         echo '[FAIL] STEP#3 // Got wrong count of item:' . count($tagsItems) .  "\n";
         $status = 1;
         goto itemTagTest4;
     }
-}
-else
+} else
 {
     echo '[FAIL] STEP#3 // Expected $tagsItems to be an array, got: ' . gettype($tagsItems) .  "\n";
     $status = 1;
@@ -192,8 +185,7 @@ 
 {
     echo "[FAIL] STEP#4 // Getter getItemsByTag() found item(s), possible memory leak \n";
     $status = 1;
-}
-else
+} else
 {
     echo "[PASS] STEP#4 // Getter getItemsByTag() found no item \n";
 }
@@ -205,8 +197,7 @@ 
     {
         echo "[FAIL] STEP#4 // Item 'tag-test{$i}' should've been deleted and is still in cache storage \n";
         $status = 1;
-    }
-    else
+    } else
     {
         echo "[PASS] STEP#4 // Item 'tag-test{$i}' have been deleted and is no longer in cache storage \n";
     }
@@ -226,8 +217,7 @@ 
 {
     echo "[FAIL] STEP#5 // Getter getItemsByTag() found item(s), possible memory leak \n";
     $status = 1;
-}
-else
+} else
 {
     echo "[PASS] STEP#5 // Getter getItemsByTag() found no item \n";
 }
@@ -239,8 +229,7 @@ 
     {
         echo "[FAIL] STEP#5 // Item 'tag-test{$i}' should've been deleted and is still in cache storage \n";
         $status = 1;
-    }
-    else
+    } else
     {
         echo "[PASS] STEP#5 // Item 'tag-test{$i}' have been deleted and is no longer in cache storage \n";
     }
@@ -263,8 +252,7 @@ 
 {
     echo "[FAIL] STEP#6 // Getter getItem() found item 'item-test_3', possible memory leak \n";
     $status = 1;
-}
-else
+} else
 {
     echo "[PASS] STEP#6 // Getter getItem() did not found item 'item-test_3' \n";
 }
@@ -285,8 +273,7 @@ 
     {
         echo "[FAIL] STEP#7 // Item '{$item->getKey()}' does not have the string part '{$appendStr}' in it's value \n";
         $status = 1;
-    }
-    else
+    } else
     {
         echo "[PASS] STEP#7 // Item 'tag-test{$item->getKey()}' does have the string part '{$appendStr}' in it's value \n";
     }
@@ -308,8 +295,7 @@ 
     {
         echo "[FAIL] STEP#7 // Item '{$item->getKey()}' does not have the string part '{$prependStr}' in it's value \n";
         $status = 1;
-    }
-    else
+    } else
     {
         echo "[PASS] STEP#7 // Item 'tag-test{$item->getKey()}' does have the string part '{$prependStr}' in it's value \n";
     }