Browse Source

Improve initialization code of Predis\Client instances.

Daniele Alessandri 14 years ago
parent
commit
bf3776b47a
1 changed files with 5 additions and 9 deletions
  1. 5 9
      lib/Predis.php

+ 5 - 9
lib/Predis.php

@@ -13,17 +13,13 @@ use Predis\Distribution\IDistributionStrategy;
 class Client {
     private $_options, $_profile, $_connection;
 
-    public function __construct($parameters = null, $clientOptions = null) {
-        $this->setupClient($clientOptions ?: new ClientOptions());
-        $this->_connection = $this->initializeConnection($parameters);
-    }
-
-    private function setupClient($options) {
-        $this->_options = $this->filterClientOptions($options);
+    public function __construct($parameters = null, $options = null) {
+        $this->_options = $this->filterOptions($options ?: new ClientOptions());
         $this->_profile = $this->_options->profile;
+        $this->_connection = $this->initializeConnection($parameters);
     }
 
-    private function filterClientOptions($options) {
+    private function filterOptions($options) {
         if ($options instanceof ClientOptions) {
             return $options;
         }
@@ -41,7 +37,7 @@ class Client {
     }
 
     private function initializeConnection($parameters = array()) {
-        if ($parameters === null) {
+        if (!isset($parameters)) {
             return $this->createConnection(array());
         }
         if ($parameters instanceof IConnection) {