Merge pull request #318 from Geolim4/final

 Fixed bug on tags that are not working on the first cache write

Author: Geolim4

src/phpFastCache/Cache/DriverBaseTrait.php

@@ -214,9 +214,22 @@ public function getDriverName()
     /**
      * @param \phpFastCache\Cache\ExtendedCacheItemInterface $item
      * @return bool
+     * @throws \LogicException
      */
     public function driverWriteTags(ExtendedCacheItemInterface $item)
     {
+        /**
+         * Do not attempt to write tags
+         * on tags item, it can leads
+         * to an infinite recursive calls
+         */
+        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());
+        }
+
+        /**
+         * @var $tagsItems ExtendedCacheItemInterface[]
+         */
         $tagsItems = $this->getItems($this->getTagKeys($item->getTags()));
 
         foreach ($tagsItems as $tagsItem) {
@@ -241,6 +254,7 @@ public function driverWriteTags(ExtendedCacheItemInterface $item)
                 $tagsItem->expiresAt($item->getExpirationDate());
             }
             $this->driverWrite($tagsItem);
+            $tagsItem->setHit(true);
         }
 
         /**
@@ -265,6 +279,7 @@ public function driverWriteTags(ExtendedCacheItemInterface $item)
             if (count($data)) {
                 $tagsItem->expiresAt(max($data));
                 $this->driverWrite($tagsItem);
+                $tagsItem->setHit(true);
             } else {
                 $this->driverDelete($tagsItem);
             }