Deprecate doctrine/cache in favor of psr/cache

composer.json

@@ -33,7 +33,7 @@
     "require": {
         "php": "^7.3 || ^8.0",
         "composer/package-versions-deprecated": "^1.11.99",
-        "doctrine/cache": "^1.0",
+        "doctrine/cache": "^1.11",
         "doctrine/deprecations": "^0.5.3",
         "doctrine/event-manager": "^1.0"
     },
@@ -44,7 +44,9 @@
         "phpstan/phpstan-strict-rules": "^0.12.2",
         "phpunit/phpunit": "9.5.0",
         "psalm/plugin-phpunit": "0.13.0",
+        "psr/cache": "^1.0 || ^2.0 || ^3.0",
         "squizlabs/php_codesniffer": "3.6.0",
+        "symfony/cache": "^5.2",
         "symfony/console": "^2.0.5|^3.0|^4.0|^5.0",
         "vimeo/psalm": "4.6.4"
     },

phpstan.neon.dist

@@ -106,5 +106,11 @@ parameters:
             message: '~^Instanceof between Doctrine\\DBAL\\Platforms\\Keywords\\KeywordList and Doctrine\\DBAL\\Platforms\\Keywords\\KeywordList will always evaluate to true\.~'
             paths:
                 - %currentWorkingDirectory%/src/Platforms/AbstractPlatform.php
+
+        # We're checking for invalid invalid input
+        -
+            message: "#^Strict comparison using \\!\\=\\= between null and null will always evaluate to false\\.$#"
+            count: 1
+            path: src/Cache/QueryCacheProfile.php
 includes:
     - vendor/phpstan/phpstan-strict-rules/rules.neon

psalm.xml.dist

@@ -45,6 +45,11 @@
                     is no longer supported.
                 -->
                 <file name="src/Tools/Console/ConsoleRunner.php"/>
+                <!--
+                    This suppression should be removed once Doctrine Cache
+                    is no longer supported.
+                -->
+                <file name="tests/Functional/ResultCacheTest.php"/>
             </errorLevel>
         </DeprecatedClass>
         <DeprecatedMethod>
@@ -87,6 +92,12 @@
                     See https://github.com/doctrine/dbal/pull/4518
                 -->
                 <file name="src/Connection.php"/>
+
+                <!--
+                    This suppression should be removed in 4.0.x
+                    See TODO
+                -->
+                <file name="src/Configuration.php"/>
             </errorLevel>
         </DeprecatedProperty>
         <DocblockTypeContradiction>
@@ -100,6 +111,7 @@
                       1. Union types not supported at the language level (require dropping PHP 7 support)
                       2. Associative arrays with typed elements used instead of classes (require breaking API changes)
                 -->
+                <file name="src/Cache/QueryCacheProfile.php"/>
                 <file name="src/Connection.php"/>
                 <file name="src/Driver/IBMDB2/Statement.php"/>
                 <file name="src/DriverManager.php"/>

src/Cache/CachingResult.php

@@ -2,11 +2,11 @@
 
 namespace Doctrine\DBAL\Cache;
 
-use Doctrine\Common\Cache\Cache;
 use Doctrine\DBAL\Driver\Exception;
 use Doctrine\DBAL\Driver\FetchUtils;
 use Doctrine\DBAL\Driver\Result as DriverResult;
 use Doctrine\DBAL\Result;
+use Psr\Cache\CacheItemPoolInterface;
 
 use function array_map;
 use function array_values;
@@ -26,7 +26,7 @@
  */
 class CachingResult implements DriverResult
 {
-    /** @var Cache */
+    /** @var CacheItemPoolInterface */
     private $cache;
 
     /** @var string */
@@ -49,7 +49,7 @@ class CachingResult implements DriverResult
      * @param string $realKey
      * @param int    $lifetime
      */
-    public function __construct(Result $result, Cache $cache, $cacheKey, $realKey, $lifetime)
+    public function __construct(Result $result, CacheItemPoolInterface $cache, $cacheKey, $realKey, $lifetime)
     {
         $this->result   = $result;
         $this->cache    = $cache;
@@ -171,14 +171,15 @@ private function saveToCache(): void
             return;
         }
 
-        $data = $this->cache->fetch($this->cacheKey);
+        $item                 = $this->cache->getItem($this->cacheKey);
+        $data                 = $item->isHit() ? $item->get() : [];
+        $data[$this->realKey] = $this->data;
 
-        if ($data === false) {
-            $data = [];
+        $item->set($data);
+        if ($this->lifetime > 0) {
+            $item->expiresAfter($this->lifetime);
         }
 
-        $data[$this->realKey] = $this->data;
-
-        $this->cache->save($this->cacheKey, $data, $this->lifetime);
+        $this->cache->save($item);
     }
 }

