Merge pull request #123 from samsonasik/update-to-use-php81-syntax

Update to use PHP 8.1 syntax
laminas · Nov 3, 2024 · 330af6b · 330af6b
2 parents b6a8450 + 3d0e01d
commit 330af6b
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 6 deletions.
diff --git a/src/Exception/RedisRuntimeException.php b/src/Exception/RedisRuntimeException.php
@@ -35,7 +35,7 @@ public static function fromRedisException(RedisException $exception, Redis $redi
     {
         try {
             $message = $redis->getLastError() ?? $exception->getMessage();
-        } catch (RedisException $exceptionThrownByGetLastErrorMethod) {
+        } catch (RedisException) {
             $message = $exception->getMessage();
         }
 

diff --git a/src/RedisCluster.php b/src/RedisCluster.php
@@ -91,7 +91,7 @@ public function flush(): bool
             $redis = new RedisFromExtension();
             try {
                 $redis->connect($host, $port);
-            } catch (RedisException $exception) {
+            } catch (RedisException) {
                 continue;
             }
 

diff --git a/src/RedisClusterResourceManager.php b/src/RedisClusterResourceManager.php
@@ -20,11 +20,8 @@
  */
 final class RedisClusterResourceManager implements RedisClusterResourceManagerInterface
 {
-    private RedisClusterOptions $options;
-
-    public function __construct(RedisClusterOptions $options)
+    public function __construct(private readonly RedisClusterOptions $options)
     {
-        $this->options = $options;
     }
 
     public function getResource(): RedisClusterFromExtension