diff options
author | Colin Fleming <cj_fleming@sky.com> | 2014-05-16 21:12:58 +0100 |
---|---|---|
committer | Colin Fleming <cj_fleming@sky.com> | 2014-05-16 21:12:58 +0100 |
commit | 0cb6064557e1d762ae4892bcb715a27dd0bcbac7 (patch) | |
tree | ca0b8c96aae124522f2570468720a08c3ce2ee91 /usr/local/www/status_captiveportal_test.php | |
parent | 024068017f98572f6bd41b7ba542fc36fccee39a (diff) | |
download | pfsense-0cb6064557e1d762ae4892bcb715a27dd0bcbac7.zip pfsense-0cb6064557e1d762ae4892bcb715a27dd0bcbac7.tar.gz |
Tidy up "status_captiveportal" XHTML
Add SUMMARY to TABLES
Remove PHP ECHO that isn't in a PHP statement
Update HTML Boolean operator
Add COLSPAN to fill the width of the table
Move NOWRAP to CLASS statement
Close INPUT and IMG tags and add ALT to IMG
Add BODY between HEAD.INC and FBEGIN.INC
Remove TYPE from TEXTAREA, invalid in XHTML
Add closing BODY and closing HTML tags
Diffstat (limited to 'usr/local/www/status_captiveportal_test.php')
-rw-r--r-- | usr/local/www/status_captiveportal_test.php | 22 |
1 files changed, 12 insertions, 10 deletions
diff --git a/usr/local/www/status_captiveportal_test.php b/usr/local/www/status_captiveportal_test.php index 47d64c9..dd065ad 100644 --- a/usr/local/www/status_captiveportal_test.php +++ b/usr/local/www/status_captiveportal_test.php @@ -59,9 +59,9 @@ $pgtitle = array(gettext("Status"), gettext("Captive portal"), gettext("Test Vou $shortcut_section = "captiveportal-vouchers"; include("head.inc"); -include("fbegin.inc"); - ?> +<body> +<?php include("fbegin.inc"); ?> <form action="status_captiveportal_test.php" method="post" enctype="multipart/form-data" name="iform" id="iform"> <table width="100%" border="0" cellpadding="0" cellspacing="0" summary="tab pane"> @@ -83,38 +83,40 @@ include("fbegin.inc"); <tr> <td valign="top" class="vncellreq"><?=gettext("Voucher(s)"); ?></td> <td class="vtable"> - <textarea name="vouchers" cols="65" rows="3" type="text" id="vouchers" class="formpre"><?=htmlspecialchars($_POST['vouchers']);?></textarea> + <textarea name="vouchers" cols="65" rows="3" id="vouchers" class="formpre"><?=htmlspecialchars($_POST['vouchers']);?></textarea> <br /> <?=gettext("Enter multiple vouchers separated by space or newline. The remaining time, if valid, will be shown for each voucher"); ?>.</td> </tr> <tr> <td width="22%" valign="top"> </td> <td width="78%"> - <input name="zone" type="hidden" value="<?=htmlspecialchars($cpzone);?>"> - <input name="Submit" type="submit" class="formbtn" value="<?=gettext("Submit"); ?>"> + <input name="zone" type="hidden" value="<?=htmlspecialchars($cpzone);?>" /> + <input name="Submit" type="submit" class="formbtn" value="<?=gettext("Submit"); ?>" /> </td> </tr> </table> </td></tr></table> </form> -<p> +<br/> <?php if ($_POST) { if ($_POST['vouchers']) { $test_results = voucher_auth($_POST['vouchers'], 1); - echo "<p><table border=\"0\" cellspacing=\"0\" cellpadding=\"4\" width=\"100%\">\n"; + echo "<table border=\"0\" cellspacing=\"0\" cellpadding=\"4\" width=\"100%\" summary=\"results\">\n"; foreach ($test_results as $result) { if (strpos($result, " good ") || strpos($result, " granted ")) { - echo "<tr><td bgcolor=\"#D9DEE8\"><img src=\"/themes/{$g['theme']}/images/icons/icon_pass.gif\"></td>"; + echo "<tr><td bgcolor=\"#D9DEE8\"><img src=\"/themes/{$g['theme']}/images/icons/icon_pass.gif\" alt=\"pass\" /></td>"; echo "<td bgcolor=\"#D9DEE8\">$result</td></tr>"; } else { - echo "<tr><td bgcolor=\"#FFD9D1\"><img src=\"/themes/{$g['theme']}/images/icons/icon_block.gif\"></td>"; + echo "<tr><td bgcolor=\"#FFD9D1\"><img src=\"/themes/{$g['theme']}/images/icons/icon_block.gif\" alt=\"block\" /></td>"; echo "<td bgcolor=\"#FFD9D1\">$result</td></tr>"; } } - echo "</table></p>"; + echo "</table>"; } } include("fend.inc"); ?> +</body> +</html> |