Fix RedisCluster::getOption() and RedisCluster::setOption argument types

See https://github.com/phpredis/phpredis/blob/aac42cd33510030bb62973f8886f4cd98b9b8003/redis_cluster.stub.php#L122 Related #6853
vimeo · Nov 30, 2021 · 39eabbe · 39eabbe
1 parent 466137a
commit 39eabbe
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 8 deletions.
diff --git a/dictionaries/CallMap.php b/dictionaries/CallMap.php
@@ -11175,7 +11175,7 @@
 'RedisCluster::getBit' => ['int', 'key'=>'string', 'offset'=>'int'],
 'RedisCluster::getLastError' => ['?string'],
 'RedisCluster::getMode' => ['int'],
-'RedisCluster::getOption' => ['int', 'name'=>'string'],
+'RedisCluster::getOption' => ['int', 'option'=>'int'],
 'RedisCluster::getRange' => ['string', 'key'=>'string', 'start'=>'int', 'end'=>'int'],
 'RedisCluster::getSet' => ['string', 'key'=>'string', 'value'=>'string'],
 'RedisCluster::hDel' => ['int|false', 'key'=>'string', 'hashKey'=>'string', '...other_hashKeys='=>'string[]'],
@@ -11250,7 +11250,7 @@
 'RedisCluster::setBit' => ['int', 'key'=>'string', 'offset'=>'int', 'value'=>'bool|int'],
 'RedisCluster::setex' => ['bool', 'key'=>'string', 'ttl'=>'int', 'value'=>'string'],
 'RedisCluster::setnx' => ['bool', 'key'=>'string', 'value'=>'string'],
-'RedisCluster::setOption' => ['bool', 'name'=>'string|int', 'value'=>'string|int'],
+'RedisCluster::setOption' => ['bool', 'name'=>'int', 'value'=>'string|int'],
 'RedisCluster::setRange' => ['string', 'key'=>'string', 'offset'=>'int', 'value'=>'string'],
 'RedisCluster::sInter' => ['list<string>', 'key'=>'string', '...other_keys='=>'string'],
 'RedisCluster::sInterStore' => ['int', 'dstKey'=>'string', 'key'=>'string', '...other_keys='=>'string'],

diff --git a/dictionaries/CallMap_74_delta.php b/dictionaries/CallMap_74_delta.php
@@ -36,10 +36,6 @@
       'old' => ['resource|false', 'command'=>'string', 'descriptorspec'=>'array', '&w_pipes'=>'resource[]', 'cwd='=>'?string', 'env='=>'?array', 'other_options='=>'array'],
       'new' => ['resource|false', 'cmd'=>'string|array', 'descriptorspec'=>'array', '&w_pipes'=>'resource[]', 'cwd='=>'?string', 'env='=>'?array', 'other_options='=>'array'],
     ],
-    'RedisCluster::setOption' => [
-      'old' => ['bool', 'name'=>'string', 'value'=>'string'],
-      'new' => ['bool', 'name'=>'string|int', 'value'=>'string|int'],
-    ]
   ],
   'removed' => [
   ],

diff --git a/dictionaries/CallMap_historical.php b/dictionaries/CallMap_historical.php
@@ -5776,7 +5776,7 @@
     'RedisCluster::getBit' => ['int', 'key'=>'string', 'offset'=>'int'],
     'RedisCluster::getLastError' => ['?string'],
     'RedisCluster::getMode' => ['int'],
-    'RedisCluster::getOption' => ['int', 'name'=>'string'],
+    'RedisCluster::getOption' => ['int', 'option'=>'int'],
     'RedisCluster::getRange' => ['string', 'key'=>'string', 'start'=>'int', 'end'=>'int'],
     'RedisCluster::getSet' => ['string', 'key'=>'string', 'value'=>'string'],
     'RedisCluster::hDel' => ['int|false', 'key'=>'string', 'hashKey'=>'string', '...other_hashKeys='=>'string[]'],
@@ -5861,7 +5861,6 @@
     'RedisCluster::script' => ['string|bool|array', 'nodeParams'=>'string|array{0:string,1:int}', 'command'=>'string', 'script='=>'string', '...other_scripts='=>'string[]'],
     'RedisCluster::set' => ['bool', 'key'=>'string', 'value'=>'string', 'timeout='=>'array|int'],
     'RedisCluster::setBit' => ['int', 'key'=>'string', 'offset'=>'int', 'value'=>'bool|int'],
-    'RedisCluster::setOption' => ['bool', 'name'=>'string', 'value'=>'string'],
     'RedisCluster::setRange' => ['string', 'key'=>'string', 'offset'=>'int', 'value'=>'string'],
     'RedisCluster::setex' => ['bool', 'key'=>'string', 'ttl'=>'int', 'value'=>'string'],
     'RedisCluster::setnx' => ['bool', 'key'=>'string', 'value'=>'string'],