diff options
author | Colin Fleming <cj_fleming@sky.com> | 2013-03-22 14:42:57 +0000 |
---|---|---|
committer | Colin Fleming <cj_fleming@sky.com> | 2013-03-22 14:42:57 +0000 |
commit | 2174be2212cc9388600c3387fd61faac1a1e745d (patch) | |
tree | 7b203c233e0b96498a482adb7c1d008f166d5f40 /usr | |
parent | b2f73235f911c800e7dfb24f78581e0356bbb615 (diff) | |
download | pfsense-2174be2212cc9388600c3387fd61faac1a1e745d.zip pfsense-2174be2212cc9388600c3387fd61faac1a1e745d.tar.gz |
Tidy up "traffic shaper limiter" XHTML
Close INPUT, BR, IMG and P tags
Add "alt" to IMG tags
Deprecate ampersand in ANCHOR tags
Add CDATA section to scripts
Move "tree.css" and "tree.js" to the HEAD statement and manually close
the HEAD statement
Spelling mistatke
Updated PHP shorthand tag
Diffstat (limited to 'usr')
-rw-r--r-- | usr/local/www/firewall_shaper_vinterface.php | 43 |
1 files changed, 24 insertions, 19 deletions
diff --git a/usr/local/www/firewall_shaper_vinterface.php b/usr/local/www/firewall_shaper_vinterface.php index 9893de1..3870e1c 100644 --- a/usr/local/www/firewall_shaper_vinterface.php +++ b/usr/local/www/firewall_shaper_vinterface.php @@ -152,7 +152,7 @@ if ($_GET) { $q = new dnqueue_class(); $q->SetPipe($pipe); $output_form .= "<input type=\"hidden\" name=\"parentqueue\" id=\"parentqueue\""; - $output_form .= " value=\"".$pipe."\">"; + $output_form .= " value=\"".$pipe."\" />"; } else if ($addnewpipe) { $q = new dnpipe_class(); $q->SetQname($pipe); @@ -321,23 +321,23 @@ if ($can_add || $addnewaltq) { $output_form .= "<a href=\"firewall_shaper_vinterface.php?pipe="; $output_form .= $pipe; if ($queue) { - $output_form .= "&queue=" . $queue->GetQname(); + $output_form .= "&queue=" . $queue->GetQname(); } - $output_form .= "&action=add\">"; - $output_form .= "<input type=\"button\" class=\"formbtn\" name=\"add\" value=\"" . gettext("Add new queue") ."\">"; + $output_form .= "&action=add\">"; + $output_form .= "<input type=\"button\" class=\"formbtn\" name=\"add\" value=\"" . gettext("Add new queue") ."\" />"; $output_form .= "</a>"; } $output_form .= "<a href=\"firewall_shaper_vinterface.php?pipe="; $output_form .= $pipe; if ($queue) { - $output_form .= "&queue=" . $queue->GetQname(); + $output_form .= "&queue=" . $queue->GetQname(); } -$output_form .= "&action=delete\">"; +$output_form .= "&action=delete\">"; $output_form .= "<input type=\"button\" class=\"formbtn\" name=\"delete\""; if ($queue) - $output_form .= " value=\"" . gettext("Delete this queue") ."\">"; + $output_form .= " value=\"" . gettext("Delete this queue") ."\" />"; else - $output_form .= " value=\"" . gettext("Delete virtual interface") ."\">"; + $output_form .= " value=\"" . gettext("Delete virtual interface") ."\" />"; $output_form .= "</a>"; $output_form .= "</td></tr>"; $output_form .= "</div>"; @@ -347,14 +347,13 @@ else $output = "<div id=\"shaperarea\" style=\"position:relative\">"; $output .= $output_form; - +$closehead = false; include("head.inc"); ?> - -<body link="#0000CC" vlink="#0000CC" alink="#0000CC" > <link rel="stylesheet" type="text/css" media="all" href="./tree/tree.css" /> <script type="text/javascript" src="./tree/tree.js"></script> <script type="text/javascript"> +//<![CDATA[ function show_source_port_range() { document.getElementById("sprtable").style.display = ''; document.getElementById("sprtable1").style.display = ''; @@ -363,7 +362,11 @@ function show_source_port_range() { document.getElementById("sprtable4").style.display = 'none'; document.getElementById("showadvancedboxspr").innerHTML=''; } +//]]> </script> +</head> + +<body link="#0000CC" vlink="#0000CC" alink="#0000CC"> <?php if ($queue) @@ -380,9 +383,9 @@ include("fbegin.inc"); <?php if ($savemsg) print_info_box($savemsg); ?> <?php if (is_subsystem_dirty('shaper')): ?><p> -<?php print_info_box_np(gettext("The traffic shaper configuration has been changed.")."<br>".gettext("You must apply the changes in order for them to take effect."));?><br> +<?php print_info_box_np(gettext("The traffic shaper configuration has been changed.")."<br/>".gettext("You must apply the changes in order for them to take effect."));?><br/></p> <?php endif; ?> -<table width="100%" border="0" cellpadding="0" cellspacing="0"> +<table width="100%" border="0" cellpadding="0" cellspacing="0" summary="traffic shaper limiter"> <tr><td> <?php $tab_array = array(); @@ -397,24 +400,24 @@ include("fbegin.inc"); <tr> <td> <div id="mainarea"> - <table class="tabcont" width="100%" border="0" cellpadding="0" cellspacing="0"> + <table class="tabcont" width="100%" border="0" cellpadding="0" cellspacing="0" summary="main area"> <?php if (count($dummynet_pipe_list) > 0): ?> <tr class="tabcont"><td width="25%" align="left"> </td><td width="75%"> </td></tr> <?php endif; ?> <tr> - <td width="25%" valign="top" algin="left"> + <td width="25%" valign="top" align="left"> <?php echo $tree; ?> <br/><br/> - <a href="firewall_shaper_vinterface.php?pipe=new&action=add"> - <img src="./themes/<?= $g['theme']; ?>/images/icons/icon_plus.gif" title="<?=gettext("Create new limiter");?>" width="17" height="17" border="0"><?=gettext("Create new limiter");?> + <a href="firewall_shaper_vinterface.php?pipe=new&action=add"> + <img src="./themes/<?= $g['theme']; ?>/images/icons/icon_plus.gif" title="<?=gettext("Create new limiter");?>" width="17" height="17" border="0" alt="add" /><?=gettext("Create new limiter");?> </a><br/> </td> <td width="75%" valign="top" align="center"> - <table> - <? + <table summary="output form"> + <?php echo $output; ?> </table> @@ -427,12 +430,14 @@ include("fbegin.inc"); </table> </form> <script type='text/javascript'> +//<![CDATA[ <?php $totalrows = 0; if (is_array($config['dnshaper']) && is_array($config['dnshaper']['queue'])) $totalrows = count($config['dnshaper']['queue']); echo "totalrows = {$totalrows}"; ?> +//]]> </script> <?php include("fend.inc"); ?> </body> |