Small corrections

AlexH-HankIT · Nov 15, 2015 · a9bde71 · a9bde71
1 parent 70898e4
commit a9bde71
Showing 1 changed file with 17 additions and 13 deletions.
diff --git a/app/models/target/Target.php b/app/models/target/Target.php
@@ -455,19 +455,23 @@ public function add_setting($option, $newvalue) {
                 }
 
                 if ($default_settings[$default_settings_key]['defaultvalue'] == $newvalue) {
-                    if (isset($targetsettings[$key])) {
-                        $return = $this->parse_file($this->ietd_config_file, [$this, 'delete_option_from_iqn'], array($option . ' ' . $targetsettings[$key][1]), false, false);
-
-                        if ($return !== 0) {
-                            $this->logging->log_action_result('The new value is the default value, so i just deleted it!', array('result' => 0, 'code_type' => 'intern'), __METHOD__);
-                        } else {
-                            $return = $this->add_config_to_daemon($option, $newvalue);
-
-                            if ($return['result'] !== 0) {
-                                $this->logging->log_action_result('Could not set the default value for ' . $option . ' in daemon config', $return, __METHOD__);
-                            }
-                        }
-                    }
+					if ($targetsettings !== 0) {
+						if (isset($targetsettings[$key])) {
+							$return = $this->parse_file($this->ietd_config_file, [$this, 'delete_option_from_iqn'], array($option . ' ' . $targetsettings[$key][1]), false, false);
+
+							if ($return !== 0) {
+								$this->logging->log_action_result('The new value is the default value, so i just deleted it!', array('result' => 0, 'code_type' => 'intern'), __METHOD__);
+							} else {
+								$return = $this->add_config_to_daemon($option, $newvalue);
+
+								if ($return['result'] !== 0) {
+									$this->logging->log_action_result('Could not set the default value for ' . $option . ' in daemon config', $return, __METHOD__);
+								}
+							}
+						}
+					} else {
+						$this->logging->log_action_result('I won\'t add the default value', array('result' => 4, 'code_type' => 'intern'), __METHOD__);
+					}
                 } else {
                     // add option
                     $return = $this->parse_file($this->ietd_config_file, [$this, 'add_option_to_iqn'], array($option . ' ' . $newvalue), false, false);