From 663caf8590b5998948f79f8cb8c4b507a8fe5140 Mon Sep 17 00:00:00 2001
From: jwarnier <jwarnier@ispconfig3>
Date: Wed, 22 Sep 2010 18:47:40 -0400
Subject: [PATCH] - replaced double quotes with single quotes whenever appropriate - replaced "exec('/etc/init.d/" calls with "exec($conf['init_scripts'] . '/' ." - fixed output wording (typos and more...)

---
 server/plugins-available/network_settings_plugin.inc.php |  148 ++++++++++++++++++++++++------------------------
 1 files changed, 74 insertions(+), 74 deletions(-)

diff --git a/server/plugins-available/network_settings_plugin.inc.php b/server/plugins-available/network_settings_plugin.inc.php
index 961e2cd..4b06006 100644
--- a/server/plugins-available/network_settings_plugin.inc.php
+++ b/server/plugins-available/network_settings_plugin.inc.php
@@ -76,8 +76,8 @@
 		global $app, $conf;
 		
 		// get the config
-		$app->uses("getconf");
-		$server_config = $app->getconf->get_server_config($conf["server_id"], 'server');
+		$app->uses('getconf');
+		$server_config = $app->getconf->get_server_config($conf['server_id'], 'server');
 		
 		// Configure the debian network card settings
 		if($server_config['auto_network_configuration'] == 'y') {
@@ -89,15 +89,15 @@
 				$app->load('tpl');
 				
 				$network_tpl = new tpl();
-				$network_tpl->newTemplate("debian_network_interfaces.master");
+				$network_tpl->newTemplate('debian_network_interfaces.master');
 					
-				$network_tpl->setVar('ip_address',$server_config["ip_address"]);
-				$network_tpl->setVar('netmask',$server_config["netmask"]);
-				$network_tpl->setVar('gateway',$server_config["gateway"]);
-				$network_tpl->setVar('broadcast',$this->broadcast($server_config["ip_address"],$server_config["netmask"]));
-				$network_tpl->setVar('network',$this->network($server_config["ip_address"],$server_config["netmask"]));
+				$network_tpl->setVar('ip_address',$server_config['ip_address']);
+				$network_tpl->setVar('netmask',$server_config['netmask']);
+				$network_tpl->setVar('gateway',$server_config['gateway']);
+				$network_tpl->setVar('broadcast',$this->broadcast($server_config['ip_address'],$server_config['netmask']));
+				$network_tpl->setVar('network',$this->network($server_config['ip_address'],$server_config['netmask']));
 				
-				$records = $app->db->queryAllRecords("SELECT ip_address FROM server_ip WHERE server_id = ".intval($conf["server_id"]) . " order by ip_address");
+				$records = $app->db->queryAllRecords("SELECT ip_address FROM server_ip WHERE server_id = ".intval($conf['server_id']) . ' order by ip_address');
 				$ip_records = array();
 				$additionl_ip_records = 0;
 				$n = 0;
@@ -106,15 +106,15 @@
 						/*
 						 * don't insert the main-ip again!
 						 */
-						if ($rec['ip_address'] != $server_config["ip_address"])
+						if ($rec['ip_address'] != $server_config['ip_address'])
 						{
 							$ip_records[$n] = array(
 								'id' => $n,
 								'ip_address' => $rec['ip_address'],
-								'netmask' => $server_config["netmask"],
-								'gateway' => $server_config["gateway"],
-								'broadcast' => $this->broadcast($rec['ip_address'],$server_config["netmask"]),
-								'network' => $this->network($rec['ip_address'],$server_config["netmask"])
+								'netmask' => $server_config['netmask'],
+								'gateway' => $server_config['gateway'],
+								'broadcast' => $this->broadcast($rec['ip_address'],$server_config['netmask']),
+								'network' => $this->network($rec['ip_address'],$server_config['netmask'])
 							);
 							$additionl_ip_records = 1;
 							$n++;
@@ -131,23 +131,23 @@
 				 */
 				if ($additionl_ip_records != 0)
 				{
-					$swap["ip_address"] = $ip_records[$n-1]["ip_address"];
-					$swap["netmask"] = $ip_records[$n-1]["netmask"];
-					$swap["gateway"] = $ip_records[$n-1]["gateway"];
+					$swap['ip_address'] = $ip_records[$n-1]['ip_address'];
+					$swap['netmask'] = $ip_records[$n-1]['netmask'];
+					$swap['gateway'] = $ip_records[$n-1]['gateway'];
 					
 					$ip_records[$n-1] = array(
 						'id' => $n-1,
 						'ip_address' => $server_config['ip_address'],
-						'netmask' => $server_config["netmask"],
-						'gateway' => $server_config["gateway"],
-						'broadcast' => $this->broadcast($server_config['ip_address'],$server_config["netmask"]),
-						'network' => $this->network($server_config['ip_address'],$server_config["netmask"])
+						'netmask' => $server_config['netmask'],
+						'gateway' => $server_config['gateway'],
+						'broadcast' => $this->broadcast($server_config['ip_address'],$server_config['netmask']),
+						'network' => $this->network($server_config['ip_address'],$server_config['netmask'])
 					);
-					$network_tpl->setVar('ip_address',$swap["ip_address"]);
-					$network_tpl->setVar('netmask',$swap["netmask"]);
-					$network_tpl->setVar('gateway',$swap["gateway"]);
-					$network_tpl->setVar('broadcast',$this->broadcast($swap["ip_address"],$swap["netmask"]));
-					$network_tpl->setVar('network',$this->network($swap["ip_address"],$swap["netmask"]));
+					$network_tpl->setVar('ip_address',$swap['ip_address']);
+					$network_tpl->setVar('netmask',$swap['netmask']);
+					$network_tpl->setVar('gateway',$swap['gateway']);
+					$network_tpl->setVar('broadcast',$this->broadcast($swap['ip_address'],$swap['netmask']));
+					$network_tpl->setVar('network',$this->network($swap['ip_address'],$swap['netmask']));
 				}
 				
 				$network_tpl->setVar('additionl_ip_records',$additionl_ip_records);
@@ -155,7 +155,7 @@
 				file_put_contents('/etc/network/interfaces',$network_tpl->grab());
 				unset($network_tpl);
 				
-				$app->log("Changed Network settings",LOGLEVEL_DEBUG);
+				$app->log('Changed Network settings',LOGLEVEL_DEBUG);
 				exec($conf['init_scripts'] . '/' . 'networking force-reload');
 			} 
 			elseif (is_file('/etc/gentoo-release')) 
@@ -165,14 +165,14 @@
 				$app->load('tpl');
 				
 				$network_tpl = new tpl();
-				$network_tpl->newTemplate("gentoo_network_interfaces.master");
+				$network_tpl->newTemplate('gentoo_network_interfaces.master');
 				
-				$network_tpl->setVar('ip_address',$server_config["ip_address"]);
-				$network_tpl->setVar('netmask',$server_config["netmask"]);
-				$network_tpl->setVar('gateway',$server_config["gateway"]);
-				$network_tpl->setVar('broadcast',$this->broadcast($server_config["ip_address"],$server_config["netmask"]));
+				$network_tpl->setVar('ip_address',$server_config['ip_address']);
+				$network_tpl->setVar('netmask',$server_config['netmask']);
+				$network_tpl->setVar('gateway',$server_config['gateway']);
+				$network_tpl->setVar('broadcast',$this->broadcast($server_config['ip_address'],$server_config['netmask']));
 				
-				$records = $app->db->queryAllRecords("SELECT ip_address FROM server_ip WHERE server_id = ".intval($conf["server_id"]) . " order by ip_address");
+				$records = $app->db->queryAllRecords("SELECT ip_address FROM server_ip WHERE server_id = ".intval($conf['server_id']) . " order by ip_address");
 				$ip_records = array();
 				$additionl_ip_records = 0;
 				$n = 0;
@@ -181,14 +181,14 @@
 						/*
 						 * don't insert the main-ip again!
 						 */
-						if ($rec['ip_address'] != $server_config["ip_address"])
+						if ($rec['ip_address'] != $server_config['ip_address'])
 						{
 							$ip_records[$n] = array(
 								'id' => $n,
 								'ip_address' => $rec['ip_address'],
-								'netmask' => $server_config["netmask"],
-								'gateway' => $server_config["gateway"],
-								'broadcast' => $this->broadcast($rec['ip_address'],$server_config["netmask"])
+								'netmask' => $server_config['netmask'],
+								'gateway' => $server_config['gateway'],
+								'broadcast' => $this->broadcast($rec['ip_address'],$server_config['netmask'])
 							);
 							$additionl_ip_records = 1;
 							$n++;
@@ -205,21 +205,21 @@
 				 */
 				if ($additionl_ip_records != 0)
 				{
-					$swap["ip_address"] = $ip_records[$n-1]["ip_address"];
-					$swap["netmask"] = $ip_records[$n-1]["netmask"];
-					$swap["gateway"] = $ip_records[$n-1]["gateway"];
+					$swap['ip_address'] = $ip_records[$n-1]['ip_address'];
+					$swap['netmask'] = $ip_records[$n-1]['netmask'];
+					$swap['gateway'] = $ip_records[$n-1]['gateway'];
 					
 					$ip_records[$n-1] = array(
 						'id' => $n-1,
 						'ip_address' => $server_config['ip_address'],
-						'netmask' => $server_config["netmask"],
-						'gateway' => $server_config["gateway"],
-						'broadcast' => $this->broadcast($server_config['ip_address'],$server_config["netmask"])
+						'netmask' => $server_config['netmask'],
+						'gateway' => $server_config['gateway'],
+						'broadcast' => $this->broadcast($server_config['ip_address'],$server_config['netmask'])
 					);
-					$network_tpl->setVar('ip_address',$swap["ip_address"]);
-					$network_tpl->setVar('netmask',$swap["netmask"]);
-					$network_tpl->setVar('gateway',$swap["gateway"]);
-					$network_tpl->setVar('broadcast',$this->broadcast($swap["ip_address"],$swap["netmask"]));
+					$network_tpl->setVar('ip_address',$swap['ip_address']);
+					$network_tpl->setVar('netmask',$swap['netmask']);
+					$network_tpl->setVar('gateway',$swap['gateway']);
+					$network_tpl->setVar('broadcast',$this->broadcast($swap['ip_address'],$swap['netmask']));
 				}
 				
 				$network_tpl->setVar('additionl_ip_records',$additionl_ip_records);
@@ -227,59 +227,59 @@
 				file_put_contents('/etc/conf.d/net',$network_tpl->grab());
 				unset($network_tpl);
 				
-				$app->log("Changed Network settings",LOGLEVEL_DEBUG);
+				$app->log('Changed Network settings',LOGLEVEL_DEBUG);
 				exec($conf['init_scripts'] . '/' . 'net.eth0 restart');
 			} 
 			else {
-				$app->log("Network configuration not available for this Linux distribution.",LOGLEVEL_DEBUG);
+				$app->log('Network configuration not available for this Linux distribution.',LOGLEVEL_DEBUG);
 			}
 			
 		} else {
-			$app->log("Network configuration disabled in server settings.",LOGLEVEL_WARN);
+			$app->log('Network configuration disabled in server settings.',LOGLEVEL_WARN);
 		}
 		
 	}
 	
 	function network($ip, $netmask){
 		$netmask = $this->netmask($netmask);
-		list($f1,$f2,$f3,$f4) = explode(".", $netmask);
-		$netmask_bin = str_pad(decbin($f1),8,"0",STR_PAD_LEFT).str_pad(decbin($f2),8,"0",STR_PAD_LEFT).str_pad(decbin($f3),8,"0",STR_PAD_LEFT).str_pad(decbin($f4),8,"0",STR_PAD_LEFT);
-		list($f1,$f2,$f3,$f4) = explode(".", $ip);
-		$ip_bin = str_pad(decbin($f1),8,"0",STR_PAD_LEFT).str_pad(decbin($f2),8,"0",STR_PAD_LEFT).str_pad(decbin($f3),8,"0",STR_PAD_LEFT).str_pad(decbin($f4),8,"0",STR_PAD_LEFT);
+		list($f1,$f2,$f3,$f4) = explode('.', $netmask);
+		$netmask_bin = str_pad(decbin($f1),8,'0',STR_PAD_LEFT).str_pad(decbin($f2),8,'0',STR_PAD_LEFT).str_pad(decbin($f3),8,'0',STR_PAD_LEFT).str_pad(decbin($f4),8,'0',STR_PAD_LEFT);
+		list($f1,$f2,$f3,$f4) = explode('.', $ip);
+		$ip_bin = str_pad(decbin($f1),8,'0',STR_PAD_LEFT).str_pad(decbin($f2),8,'0',STR_PAD_LEFT).str_pad(decbin($f3),8,'0',STR_PAD_LEFT).str_pad(decbin($f4),8,'0',STR_PAD_LEFT);
 		for($i=0;$i<32;$i++){
 			$network_bin .= substr($netmask_bin,$i,1) * substr($ip_bin,$i,1);
 		}
-		$network_bin = wordwrap($network_bin, 8, ".", 1);
-		list($f1,$f2,$f3,$f4) = explode(".", trim($network_bin));
-		return bindec($f1).".".bindec($f2).".".bindec($f3).".".bindec($f4);
+		$network_bin = wordwrap($network_bin, 8, '.', 1);
+		list($f1,$f2,$f3,$f4) = explode('.', trim($network_bin));
+		return bindec($f1).'.'.bindec($f2).'.'.bindec($f3).'.'.bindec($f4);
 	}
 
 	function broadcast($ip, $netmask){
 		$netmask = $this->netmask($netmask);
 		$binary_netmask = $this->binary_netmask($netmask);
-		list($f1,$f2,$f3,$f4) = explode(".", $ip);
-		$ip_bin = str_pad(decbin($f1),8,"0",STR_PAD_LEFT).str_pad(decbin($f2),8,"0",STR_PAD_LEFT).str_pad(decbin($f3),8,"0",STR_PAD_LEFT).str_pad(decbin($f4),8,"0",STR_PAD_LEFT);
-		$broadcast_bin = str_pad(substr($ip_bin, 0, $binary_netmask),32,"1",STR_PAD_RIGHT);
-		$broadcast_bin = wordwrap($broadcast_bin, 8, ".", 1);
-		list($f1,$f2,$f3,$f4) = explode(".", trim($broadcast_bin));
-		return bindec($f1).".".bindec($f2).".".bindec($f3).".".bindec($f4);
+		list($f1,$f2,$f3,$f4) = explode('.', $ip);
+		$ip_bin = str_pad(decbin($f1),8,'0',STR_PAD_LEFT).str_pad(decbin($f2),8,'0',STR_PAD_LEFT).str_pad(decbin($f3),8,'0',STR_PAD_LEFT).str_pad(decbin($f4),8,'0',STR_PAD_LEFT);
+		$broadcast_bin = str_pad(substr($ip_bin, 0, $binary_netmask),32,'1',STR_PAD_RIGHT);
+		$broadcast_bin = wordwrap($broadcast_bin, 8, '.', 1);
+		list($f1,$f2,$f3,$f4) = explode('.', trim($broadcast_bin));
+		return bindec($f1).'.'.bindec($f2).'.'.bindec($f3).'.'.bindec($f4);
 	}
 	
 	function netmask($netmask){
-		list($f1,$f2,$f3,$f4) = explode(".", trim($netmask));
-		$bin = str_pad(decbin($f1),8,"0",STR_PAD_LEFT).str_pad(decbin($f2),8,"0",STR_PAD_LEFT).str_pad(decbin($f3),8,"0",STR_PAD_LEFT).str_pad(decbin($f4),8,"0",STR_PAD_LEFT);
-		$parts = explode("0", $bin);
-		$bin = str_pad($parts[0], 32, "0", STR_PAD_RIGHT);
-		$bin = wordwrap($bin, 8, ".", 1);
-		list($f1,$f2,$f3,$f4) = explode(".", trim($bin));
-		return bindec($f1).".".bindec($f2).".".bindec($f3).".".bindec($f4);
+		list($f1,$f2,$f3,$f4) = explode('.', trim($netmask));
+		$bin = str_pad(decbin($f1),8,'0',STR_PAD_LEFT).str_pad(decbin($f2),8,'0',STR_PAD_LEFT).str_pad(decbin($f3),8,'0',STR_PAD_LEFT).str_pad(decbin($f4),8,'0',STR_PAD_LEFT);
+		$parts = explode('0', $bin);
+		$bin = str_pad($parts[0], 32, '0', STR_PAD_RIGHT);
+		$bin = wordwrap($bin, 8, '.', 1);
+		list($f1,$f2,$f3,$f4) = explode('.', trim($bin));
+		return bindec($f1).'.'.bindec($f2).'.'.bindec($f3).'.'.bindec($f4);
 	}
 
 	function binary_netmask($netmask){
-		list($f1,$f2,$f3,$f4) = explode(".", trim($netmask));
-		$bin = str_pad(decbin($f1),8,"0",STR_PAD_LEFT).str_pad(decbin($f2),8,"0",STR_PAD_LEFT).str_pad(decbin($f3),8,"0",STR_PAD_LEFT).str_pad(decbin($f4),8,"0",STR_PAD_LEFT);
-		$parts = explode("0", $bin);
-		return substr_count($parts[0], "1");
+		list($f1,$f2,$f3,$f4) = explode('.', trim($netmask));
+		$bin = str_pad(decbin($f1),8,'0',STR_PAD_LEFT).str_pad(decbin($f2),8,'0',STR_PAD_LEFT).str_pad(decbin($f3),8,'0',STR_PAD_LEFT).str_pad(decbin($f4),8,'0',STR_PAD_LEFT);
+		$parts = explode('0', $bin);
+		return substr_count($parts[0], '1');
 	}
 
 } // end class

--
Gitblit v1.9.1