From dd20e8557fdb9c7da01ff5dfa5cc9f44704da115 Mon Sep 17 00:00:00 2001 From: Daniele Alessandri Date: Fri, 2 Dec 2011 15:12:47 +0100 Subject: [PATCH] Pass the client options to the internal cluster initializer method. This change is useless right now, but when Predis will support Redis cluster we will need to have access the connection factory object since it will be required by our future RedisCluster class. --- lib/Predis/Options/ClientCluster.php | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/lib/Predis/Options/ClientCluster.php b/lib/Predis/Options/ClientCluster.php index 53cf5a22..6f8ee530 100644 --- a/lib/Predis/Options/ClientCluster.php +++ b/lib/Predis/Options/ClientCluster.php @@ -46,7 +46,7 @@ class ClientCluster extends Option if (is_callable($value)) { return $this->checkInstance(call_user_func($value)); } - $initializer = $this->getInitializer($value); + $initializer = $this->getInitializer($options, $value); return $this->checkInstance($initializer()); } @@ -54,10 +54,11 @@ class ClientCluster extends Option /** * Returns an initializer for the specified FQN or type. * - * @param string $fqnOrType Type of cluster of FQN of a class implementing IConnectionCluster + * @param string $fqnOrType Type of cluster of FQN of a class implementing IConnectionCluster. + * @param IClientOptions $options Instance of the client options. * @return \Closure */ - protected function getInitializer($fqnOrType) + protected function getInitializer(IClientOptions $options, $fqnOrType) { switch ($fqnOrType) { case 'predis':