Default config keys are now fixed (Confusing default configuration va…

…lues #66)
Webklex · Mar 28, 2018 · d51e265 · d51e265
1 parent 665c23a
commit d51e265
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/src/IMAP/Client.php b/src/IMAP/Client.php
@@ -101,6 +101,13 @@ class Client {
      */
     protected $errors = [];
 
+    /**
+     * All valid and available account config parameters
+     *
+     * @var array $validConfigKeys
+     */
+    protected $validConfigKeys = ['host', 'port', 'encryption', 'validate_cert', 'username', 'password'];
+
     /**
      * Client constructor.
      *
@@ -128,8 +135,8 @@ public function setConfig(array $config) {
         $defaultAccount = config('imap.default');
         $defaultConfig  = config("imap.accounts.$defaultAccount");
 
-        foreach($defaultConfig as $key => $default){
-            $this->$key = isset($config[$key]) ? $config[$key] : $default;
+        foreach($this->validConfigKeys as $key){
+            $this->$key = isset($config[$key]) ? $config[$key] : $defaultConfig[$key];
         }
 
         return $this;