src/Cache/QueryCacheProfile.php

@@ -3,11 +3,18 @@
 namespace Doctrine\DBAL\Cache;
 
 use Doctrine\Common\Cache\Cache;
+use Doctrine\Common\Cache\Psr6\CacheAdapter;
+use Doctrine\Common\Cache\Psr6\DoctrineProvider;
 use Doctrine\DBAL\Types\Type;
+use Doctrine\Deprecations\Deprecation;
+use Psr\Cache\CacheItemPoolInterface;
+use TypeError;
 
+use function get_class;
 use function hash;
 use function serialize;
 use function sha1;
+use function sprintf;
 
 /**
  * Query Cache Profile handles the data relevant for query caching.
@@ -16,8 +23,8 @@
  */
 class QueryCacheProfile
 {
-    /** @var Cache|null */
-    private $resultCacheDriver;
+    /** @var CacheItemPoolInterface|null */
+    private $resultCache;
 
     /** @var int */
     private $lifetime = 0;
@@ -26,22 +33,56 @@ class QueryCacheProfile
     private $cacheKey;
 
     /**
-     * @param int         $lifetime
-     * @param string|null $cacheKey
+     * @param int                               $lifetime
+     * @param string|null                       $cacheKey
+     * @param CacheItemPoolInterface|Cache|null $resultCache
      */
-    public function __construct($lifetime = 0, $cacheKey = null, ?Cache $resultCache = null)
+    public function __construct($lifetime = 0, $cacheKey = null, ?object $resultCache = null)
+    {
+        $this->lifetime = $lifetime;
+        $this->cacheKey = $cacheKey;
+        if ($resultCache instanceof CacheItemPoolInterface) {
+            $this->resultCache = $resultCache;
+        } elseif ($resultCache instanceof Cache) {
+            Deprecation::trigger(
+                'doctrine/dbal',
+                'TODO',
+                'Passing an instance of %s as $resultCache is deprecated. Pass an instance of %s instead.',
+                Cache::class,
+                CacheItemPoolInterface::class
+            );
+
+            $this->resultCache = CacheAdapter::wrap($resultCache);
+        } elseif ($resultCache !== null) {
+            throw new TypeError(sprintf(
+                '$resultCache: Expected either null or an instance of %s or %s, got %s.',
+                CacheItemPoolInterface::class,
+                Cache::class,
+                get_class($resultCache)
+            ));
+        }
+    }
+
+    public function getResultCache(): ?CacheItemPoolInterface
     {
-        $this->lifetime          = $lifetime;
-        $this->cacheKey          = $cacheKey;
-        $this->resultCacheDriver = $resultCache;
+        return $this->resultCache;
     }
 
     /**
+     * @deprecated Use {@see getResultCache()} instead.
+     *
      * @return Cache|null
      */
     public function getResultCacheDriver()
     {
-        return $this->resultCacheDriver;
+        Deprecation::trigger(
+            'doctrine/dbal',
+            'TODO',
+            '%s is deprecated, call getResultCache() instead.',
+            __METHOD__
+        );
+
+        return $this->resultCache !== null ? DoctrineProvider::wrap($this->resultCache) : null;
     }
 
     /**
@@ -93,12 +134,24 @@ public function generateCacheKeys($sql, $params, $types, array $connectionParams
         return [$cacheKey, $realCacheKey];
     }
 
+    public function setResultCache(CacheItemPoolInterface $cache): QueryCacheProfile
+    {
+        return new QueryCacheProfile($this->lifetime, $this->cacheKey, $cache);
+    }
+
     /**
      * @return QueryCacheProfile
      */
     public function setResultCacheDriver(Cache $cache)
     {
-        return new QueryCacheProfile($this->lifetime, $this->cacheKey, $cache);
+        Deprecation::trigger(
+            'doctrine/dbal',
+            'TODO',
+            '%s is deprecated, call setResultCache() instead.',
+            __METHOD__
+        );
+
+        return new QueryCacheProfile($this->lifetime, $this->cacheKey, CacheAdapter::wrap($cache));
     }
 
     /**
@@ -108,7 +161,7 @@ public function setResultCacheDriver(Cache $cache)
      */
     public function setCacheKey($cacheKey)
     {
-        return new QueryCacheProfile($this->lifetime, $cacheKey, $this->resultCacheDriver);
+        return new QueryCacheProfile($this->lifetime, $cacheKey, $this->resultCache);
     }
 
     /**
@@ -118,6 +171,6 @@ public function setCacheKey($cacheKey)
      */
     public function setLifetime($lifetime)
     {
-        return new QueryCacheProfile($lifetime, $this->cacheKey, $this->resultCacheDriver);
+        return new QueryCacheProfile($lifetime, $this->cacheKey, $this->resultCache);
     }
 }

