Merge pull request #325 from dmason30/master

Handle json contains with array value

Author: mikebronner

src/CacheKey.php

@@ -164,6 +164,10 @@ protected function getValuesFromWhere(array $where) : string
             return implode("_", collect($where["values"])->flatten()->toArray());
         }
 
+        if (is_array((new Arr)->get($where, "value"))) {
+            return implode("_", collect($where["value"])->flatten()->toArray());
+        }
+
         return (new Arr)->get($where, "value", "");
     }
 

tests/Integration/CachedBuilder/WhereJsonContainsTest.php

@@ -49,4 +49,27 @@ public function testWithInUsingCollectionQuery()
         $this->assertEquals($liveResults->pluck("id"), $authors->pluck("id"));
         $this->assertEquals($liveResults->pluck("id"), $cachedResults->pluck("id"));
     }
+
+    public function testWithInUsingCollectionQueryWithArrayValues()
+    {
+        $key = sha1("genealabs:laravel-model-caching:pgsql:testing:authors:genealabslaravelmodelcachingtestsfixturesauthor-finances->tags_jsoncontains_[\"foo\",\"bar\"]-authors.deleted_at_null");
+        $tags = [
+            'genealabs:laravel-model-caching:pgsql:testing:genealabslaravelmodelcachingtestsfixturesauthor',
+        ];
+
+        $authors = (new Author)
+            ->whereJsonContains("finances->tags", ['foo', 'bar'])
+            ->get();
+        $liveResults = (new UncachedAuthor)
+            ->whereJsonContains("finances->tags", ['foo', 'bar'])
+            ->get();
+
+        $cachedResults = $this
+            ->cache()
+            ->tags($tags)
+            ->get($key)['value'];
+
+        $this->assertEquals($liveResults->pluck("id"), $authors->pluck("id"));
+        $this->assertEquals($liveResults->pluck("id"), $cachedResults->pluck("id"));
+    }
 }

tests/database/factories/AuthorFactory.php

@@ -11,6 +11,7 @@
             "total" => 5000,
             "weekly" => 100,
             "daily" => 20,
+            'tags' => ['foo', 'bar'],
         ],
         "is_famous" => $faker->boolean(),
     ];