[Cache] Fix calling the callback wrapper for ChainAdapter

nicolas-grekas · nicolas-grekas · commit b3bc0c35f96e · 2025-12-16T21:15:46.000Z
diff --git a/src/Symfony/Component/Cache/Adapter/ChainAdapter.php b/src/Symfony/Component/Cache/Adapter/ChainAdapter.php
@@ -106,7 +106,7 @@ public function get(string $key, callable $callback, ?float $beta = null, ?array
                 $callback = $wrap;
                 $beta = \INF === $beta ? \INF : 0;
             }
-            if ($adapter instanceof CacheInterface) {
+            if ($adapter instanceof CacheInterface && $i !== $this->adapterCount) {
                 $value = $adapter->get($key, $callback, $beta, $metadata);
             } else {
                 $value = $this->doGet($adapter, $key, $callback, $beta, $metadata);
diff --git a/src/Symfony/Component/Cache/Tests/Adapter/ChainAdapterTest.php b/src/Symfony/Component/Cache/Tests/Adapter/ChainAdapterTest.php
@@ -266,4 +266,28 @@ private function getNonPruneableMock(): AdapterInterface
     {
         return $this->createMock(AdapterInterface::class);
     }
+
+    public function testSetCallbackWrapperPropagation()
+    {
+        $adapter1 = new ArrayAdapter();
+        $adapter2 = new FilesystemAdapter();
+
+        $chain = new ChainAdapter([$adapter1, $adapter2]);
+
+        $callbackWrapperCalled = false;
+        $customWrapper = static function (callable $callback, CacheItem $item, bool &$save) use (&$callbackWrapperCalled) {
+            $callbackWrapperCalled = true;
+
+            return $callback($item, $save);
+        };
+
+        $chain->setCallbackWrapper($customWrapper);
+
+        $chain->delete('test-callback-wrapper');
+
+        $result = $chain->get('test-callback-wrapper', static fn () => 'computed-value');
+
+        $this->assertTrue($callbackWrapperCalled);
+        $this->assertSame('computed-value', $result);
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -106,7 +106,7 @@ public function get(string $key, callable $callback, ?float $beta = null, ?array`
`106`	`106`	`$callback = $wrap;`
`107`	`107`	`$beta = \INF === $beta ? \INF : 0;`
`108`	`108`	`}`
`109`		`- if ($adapter instanceof CacheInterface) {`
	`109`	`+ if ($adapter instanceof CacheInterface && $i !== $this->adapterCount) {`
`110`	`110`	`$value = $adapter->get($key, $callback, $beta, $metadata);`
`111`	`111`	`} else {`
`112`	`112`	`$value = $this->doGet($adapter, $key, $callback, $beta, $metadata);`