From 73672832043d999c848c9cc578f48490cbdd03ec Mon Sep 17 00:00:00 2001 From: Colin Fleming Date: Thu, 3 Apr 2014 15:18:53 +0100 Subject: Tidy up "services_captiveportal.php" XHTML Add "closehead" variable, move script into HEAD and close head manually Add CDATA sections to SCRIPT Add SUMMARY to tables Update HTML Boolean Operators Close INPUT Remove duplicate closing TR Add opening TR Add closing TD Change "_new" to "_blank" in anchor tag --- usr/local/www/services_captiveportal.php | 159 ++++++++++++++++--------------- 1 file changed, 82 insertions(+), 77 deletions(-) (limited to 'usr/local/www') diff --git a/usr/local/www/services_captiveportal.php b/usr/local/www/services_captiveportal.php index 4cd2127..d51d8e4 100644 --- a/usr/local/www/services_captiveportal.php +++ b/usr/local/www/services_captiveportal.php @@ -412,10 +412,11 @@ if ($_POST) { $pconfig['cinterface'] = implode(",", $_POST['cinterface']); } } +$closehead = false; include("head.inc"); ?> +
- +
- +
@@ -600,21 +602,21 @@ function enable_change(enable_change) { @@ -655,59 +657,61 @@ function enable_change(enable_change) { @@ -960,7 +965,7 @@ function enable_change(enable_change) { foreach($a_cert as $cert): $selected = ""; if ($pconfig['certref'] == $cert['refid']) - $selected = "selected"; + $selected = "selected=\"selected\""; ?> @@ -973,7 +978,7 @@ function enable_change(enable_change) { @@ -1077,10 +1082,10 @@ function enable_change(enable_change) {
  - onClick="enable_change(false)"> + onclick="enable_change(false)" />
- $ifacename): ?> - @@ -531,9 +533,9 @@ function enable_change(enable_change) {
- +
- +
- +
- +
- +
- +
- > + />
- > + />
-
+
- +
-
+
- > + />
- > + />
@@ -623,13 +625,13 @@ function enable_change(enable_change) {
- > + />


- > + />
@@ -638,16 +640,16 @@ function enable_change(enable_change) {
- > + />

- +
- + - +

- +
- - - + - - +
> + />
> + /> /
   
 > + />
> + />
   
- +
- - - -
> + />
> + />
> + />
> + />
   
+
   
- +
@@ -716,17 +720,17 @@ function enable_change(enable_change) { - - - @@ -737,16 +741,16 @@ function enable_change(enable_change) { - - + - + @@ -759,16 +763,16 @@ function enable_change(enable_change) { - - + - + @@ -778,16 +782,16 @@ function enable_change(enable_change) { - - + - + @@ -797,13 +801,13 @@ function enable_change(enable_change) { - - @@ -812,9 +816,9 @@ function enable_change(enable_change) { @@ -825,7 +829,7 @@ function enable_change(enable_change) { - @@ -833,13 +837,13 @@ function enable_change(enable_change) { - + @@ -851,7 +855,7 @@ function enable_change(enable_change) { if (is_ipaddr($ipaddr)) { $selected = ""; if ($ifdesc == $pconfig['radiussrcip_attribute']) - $selected= "selected"; + $selected= "selected=\"selected\""; echo "\n"; } } @@ -878,7 +882,7 @@ function enable_change(enable_change) { - @@ -890,7 +894,7 @@ function enable_change(enable_change) { $radiusvendors = array("cisco"); foreach ($radiusvendors as $radiusvendor){ if ($pconfig['radiusvendor'] == $radiusvendor) - echo "\n"; + echo "\n"; else echo "\n"; } @@ -902,13 +906,13 @@ function enable_change(enable_change) { - - @@ -920,7 +924,7 @@ function enable_change(enable_change) { $macformats = array("singledash","ietf","cisco","unformatted"); foreach ($macformats as $macformat) { if ($pconfig['radmac_format'] == $macformat) { - echo "\n"; + echo "\n"; } else { echo "\n"; } @@ -937,18 +941,19 @@ function enable_change(enable_change) {
Primary Authentication Source

+

+
  
+

+
  

+
  

+
  
 > + />

+
- >
- >
- > + />
+ />
+ />
> + />
- >
+ />
>
+ 		/>
>
+ 		/>

+
+
- > + />
-
+
-
+
- +

@@ -1024,7 +1029,7 @@ function enable_change(enable_change) {
-
+

@@ -1044,7 +1049,7 @@ function enable_change(enable_change) {
-
+

@@ -1061,9 +1066,9 @@ function enable_change(enable_change) {
  - "; ?> - " onClick="enable_change(true)"> - " onClick="enable_change(true)"> + "; ?> + " onclick="enable_change(true)" /> + " onclick="enable_change(true)" />
- + \ No newline at end of file -- cgit v1.1