Merge pull request #8 from ethaizone/patch-1

[Critical] Bug when model want to purge multi tags and key lose before send to delete.
spiritix · Mar 21, 2016 · 3133700 · 3133700
2 parents b988084 + 36cd596
commit 3133700
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 3 deletions.
diff --git a/src/Spiritix/LadaCache/Invalidator.php b/src/Spiritix/LadaCache/Invalidator.php
@@ -70,7 +70,7 @@ private function getHashes(array $tags)
                 continue;
             }
 
-            $hashes += $this->redis->smembers($tag);
+            $hashes = array_merge($hashes, $this->redis->smembers($tag));
         }
 
         return array_unique($hashes);
@@ -88,4 +88,4 @@ private function deleteItems(array $items)
             $this->redis->del($item);
         }
     }
-}
+}
diff --git a/tests/InvalidatorTest.php b/tests/InvalidatorTest.php
@@ -24,4 +24,14 @@ public function testInvalidate()
 
         $this->assertFalse($this->cache->has('key'));
     }
-}
+
+    public function testInvalidateMultiTags()
+    {
+        $this->cache->set('key1', ['tag1'], 'data');
+        $this->cache->set('key2', ['tag2'], 'data');
+
+        $this->invalidator->invalidate(['tag1', 'tag2']);
+
+        $this->assertFalse($this->cache->has('key2'));
+    }
+}