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/firewall_plugin.inc.php | 20 ++++++++++---------- 1 files changed, 10 insertions(+), 10 deletions(-) diff --git a/server/plugins-available/firewall_plugin.inc.php b/server/plugins-available/firewall_plugin.inc.php index 9039946..d9d1272 100644 --- a/server/plugins-available/firewall_plugin.inc.php +++ b/server/plugins-available/firewall_plugin.inc.php @@ -76,7 +76,7 @@ $tcp_ports = ''; $udp_ports = ''; - $ports = explode(',',$data["new"]["tcp_port"]); + $ports = explode(',',$data['new']['tcp_port']); if(is_array($ports)) { foreach($ports as $p) { if(strstr($p,':')) { @@ -90,7 +90,7 @@ } $tcp_ports = trim($tcp_ports); - $ports = explode(',',$data["new"]["udp_port"]); + $ports = explode(',',$data['new']['udp_port']); if(is_array($ports)) { foreach($ports as $p) { if(strstr($p,':')) { @@ -106,21 +106,21 @@ $app->load('tpl'); $tpl = new tpl(); - $tpl->newTemplate("bastille-firewall.cfg.master"); + $tpl->newTemplate('bastille-firewall.cfg.master'); - $tpl->setVar("TCP_PUBLIC_SERVICES",$tcp_ports); - $tpl->setVar("UDP_PUBLIC_SERVICES",$udp_ports); + $tpl->setVar('TCP_PUBLIC_SERVICES',$tcp_ports); + $tpl->setVar('UDP_PUBLIC_SERVICES',$udp_ports); file_put_contents('/etc/Bastille/bastille-firewall.cfg',$tpl->grab()); $app->log('Writing firewall configuration /etc/Bastille/bastille-firewall.cfg',LOGLEVEL_DEBUG); unset($tpl); - if($data["new"]["active"] == 'y') { - exec('/etc/init.d/bastille-firewall restart'); + if($data['new']['active'] == 'y') { + exec($conf['init_scripts'] . '/' . 'bastille-firewall restart'); if(@is_file('/etc/debian_version')) exec('update-rc.d bastille-firewall defaults'); $app->log('Restarting the firewall',LOGLEVEL_DEBUG); } else { - exec('/etc/init.d/bastille-firewall stop'); + exec($conf['init_scripts'] . '/' . 'bastille-firewall stop'); if(@is_file('/etc/debian_version')) exec('update-rc.d -f bastille-firewall remove'); $app->log('Stopping the firewall',LOGLEVEL_DEBUG); } @@ -131,7 +131,7 @@ function delete($event_name,$data) { global $app, $conf; - exec('/etc/init.d/bastille-firewall stop'); + exec($conf['init_scripts'] . '/' . 'bastille-firewall stop'); if(@is_file('/etc/debian_version')) exec('update-rc.d -f bastille-firewall remove'); $app->log('Stopping the firewall',LOGLEVEL_DEBUG); @@ -142,4 +142,4 @@ } // end class -?> \ No newline at end of file +?> -- Gitblit v1.9.1