src/Configuration.php

@@ -3,8 +3,12 @@
 namespace Doctrine\DBAL;
 
 use Doctrine\Common\Cache\Cache;
+use Doctrine\Common\Cache\Psr6\CacheAdapter;
+use Doctrine\Common\Cache\Psr6\DoctrineProvider;
 use Doctrine\DBAL\Driver\Middleware;
 use Doctrine\DBAL\Logging\SQLLogger;
+use Doctrine\Deprecations\Deprecation;
+use Psr\Cache\CacheItemPoolInterface;
 
 /**
  * Configuration container for the Doctrine DBAL.
@@ -24,6 +28,15 @@ class Configuration
     /**
      * The cache driver implementation that is used for query result caching.
      *
+     * @var CacheItemPoolInterface|null
+     */
+    protected $resultCache;
+
+    /**
+     * The cache driver implementation that is used for query result caching.
+     *
+     * @deprecated Use {@see $resultCache} instead.
+     *
      * @var Cache|null
      */
     protected $resultCacheImpl;
@@ -61,17 +74,53 @@ public function getSQLLogger(): ?SQLLogger
     /**
      * Gets the cache driver implementation that is used for query result caching.
      */
+    public function getResultCache(): ?CacheItemPoolInterface
+    {
+        return $this->resultCache;
+    }
+
+    /**
+     * Gets the cache driver implementation that is used for query result caching.
+     *
+     * @deprecated Use {@see getResultCache()} instead.
+     */
     public function getResultCacheImpl(): ?Cache
     {
+        Deprecation::trigger(
+            'doctrine/dbal',
+            'TODO',
+            '%s is deprecated, call getResultCache() instead.',
+            __METHOD__
+        );
+
         return $this->resultCacheImpl;
     }
 
     /**
      * Sets the cache driver implementation that is used for query result caching.
      */
+    public function setResultCache(CacheItemPoolInterface $cache): void
+    {
+        $this->resultCacheImpl = DoctrineProvider::wrap($cache);
+        $this->resultCache     = $cache;
+    }
+
+    /**
+     * Sets the cache driver implementation that is used for query result caching.
+     *
+     * @deprecated Use {@see setResultCache()} instead.
+     */
     public function setResultCacheImpl(Cache $cacheImpl): void
     {
+        Deprecation::trigger(
+            'doctrine/dbal',
+            'TODO',
+            '%s is deprecated, call setResultCache() instead.',
+            __METHOD__
+        );
+
         $this->resultCacheImpl = $cacheImpl;
+        $this->resultCache     = CacheAdapter::wrap($cacheImpl);
     }
 
     /**

src/Connection.php

@@ -1087,7 +1087,7 @@ public function executeQuery(
      */
     public function executeCacheQuery($sql, $params, $types, QueryCacheProfile $qcp): Result
     {
-        $resultCache = $qcp->getResultCacheDriver() ?? $this->_config->getResultCacheImpl();
+        $resultCache = $qcp->getResultCache() ?? $this->_config->getResultCache();
 
         if ($resultCache === null) {
             throw CacheException::noResultDriverConfigured();
@@ -1099,9 +1099,10 @@ public function executeCacheQuery($sql, $params, $types, QueryCacheProfile $qcp)
         [$cacheKey, $realKey] = $qcp->generateCacheKeys($sql, $params, $types, $connectionParams);
 
         // fetch the row pointers entry
-        $data = $resultCache->fetch($cacheKey);
+        $item = $resultCache->getItem($cacheKey);
 
-        if ($data !== false) {
+        if ($item->isHit()) {
+            $data = $item->get();
             // is the real key part of this row pointers map or is the cache only pointing to other cache keys?
             if (isset($data[$realKey])) {
                 $result = new ArrayResult($data[$realKey]);