Merge pull request #772 from GeoSot/fix-inconsistency-on-strategy-usage

DeleteItemsByTags was ignoring strategy

Author: Geolim4

README.md

@@ -109,7 +109,7 @@ PhpFastCache provides you a lot of useful APIs:
 | `deleteItem($key)`                                              | bool                           | Deletes an item                                                                                  | 
 | `deleteItems(array $keys)`                                      | bool                           | Deletes one or more items                                                                        | 
 | `deleteItemsByTag($tagName)`                                    | bool                           | Deletes items by a tag                                                                           | 
-| `deleteItemsByTags(array $tagNames)`                            | bool                           | Deletes items  by one of multiple tag names                                                      | 
+| `deleteItemsByTags(array $tagNames, int $strategy)`                            | bool                           | Deletes items  by one of multiple tag names                                                      | 
 | `detachItem($item)`                                             | void                           | Detaches an item from the pool                                                                   | 
 | `getConfig()`                                                   | ConfigurationOption            | Returns the configuration object                                                                 | 
 | `getConfigOption($optionName);`                                 | mixed                          | Returns a configuration value by its key `$optionName`                                           | 

lib/Phpfastcache/Core/Pool/TaggableCacheItemPoolTrait.php

@@ -127,8 +127,9 @@ protected function getTagKey(string $key): string
     public function deleteItemsByTags(array $tagNames, int $strategy = TaggableCacheItemPoolInterface::TAG_STRATEGY_ONE): bool
     {
         $return = null;
-        foreach ($tagNames as $tagName) {
-            $result = $this->deleteItemsByTag($tagName, $strategy);
+
+        foreach ($this->getItemsByTags($tagNames, $strategy) as $item) {
+            $result = $this->deleteItem($item->getKey());
             if ($return !== false) {
                 $return = $result;
             }