Merge pull request #12400 from isaackaara/fix/cache-mode-refresh-put

Fix Cache::MODE_REFRESH not writing to query cache

Author: greg0ire

src/Cache/DefaultQueryCache.php

@@ -215,7 +215,7 @@ public function put(QueryCacheKey $key, ResultSetMapping $rsm, mixed $result, ar
             throw FeatureNotImplemented::partialEntities();
         }
 
-        if (! ($key->cacheMode & Cache::MODE_PUT)) {
+        if (! ($key->cacheMode & Cache::MODE_PUT) && ! ($key->cacheMode & Cache::MODE_REFRESH)) {
             return false;
         }
 

tests/Tests/ORM/Cache/DefaultQueryCacheTest.php

@@ -496,6 +496,28 @@ public function testIgnoreCacheNonPutMode(): void
         self::assertFalse($this->queryCache->put($key, $rsm, $result));
     }
 
+    public function testRefreshModeWritesToCache(): void
+    {
+        $result   = [];
+        $rsm      = new ResultSetMappingBuilder($this->em);
+        $metadata = $this->em->getClassMetadata(Country::class);
+        $key      = new QueryCacheKey('query.key1', 0, Cache::MODE_REFRESH);
+
+        $rsm->addRootEntityFromClassMetadata(Country::class, 'c');
+
+        for ($i = 0; $i < 2; $i++) {
+            $name     = 'Country ' . $i;
+            $entity   = new Country($name);
+            $result[] = $entity;
+
+            $metadata->setFieldValue($entity, 'id', $i);
+            $this->em->getUnitOfWork()->registerManaged($entity, ['id' => $i], ['name' => $name]);
+        }
+
+        self::assertTrue($this->queryCache->put($key, $rsm, $result));
+        self::assertArrayHasKey('put', $this->region->calls);
+    }
+
     public function testGetShouldIgnoreOldQueryCacheEntryResult(): void
     {
         $rsm      = new ResultSetMappingBuilder($this->em);