minor fixes for setting config options

Author: dmolsen

src/PatternLab/Config.php

@@ -266,7 +266,7 @@ public static function setOption($optionName = "", $optionValue = "") {
 		}
 
 		$arrayFinder = new ArrayFinder(self::$options);
-		$arrayFinder->set($key, $value);
+		$arrayFinder->set($optionName, $optionValue);
 		self::$options = $arrayFinder->get();
 
 	}
@@ -323,7 +323,8 @@ public static function updateConfigOption($optionName,$optionValue, $force = fal
 		} else if (self::$options["overrideConfig"] == "q") {
 
 			// standardize the values for comparison
-			$currentOptionValue = is_array(self::$options[$optionName]) ? implode(", ",self::$options[$optionName]) : self::$options[$optionName];
+			$currentOption      = self::getOption($optionName);
+			$currentOptionValue = is_array($currentOption) ? implode(", ",$currentOption) : $currentOption;
 			$newOptionValue     = is_array($optionValue) ? implode(", ",$optionValue) : $optionValue;
 
 			if ($currentOptionValue != $newOptionValue) {
@@ -373,18 +374,11 @@ protected static function writeUpdateConfigOption($optionName,$optionValue) {
 			Console::writeError("Config parse error in <path>".self::$userConfigPath."</path>: ".$e->getMessage());
 		}
 
-		if (isset($options[$optionName]) && is_array($options[$optionName])) {
-			$optionValue = is_array($optionValue) ? $optionValue : array($optionValue);
-			$options[$optionName] = array_merge($options[$optionName], $optionValue);
-		} else {
-			$options[$optionName] = $optionValue;
-		}
-		
-		// reload the options
-		self::$options = $options;
+		print "foo ".$optionName." ".$optionValue;
+		self::setOption($optionName, $optionValue);
 
 		// dump the YAML
-		$configOutput = Yaml::dump($options, 3);
+		$configOutput = Yaml::dump(self::$options, 3);
 
 		// write out the new config file
 		file_put_contents(self::$userConfigPath,$configOutput);

src/PatternLab/Console/Commands/ConfigCommand.php

@@ -77,7 +77,7 @@ public function run() {
 			$updateOptionBits = explode("=",$updateOption);
 			if (count($updateOptionBits) == 1) {
 				Console::writeError("the --set value should look like <info>optionName=\"optionValue\"</info>. nothing was updated...");
-			} 
+			}
 
 			// set the name and value that were passed
 			$updateName   = $updateOptionBits[0];
@@ -90,6 +90,7 @@ public function run() {
 				Console::writeError("the --set option you provided, <info>".$updateName."</info>, does not exists in the config. nothing will be updated...");
 			} else {
 				Config::updateConfigOption($updateName,$updateValue);
+				Console::writeInfo("config option updated...");
 			}
 
 		} else {