Coe style firewall *

1 files changed, 15 insertions, 9 deletions

diff --git a/src/usr/local/www/firewall_shaper_queues.php b/src/usr/local/www/firewall_shaper_queues.php
index 2794a07..0e67215 100644
--- a/src/usr/local/www/firewall_shaper_queues.php
+++ b/src/usr/local/www/firewall_shaper_queues.php
@@ -160,8 +160,9 @@ if ($_GET) {
 						$config['shaper']['queue'][] = $newroot;
 					}
 
-					if (write_config())
+					if (write_config()) {
 						mark_subsystem_dirty('shaper');
+					}
 
 					break;
 					}
@@ -177,17 +178,19 @@ if ($_GET) {
 				if ($altq) {
 					$qtmp =& $altq->find_queue("", $qname);
 
-					if ($qtmp)
+					if ($qtmp) {
 						$output .= $qtmp->build_shortform();
-					else
+					} else {
 						$output .= build_iface_without_this_queue($if, $qname);
-
+					}
 				} else {
-					if (!is_altq_capable($ifdesc['if']))
+					if (!is_altq_capable($ifdesc['if'])) {
 						continue;
+					}
 
-					if (!isset($ifdesc['enable']) && $if != "lan" && $if != "wan")
+					if (!isset($ifdesc['enable']) && $if != "lan" && $if != "wan") {
 						continue;
+					}
 
 					$output .= build_iface_without_this_queue($if, $qname);
 				}
@@ -228,14 +231,17 @@ include("head.inc");
 <script type="text/javascript" src="./tree/tree.js"></script>
 
 <?php
-if ($input_errors)
+if ($input_errors) {
 	print_input_errors($input_errors);
+}
 
-if ($savemsg)
+if ($savemsg) {
 	print_info_box($savemsg);
+}
 
-if (is_subsystem_dirty('shaper'))
+if (is_subsystem_dirty('shaper')) {
 	print_info_box_np(gettext("The traffic shaper configuration has been changed. You must apply the changes in order for them to take effect."));
+}
 
 $tab_array = array();
 $tab_array[] = array(gettext("By Interface"), false, "firewall_shaper.php");