fix: prevent Redis error when deleteMatching finds no keys (#9829)

Author: michalsn

system/Cache/Handlers/PredisHandler.php

@@ -162,6 +162,10 @@ public function deleteMatching(string $pattern)
             $matchedKeys[] = $key;
         }
 
+        if ($matchedKeys === []) {
+            return 0;
+        }
+
         return $this->redis->del($matchedKeys);
     }
 

system/Cache/Handlers/RedisHandler.php

@@ -195,7 +195,7 @@ public function deleteMatching(string $pattern)
             }
         } while ($iterator > 0);
 
-        return $this->redis->del($matchedKeys);
+        return (int) $this->redis->del($matchedKeys);
     }
 
     /**

tests/system/Cache/Handlers/PredisHandlerTest.php

@@ -160,6 +160,11 @@ public function testDeleteMatchingSuffix(): void
         $this->assertSame('90', $this->handler->getCacheInfo()['Keyspace']['db0']['keys']);
     }
 
+    public function testDeleteMatchingNothing(): void
+    {
+        $this->assertSame(0, $this->handler->deleteMatching('user_1_info*'));
+    }
+
     public function testClean(): void
     {
         $this->handler->save(self::$key1, 1);

tests/system/Cache/Handlers/RedisHandlerTest.php

@@ -184,6 +184,11 @@ public static function provideDeleteMatching(): iterable
         yield 'cache-prefix' => ['key_1*', 13, 'foo_'];
     }
 
+    public function testDeleteMatchingNothing(): void
+    {
+        $this->assertSame(0, $this->handler->deleteMatching('user_1_info*'));
+    }
+
     public function testIncrementAndDecrement(): void
     {
         $this->handler->save('counter', 100);

user_guide_src/source/changelogs/v4.6.4.rst

@@ -37,6 +37,8 @@ Deprecations
 Bugs Fixed
 **********
 
+- **Cache:** Fixed a bug in ``PredisHandler::deleteMatching()`` causing Redis error when no keys match the pattern.
+- **Cache:** Fixed a bug in ``RedisHandler::deleteMatching()`` returning ``false`` instead of ``int`` when no keys match the pattern.
 - **Database:** Fixed a bug in ``Database::connect()`` which was causing to store non-shared connection instances in shared cache.
 - **Database:** Fixed a bug in ``Connection::getFieldData()`` for ``SQLSRV`` and ``OCI8`` where extra characters were returned in column default values (specific to those handlers), instead of following the convention used by other drivers.
 - **Database:** Fixed a bug in ``BaseBuilder::compileOrderBy()`` where the method could overwrite ``QBOrderBy`` with a string instead of keeping it as an array, causing type errors and preventing additional ``ORDER BY`` clauses from being appended.