Credis_Cluster - Fix hashing mechanism for array arguments (#111)

andrey-legayev · colinmollenhour · commit c1cab895f233 · 2019-03-05T11:29:14.000-05:00
diff --git a/Cluster.php b/Cluster.php
@@ -240,7 +240,11 @@ public function __call($name, $args)
       $client = $this->clients[0];
     }
     else {
-      $client = $this->byHash($args[0]);
+      $hashKey = $args[0];
+      if (is_array($hashKey)) {
+        $hashKey = join('|', $hashKey);
+      }
+      $client = $this->byHash($hashKey);
     }
     // Ensure that current client is working on the same database as expected.
     if ($client->getSelectedDb() != $this->selectedDb) {

Original file line number	Diff line number	Diff line change
`@@ -240,7 +240,11 @@ public function __call($name, $args)`
`240`	`240`	`$client = $this->clients[0];`
`241`	`241`	`}`
`242`	`242`	`else {`
`243`		`- $client = $this->byHash($args[0]);`
	`243`	`+ $hashKey = $args[0];`
	`244`	`+ if (is_array($hashKey)) {`
	`245`	`+ $hashKey = join('\|', $hashKey);`
	`246`	`+ }`
	`247`	`+ $client = $this->byHash($hashKey);`
`244`	`248`	`}`
`245`	`249`	`// Ensure that current client is working on the same database as expected.`
`246`	`250`	`if ($client->getSelectedDb() != $this->selectedDb) {`