X-Git-Url: http://git.pjr.cc/?p=CBFWR.git;a=blobdiff_plain;f=libcbfwr%2Fconfig.php;h=e0b4f37702f782d6fc883cef23ec4420677cfea2;hp=372bd04d50123d7ff54d2707053e3f91d3c02417;hb=b571fcf2648a6d5a4893125cb97db4f3f73ee134;hpb=60673f4f00bab805471fcbb85c8816aebb3a9582

diff --git a/libcbfwr/config.php b/libcbfwr/config.php
index 372bd04..e0b4f37 100644
--- a/libcbfwr/config.php
+++ b/libcbfwr/config.php
@@ -76,6 +76,10 @@ class Config {
 		return $this->config;
 	}
 	
+	function setConfig($config) {
+		$this->config = $config;
+	}
+	
 	function applyConfig()
 	{
 		global $AM_DAEMON;
@@ -85,87 +89,7 @@ class Config {
 		return true;
 	}
 	
-	function mergeConfig($configone, $configtwo)
-	{
-		// yep this means apply changes in configtwo to configone
-		$newconf = $configone;
-		
-		// this is gunna be tough its gunna be a meet on meet sandwitch
-		foreach($configtwo as $key => $var) {
-			if($key != "status") {
-				error_log("apply change $key");
-				foreach($var as $key2=>$var2) {
-					error_log("which is $key2");
-					$splits = explode(" ", $var2);
-					
-					// find a description
-					$description = null;
-					$hasdescription = false;
-					for($i=0; $i<count($splits); $i++) {
-						if($hasdescription !== false) {
-							if($description != null) {
-								$description .= " ";
-							}
-							$description .= $splits[$i];
-						} 
-						if($splits[$i] == "#") {
-							$hasdescription = $i;
-						}
-					}
-					
-					
-					switch($key2) {
-						case "add":
-							error_log("would add $var2");
-							$thing = $splits[0];
-							switch($thing) {
-								case "zone":
-									$newconf["zone"][$splits[1]]["name"] = 1;
-									if($hasdescription) {
-										$newconf["zone"][$splits[1]]["description"] = $description;
-									}
-									break;
-								
-							}
-							break;
-						case "modify":
-							error_log("would modify $var2");
-							$thing = $splits[0];
-							switch($thing) {
-								case "zone":
-									$zonename = $splits[1];
-									if($hasdescription) {
-										$newconf["zone"][$splits[1]]["description"] = $description;
-									}
-									if($splits[2] == "rename") {
-										$newname = $splits[3];
-										$tree = $newconf["zone"][$splits[1]];
-										unset($newconf["zone"][$splits[1]]);
-										$newconf["zone"][$newname] = $tree;
-									}
-									break;
-								
-							}
-							break;
-							
-							
-						case "delete":
-							error_log("would delete $var2");
-							$thing = $splits[0];
-							switch($thing) {
-								case "zone":
-									$deletezone = $splits[1];
-									unset($newconf["zone"][$deletezone]);
-									break;
-								
-							}
-							break;
-					}
-				} 
-			}
-		}
-		return $newconf;
-	}
+
 	
 	function loadConfigFile($file=null)
 	{
@@ -184,14 +108,21 @@ class Config {
 		while($line = fgets($fp)) {
 			$line = trim($line);
 			echo "read line $line\n";
-			if($line != "") $this->parseLine($line, $i++);
+			if($line != "") $this->addConfigLine($line, $i++);
 		}
 	}
 	
-	function parseLine($line, $lineno)
+	function addConfigLine($line, $lineno=0)
 	{
+		$line_n = trim($line);
+		$line = $line_n;
 		$expl = preg_split("/ +/", $line);
 		
+		if(count($expl) < 2) {
+			$c = count($expl);
+			error_log("Not a valid config line ($c), $line");
+			return 0;
+		}
 		echo "process command ".$expl[0]."\n";
 		
 		// find a description
@@ -209,7 +140,7 @@ class Config {
 			}
 		}
 		
-		// now rebuild the array if there was one
+		// now rebuild the array if there was a description
 		if($hasdescription) {
 			for($i=0; $i<$hasdescription; $i++) {
 				$expl_r[$i] = $expl[$i];
@@ -217,22 +148,64 @@ class Config {
 			$expl = $expl_r;
 		}
 		
+		$delete = false;
+		// check for delete on the line
+		if($expl[0] == "delete") {
+			$delete = true;
+			
+			// re-order the array
+			for($i=0; $i < (count($expl)-1); $i++) {
+		        $expl[$i] = $expl[$i+1];
+			}
+			unset($expl[count($expl)-1]);
+		}
+		
+		// now for modify
+		$modify = false;
+		// check for delete on the line
+		if($expl[0] == "modify") {
+			$modify = true;
+			
+			// re-order the array
+			for($i=0; $i < (count($expl)-1); $i++) {
+		        $expl[$i] = $expl[$i+1];
+			}
+			unset($expl[count($expl)-1]);
+		}
+		
 		switch($expl[0]) {
 			case "hostname":
 				// set the hostname to $1
-				$this->config["hostname"] = $expl[1];
+				if($delete) {
+					unset($this->config["hostname"]);				
+				} else {
+					$this->config["hostname"] = $expl[1];
+				}
 				break;
 				
 			case "domainname":
 				// set the hostname to $1
-				$this->config["domainname"] = $expl[1];
+				if($delete) {
+					unset($this->config["domainname"]);
+				} else {
+					$this->config["domainname"] = $expl[1];
+				}
 				break;
 				
 			case "zone":
-				if($hasdescription) {
-					$this->config["zone"][$expl[2]]["description"] = $description;
+				if($delete) {
+					error_log("delete zone ".$expl[2]);
+					unset($this->config["zone"][$expl[2]]);
+					if(count($this->config["zone"]) < 1) {
+						error_log("zone now empty, delete zones");
+						unset($this->config["zone"]);
+					}
+				} else {
+					if($hasdescription) {
+						$this->config["zone"][$expl[2]]["description"] = $description;
+					}
+					$this->config["zone"][$expl[2]]["name"] = true;
 				}
-				$this->config["zone"][$expl[2]]["name"] = true;
 				break;
 				
 			case "interface":
@@ -308,54 +281,65 @@ class Config {
 				
 				
 			case "login":
-				$this->config["login"][$expl[1]] = $expl[3];
-				if($hasdescription) {
-					$this->config["login"][$expl[1]]["description"] = $description;
+				if($delete) {
+					unset($this->config["login"][$expl[1]]);
+				} else {
+					$this->config["login"][$expl[1]] = $expl[3];
+					if($hasdescription) {
+						$this->config["login"][$expl[1]]["description"] = $description;
+					}
 				}
 				break;
 			
 			case "route4":
-				$route = $expl[1];
-				$via = $expl[2];
-				$dest = $expl[3];
-				if($via == "to") {
-					$this->config["route4"][$route]["address"] = $dest;
+				if($delete) {
+					unset($this->config["route4"][$expl[1]]);
 				} else {
-					$this->config["route4"][$route]["device"] = $dest;
-				}
-				if(isset($expl[4])) {
-					if($expl[4] == "dev") {
-						if(isset($expl[5])) {
-							$this->config["route4"][$route]["device"] = $expl[5];
+					$route = $expl[1];
+					$via = $expl[2];
+					$dest = $expl[3];
+					if($via == "to") {
+						$this->config["route4"][$route]["address"] = $dest;
+					} else {
+						$this->config["route4"][$route]["device"] = $dest;
+					}
+					if(isset($expl[4])) {
+						if($expl[4] == "dev") {
+							if(isset($expl[5])) {
+								$this->config["route4"][$route]["device"] = $expl[5];
+							}
 						}
 					}
-				}
-				if($hasdescription) {
-					$this->config["route4"][$route]["description"] = $description;
+					if($hasdescription) {
+						$this->config["route4"][$route]["description"] = $description;
+					}
 				}
 				break;
 				
 				
 			case "route6":
-				$route = $expl[1];
-				$via = $expl[2];
-				$dest = $expl[3];
-				if($via == "to") {
-					$this->config["route6"][$route]["address"] = $dest;
+				if($delete) {
+					unset($this->config["route6"][$expl[1]]);
 				} else {
-					$this->config["route6"][$route]["device"] = $dest;
-				}
-				if(isset($expl[4])) {
-					if($expl[4] == "dev") {
-						if(isset($expl[5])) {
-							$this->config["route6"][$route]["device"] = $expl[5];
+					$route = $expl[1];
+					$via = $expl[2];
+					$dest = $expl[3];
+					if($via == "to") {
+						$this->config["route6"][$route]["address"] = $dest;
+					} else {
+						$this->config["route6"][$route]["device"] = $dest;
+					}
+					if(isset($expl[4])) {
+						if($expl[4] == "dev") {
+							if(isset($expl[5])) {
+								$this->config["route6"][$route]["device"] = $expl[5];
+							}
 						}
 					}
+					if($hasdescription) {
+						$this->config["route6"][$route]["description"] = $description;
+					}
 				}
-				if($hasdescription) {
-					$this->config["route6"][$route]["description"] = $description;
-				}
-				
 				// here we should check "$route"
 				break;