[Cache] fix warming up cache.system and apcu

nicolas-grekas · nicolas-grekas · commit 70b6994e71f2 · 2019-02-21T14:37:01.000+01:00
diff --git a/src/Symfony/Component/Cache/Adapter/AbstractAdapter.php b/src/Symfony/Component/Cache/Adapter/AbstractAdapter.php
@@ -108,16 +108,12 @@ function ($deferred, $namespace, &$expiredIds) use ($getId) {
      */
     public static function createSystemCache($namespace, $defaultLifetime, $version, $directory, LoggerInterface $logger = null)
     {
-        if (null === self::$apcuSupported) {
-            self::$apcuSupported = ApcuAdapter::isSupported();
+        $opcache = new PhpFilesAdapter($namespace, $defaultLifetime, $directory, true);
+        if (null !== $logger) {
+            $opcache->setLogger($logger);
         }
 
-        if (!self::$apcuSupported) {
-            $opcache = new PhpFilesAdapter($namespace, $defaultLifetime, $directory, true);
-            if (null !== $logger) {
-                $opcache->setLogger($logger);
-            }
-
+        if (!self::$apcuSupported = self::$apcuSupported ?? ApcuAdapter::isSupported()) {
             return $opcache;
         }
 
@@ -128,7 +124,7 @@ public static function createSystemCache($namespace, $defaultLifetime, $version,
             $apcu->setLogger($logger);
         }
 
-        return $apcu;
+        return new ChainAdapter([$apcu, $opcache]);
     }
 
     public static function createConnection($dsn, array $options = [])

Original file line number	Diff line number	Diff line change
`@@ -108,16 +108,12 @@ function ($deferred, $namespace, &$expiredIds) use ($getId) {`
`108`	`108`	`*/`
`109`	`109`	`public static function createSystemCache($namespace, $defaultLifetime, $version, $directory, LoggerInterface $logger = null)`
`110`	`110`	`{`
`111`		`- if (null === self::$apcuSupported) {`
`112`		`- self::$apcuSupported = ApcuAdapter::isSupported();`
	`111`	`+ $opcache = new PhpFilesAdapter($namespace, $defaultLifetime, $directory, true);`
	`112`	`+ if (null !== $logger) {`
	`113`	`+ $opcache->setLogger($logger);`
`113`	`114`	`}`
`114`	`115`
`115`		`- if (!self::$apcuSupported) {`
`116`		`- $opcache = new PhpFilesAdapter($namespace, $defaultLifetime, $directory, true);`
`117`		`- if (null !== $logger) {`
`118`		`- $opcache->setLogger($logger);`
`119`		`- }`
`120`		`-`
	`116`	`+ if (!self::$apcuSupported = self::$apcuSupported ?? ApcuAdapter::isSupported()) {`
`121`	`117`	`return $opcache;`
`122`	`118`	`}`
`123`	`119`
`@@ -128,7 +124,7 @@ public static function createSystemCache($namespace, $defaultLifetime, $version,`
`128`	`124`	`$apcu->setLogger($logger);`
`129`	`125`	`}`
`130`	`126`
`131`		`- return $apcu;`
	`127`	`+ return new ChainAdapter([$apcu, $opcache]);`
`132`	`128`	`}`
`133`	`129`
`134`	`130`	`public static function createConnection($dsn, array $options = [])`