Fix RedisCluster::setOption|getOption types

See phpredis/phpredis#2038 and vimeo/psalm#7030
ostrolucky · Dec 2, 2021 · 10e9fd1 · 10e9fd1
1 parent ec24a9a
commit 10e9fd1
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/redis/RedisCluster.php b/redis/RedisCluster.php
@@ -2817,20 +2817,20 @@ public function clearLastError() {}
     /**
      * Get client option
      *
-     * @param   string $name parameter name
+     * @param   string $option parameter name
      *
      * @return  int     Parameter value.
      * @example
      * // return RedisCluster::SERIALIZER_NONE, RedisCluster::SERIALIZER_PHP, or RedisCluster::SERIALIZER_IGBINARY.
      * $redisCluster->getOption(RedisCluster::OPT_SERIALIZER);
      */
-    public function getOption($name) {}
+    public function getOption($option) {}
 
     /**
      * Set client option.
      *
-     * @param   string $name  parameter name
-     * @param   string $value parameter value
+     * @param   int        $option parameter name
+     * @param   int|string $value  parameter value
      *
      * @return  bool   TRUE on success, FALSE on error.
      * @example
@@ -2841,7 +2841,7 @@ public function getOption($name) {}
      * $redisCluster->setOption(RedisCluster::OPT_PREFIX, 'myAppName:');                             // use custom prefix on all keys
      * </pre>
      */
-    public function setOption($name, $value) {}
+    public function setOption($option, $value) {}
 
     /**
      * A utility method to prefix the value with the prefix setting for phpredis.