diff options
author | Phil Davis <phil.davis@inf.org> | 2015-05-24 12:54:59 +0545 |
---|---|---|
committer | Phil Davis <phil.davis@inf.org> | 2015-05-24 12:54:59 +0545 |
commit | c0bf78583c54a68b7343d47589e3bafaca5f5036 (patch) | |
tree | 4324b3c55054f9420542a71bbb64c589ecab9def /usr/local | |
parent | 88cbd0049b78bd731648be68c13e08161e5e943c (diff) | |
download | pfsense-c0bf78583c54a68b7343d47589e3bafaca5f5036.zip pfsense-c0bf78583c54a68b7343d47589e3bafaca5f5036.tar.gz |
Code style services igmpproxy
Diffstat (limited to 'usr/local')
-rw-r--r-- | usr/local/www/services_igmpproxy.php | 175 | ||||
-rw-r--r-- | usr/local/www/services_igmpproxy_edit.php | 265 |
2 files changed, 235 insertions, 205 deletions
diff --git a/usr/local/www/services_igmpproxy.php b/usr/local/www/services_igmpproxy.php index 99c5181..8749b2d 100644 --- a/usr/local/www/services_igmpproxy.php +++ b/usr/local/www/services_igmpproxy.php @@ -2,7 +2,7 @@ /* $Id$ */ /* services_igmpproxy.php - + Copyright (C) 2013-2015 Electric Sheep Fencing, LP Copyright (C) 2009 Ermal Luçi Copyright (C) 2004 Scott Ullrich @@ -34,7 +34,7 @@ POSSIBILITY OF SUCH DAMAGE. */ /* - pfSense_MODULE: dyndns + pfSense_MODULE: igmpproxy */ ##|+PRIV @@ -46,8 +46,9 @@ require("guiconfig.inc"); -if (!is_array($config['igmpproxy']['igmpentry'])) +if (!is_array($config['igmpproxy']['igmpentry'])) { $config['igmpproxy']['igmpentry'] = array(); +} //igmpproxy_sort(); $a_igmpproxy = &$config['igmpproxy']['igmpentry']; @@ -60,10 +61,11 @@ if ($_POST) { /* reload all components that use igmpproxy */ $retval = services_igmpproxy_configure(); - if(stristr($retval, "error") <> true) - $savemsg = get_std_save_message($retval); - else - $savemsg = $retval; + if (stristr($retval, "error") <> true) { + $savemsg = get_std_save_message($retval); + } else { + $savemsg = $retval; + } clear_subsystem_dirty('igmpproxy'); } @@ -78,7 +80,7 @@ if ($_GET['act'] == "del") { } } -$pgtitle = array(gettext("Services"),gettext("IGMP Proxy")); +$pgtitle = array(gettext("Services"), gettext("IGMP Proxy")); include("head.inc"); ?> @@ -92,75 +94,94 @@ include("head.inc"); <?php endif; ?> <table class="tabcont" width="100%" border="0" cellpadding="0" cellspacing="0" summary="igmp proxy"> -<tr> - <td width="15%" class="listhdrr"><?=gettext("Name");?></td> - <td width="10%" class="listhdrr"><?=gettext("Type");?></td> - <td width="25%" class="listhdrr"><?=gettext("Values");?></td> - <td width="25%" class="listhdr"><?=gettext("Description");?></td> - <td width="5%" class="list"> - <table border="0" cellspacing="0" cellpadding="1" summary="add"> - <tr> - <td valign="middle" width="17"> </td> - <td valign="middle"><a href="services_igmpproxy_edit.php"><img src="/themes/<?= $g['theme']; ?>/images/icons/icon_plus.gif" width="17" height="17" border="0" title="<?=gettext("add a new igmpentry");?>" alt="add" /></a></td> - </tr> - </table> - </td> -</tr> - <?php $i = 0; foreach ($a_igmpproxy as $igmpentry): ?> -<tr> - <td class="listlr" ondblclick="document.location='services_igmpproxy_edit.php?id=<?=$i;?>';"> - <?=htmlspecialchars(convert_friendly_interface_to_friendly_descr($igmpentry['ifname']));?> - </td> - <td class="listlr" ondblclick="document.location='services_igmpproxy_edit.php?id=<?=$i;?>';"> - <?=htmlspecialchars($igmpentry['type']);?> - </td> - <td class="listr" ondblclick="document.location='services_igmpproxy_edit.php?id=<?=$i;?>';"> - <?php - $addresses = implode(", ", array_slice(explode(" ", $igmpentry['address']), 0, 10)); - echo $addresses; - if(count($addresses) < 10) { - echo " "; - } else { - echo "..."; - } - ?> - </td> - <td class="listbg" ondblclick="document.location='services_igmpproxy_edit.php?id=<?=$i;?>';"> - <?=htmlspecialchars($igmpentry['descr']);?> - </td> - <td valign="middle" class="list nowrap"> - <table border="0" cellspacing="0" cellpadding="1" summary="icons"> - <tr> - <td valign="middle"><a href="services_igmpproxy_edit.php?id=<?=$i;?>"><img src="/themes/<?= $g['theme']; ?>/images/icons/icon_e.gif" width="17" height="17" border="0" title="<?=gettext("edit igmpentry"); ?>" alt="edit" /></a></td> - <td><a href="services_igmpproxy.php?act=del&id=<?=$i;?>" onclick="return confirm('<?=gettext("Do you really want to delete this igmp entry? All elements that still use it will become invalid (e.g. filter rules)!");?>')"><img src="/themes/<?= $g['theme']; ?>/images/icons/icon_x.gif" width="17" height="17" border="0" title="<?=gettext("delete igmpentry");?>" alt="delete" /></a></td> - </tr> - </table> - </td> -</tr> - <?php $i++; endforeach; ?> -<tr> - <td class="list" colspan="4"></td> - <td class="list"> - <table border="0" cellspacing="0" cellpadding="1"> - <tr> - <td valign="middle" width="17"> </td> - <td valign="middle"><a href="services_igmpproxy_edit.php"><img src="/themes/<?= $g['theme']; ?>/images/icons/icon_plus.gif" width="17" height="17" border="0" title="<?=gettext("add a new igmpentry");?>" alt="add" /></a></td> - </tr> - </table> - </td> -</tr> - <tr> - <td colspan="2" width="78%"> - <br /> - <input id="submit" name="submit" type="submit" class="formbtn" value="<?=gettext("Save");?>" /> - <br /> - </td> - </tr> -<tr> - <td class="tabcont" colspan="4"> - <p><span class="vexpl"><span class="red"><strong><?=gettext("Note:");?><br /></strong></span><?=gettext("Please add the interface for upstream, the allowed subnets, and the downstream interfaces you would like the proxy to allow. Only one 'upstream' interface can be configured.");?></span></p> - </td> -</tr> + <tr> + <td width="15%" class="listhdrr"><?=gettext("Name");?></td> + <td width="10%" class="listhdrr"><?=gettext("Type");?></td> + <td width="25%" class="listhdrr"><?=gettext("Values");?></td> + <td width="25%" class="listhdr"><?=gettext("Description");?></td> + <td width="5%" class="list"> + <table border="0" cellspacing="0" cellpadding="1" summary="add"> + <tr> + <td valign="middle" width="17"> </td> + <td valign="middle"> + <a href="services_igmpproxy_edit.php"><img src="/themes/<?= $g['theme']; ?>/images/icons/icon_plus.gif" width="17" height="17" border="0" title="<?=gettext("add a new igmpentry");?>" alt="add" /></a> + </td> + </tr> + </table> + </td> + </tr> +<?php + $i = 0; + foreach ($a_igmpproxy as $igmpentry): +?> + <tr> + <td class="listlr" ondblclick="document.location='services_igmpproxy_edit.php?id=<?=$i;?>';"> + <?=htmlspecialchars(convert_friendly_interface_to_friendly_descr($igmpentry['ifname']));?> + </td> + <td class="listlr" ondblclick="document.location='services_igmpproxy_edit.php?id=<?=$i;?>';"> + <?=htmlspecialchars($igmpentry['type']);?> + </td> + <td class="listr" ondblclick="document.location='services_igmpproxy_edit.php?id=<?=$i;?>';"> +<?php + $addresses = implode(", ", array_slice(explode(" ", $igmpentry['address']), 0, 10)); + echo $addresses; + if (count($addresses) < 10) { + echo " "; + } else { + echo "..."; + } +?> + </td> + <td class="listbg" ondblclick="document.location='services_igmpproxy_edit.php?id=<?=$i;?>';"> + <?=htmlspecialchars($igmpentry['descr']);?> + </td> + <td valign="middle" class="list nowrap"> + <table border="0" cellspacing="0" cellpadding="1" summary="icons"> + <tr> + <td valign="middle"><a href="services_igmpproxy_edit.php?id=<?=$i;?>"><img src="/themes/<?= $g['theme']; ?>/images/icons/icon_e.gif" width="17" height="17" border="0" title="<?=gettext("edit igmpentry"); ?>" alt="edit" /></a></td> + <td> + <a href="services_igmpproxy.php?act=del&id=<?=$i;?>" onclick="return confirm('<?=gettext("Do you really want to delete this igmp entry? All elements that still use it will become invalid (e.g. filter rules)!");?>')"><img src="/themes/<?= $g['theme']; ?>/images/icons/icon_x.gif" width="17" height="17" border="0" title="<?=gettext("delete igmpentry");?>" alt="delete" /></a> + </td> + </tr> + </table> + </td> + </tr> +<?php + $i++; + endforeach; +?> + <tr> + <td class="list" colspan="4"></td> + <td class="list"> + <table border="0" cellspacing="0" cellpadding="1"> + <tr> + <td valign="middle" width="17"> </td> + <td valign="middle"><a href="services_igmpproxy_edit.php"><img src="/themes/<?= $g['theme']; ?>/images/icons/icon_plus.gif" width="17" height="17" border="0" title="<?=gettext("add a new igmpentry");?>" alt="add" /></a></td> + </tr> + </table> + </td> + </tr> + <tr> + <td colspan="2" width="78%"> + <br /> + <input id="submit" name="submit" type="submit" class="formbtn" value="<?=gettext("Save");?>" /> + <br /> + </td> + </tr> + <tr> + <td class="tabcont" colspan="4"> + <p> + <span class="vexpl"> + <span class="red"> + <strong> + <?=gettext("Note:");?><br /> + </strong> + </span> + <?=gettext("Please add the interface for upstream, the allowed subnets, and the downstream interfaces you would like the proxy to allow. Only one 'upstream' interface can be configured.");?> + </span> + </p> + </td> + </tr> </table> </form> <?php include("fend.inc"); ?> diff --git a/usr/local/www/services_igmpproxy_edit.php b/usr/local/www/services_igmpproxy_edit.php index 8704978..935bf49 100644 --- a/usr/local/www/services_igmpproxy_edit.php +++ b/usr/local/www/services_igmpproxy_edit.php @@ -43,20 +43,23 @@ ##|*MATCH=services_igmpproxy_edit.php* ##|-PRIV -$pgtitle = array(gettext("Firewall"),gettext("IGMP Proxy"), gettext("Edit")); +$pgtitle = array(gettext("Firewall"), gettext("IGMP Proxy"), gettext("Edit")); require("guiconfig.inc"); -if (!is_array($config['igmpproxy']['igmpentry'])) +if (!is_array($config['igmpproxy']['igmpentry'])) { $config['igmpproxy']['igmpentry'] = array(); +} //igmpproxy_sort(); $a_igmpproxy = &$config['igmpproxy']['igmpentry']; -if (is_numericint($_GET['id'])) +if (is_numericint($_GET['id'])) { $id = $_GET['id']; -if (isset($_POST['id']) && is_numericint($_POST['id'])) +} +if (isset($_POST['id']) && is_numericint($_POST['id'])) { $id = $_POST['id']; +} if (isset($id) && $a_igmpproxy[$id]) { $pconfig['ifname'] = $a_igmpproxy[$id]['ifname']; @@ -74,10 +77,12 @@ if ($_POST) { if ($_POST['type'] == "upstream") { foreach ($a_igmpproxy as $pid => $proxyentry) { - if (isset($id) && $id == $pid) + if (isset($id) && $id == $pid) { continue; - if ($proxyentry['type'] == "upstream" && $proxyentry['ifname'] != $_POST['interface']) + } + if ($proxyentry['type'] == "upstream" && $proxyentry['ifname'] != $_POST['interface']) { $input_errors[] = gettext("Only one 'upstream' interface can be configured."); + } } } $igmpentry = array(); @@ -87,10 +92,11 @@ if ($_POST) { $address = ""; $isfirst = 0; /* item is a normal igmpentry type */ - for($x=0; $x<4999; $x++) { - if($_POST["address{$x}"] <> "") { - if ($isfirst > 0) + for ($x=0; $x<4999; $x++) { + if ($_POST["address{$x}"] <> "") { + if ($isfirst > 0) { $address .= " "; + } $address .= $_POST["address{$x}"]; $address .= "/" . $_POST["address_subnet{$x}"]; $isfirst++; @@ -101,20 +107,19 @@ if ($_POST) { $igmpentry['address'] = $address; $igmpentry['descr'] = $_POST['descr']; - if (isset($id) && $a_igmpproxy[$id]) + if (isset($id) && $a_igmpproxy[$id]) { $a_igmpproxy[$id] = $igmpentry; - else + } else { $a_igmpproxy[] = $igmpentry; + } write_config(); mark_subsystem_dirty('igmpproxy'); header("Location: services_igmpproxy.php"); - exit; - } - //we received input errors, copy data to prevent retype - else - { + exit; + } else { + //we received input errors, copy data to prevent retype $pconfig['descr'] = $_POST['descr']; $pconfig['address'] = $address; $pconfig['type'] = $_POST['type']; @@ -158,135 +163,139 @@ include("head.inc"); <div id="inputerrors"></div> <form action="services_igmpproxy_edit.php" method="post" name="iform" id="iform"> -<table width="100%" border="0" cellpadding="6" cellspacing="0" summary="igmp proxy edit"> - <tr> - <td colspan="2" valign="top" class="listtopic"><?=gettext("IGMP Proxy Edit");?></td> - </tr> - <tr> - <td valign="top" class="vncellreq"><?=gettext("Interface");?></td> - <td class="vtable"> <select name="ifname" id="ifname" > - <?php $iflist = get_configured_interface_with_descr(); - foreach ($iflist as $ifnam => $ifdescr) { - echo "<option value=\"{$ifnam}\""; - if ($ifnam == $pconfig['ifname']) - echo " selected=\"selected\""; - echo ">{$ifdescr}</option>"; - } - ?> - </select> - </td> - </tr> - <tr> - <td width="22%" valign="top" class="vncell"><?=gettext("Description");?></td> - <td width="78%" class="vtable"> - <input name="descr" type="text" class="formfld unknown" id="descr" size="40" value="<?=htmlspecialchars($pconfig['descr']);?>" /> - <br /> - <span class="vexpl"> - <?=gettext("You may enter a description here for your reference (not parsed).");?> - </span> - </td> - </tr> - <tr> - <td valign="top" class="vncellreq"><?=gettext("Type");?></td> - <td class="vtable"> - <select name="type" class="formselect" id="type" > - <option value="upstream" <?php if ($pconfig['type'] == "upstream") echo "selected=\"selected\""; ?>><?=gettext("Upstream Interface");?></option> - <option value="downstream" <?php if ($pconfig['type'] == "downstream") echo "selected=\"selected\""; ?>><?=gettext("Downstream Interface");?></option> - </select> - <br /> - <span class="vexpl"> - <?=gettext("The <b>upstream</b> network interface is the outgoing interface which is". - " responsible for communicating to available multicast data sources.". - " There can only be one upstream interface.");?> - </span> - <br /> - <span class="vexpl"> - <b><?=gettext("Downstream"); ?></b> <?=gettext("network interfaces are the distribution interfaces to the". - " destination networks, where multicast clients can join groups and". - " receive multicast data. One or more downstream interfaces must be configured.");?> - </span> - </td> - </tr> - <tr> - <td valign="top" class="vncell"><?=gettext("Threshold");?></td> - <td class="vtable"> - <input name="threshold" class="formfld unknown" id="threshold" value="<?php echo htmlspecialchars($pconfig['threshold']);?>" /> - <br /> - <span class="vexpl"> - <?=gettext("Defines the TTL threshold for the network interface. Packets". - " with a lower TTL than the threshold value will be ignored. This". - " setting is optional, and by default the threshold is 1.");?> - </span> - </td> - </tr> - <tr> - <td width="22%" valign="top" class="vncellreq"><div id="addressnetworkport"><?=gettext("Network (s)");?></div></td> - <td width="78%" class="vtable"> - <table id="maintable"> - <tbody> - <tr> - <td><div id="onecolumn"><?=gettext("Network");?></div></td> - <td><div id="twocolumn"><?=gettext("CIDR");?></div></td> - </tr> - - <?php + <table width="100%" border="0" cellpadding="6" cellspacing="0" summary="igmp proxy edit"> + <tr> + <td colspan="2" valign="top" class="listtopic"><?=gettext("IGMP Proxy Edit");?></td> + </tr> + <tr> + <td valign="top" class="vncellreq"><?=gettext("Interface");?></td> + <td class="vtable"> + <select name="ifname" id="ifname" > +<?php + $iflist = get_configured_interface_with_descr(); + foreach ($iflist as $ifnam => $ifdescr) { + echo "<option value=\"{$ifnam}\""; + if ($ifnam == $pconfig['ifname']) { + echo " selected=\"selected\""; + } + echo ">{$ifdescr}</option>"; + } +?> + </select> + </td> + </tr> + <tr> + <td width="22%" valign="top" class="vncell"><?=gettext("Description");?></td> + <td width="78%" class="vtable"> + <input name="descr" type="text" class="formfld unknown" id="descr" size="40" value="<?=htmlspecialchars($pconfig['descr']);?>" /> + <br /> + <span class="vexpl"> + <?=gettext("You may enter a description here for your reference (not parsed).");?> + </span> + </td> + </tr> + <tr> + <td valign="top" class="vncellreq"><?=gettext("Type");?></td> + <td class="vtable"> + <select name="type" class="formselect" id="type" > + <option value="upstream" <?php if ($pconfig['type'] == "upstream") echo "selected=\"selected\""; ?>><?=gettext("Upstream Interface");?></option> + <option value="downstream" <?php if ($pconfig['type'] == "downstream") echo "selected=\"selected\""; ?>><?=gettext("Downstream Interface");?></option> + </select> + <br /> + <span class="vexpl"> + <?=gettext("The <b>upstream</b> network interface is the outgoing interface which is". + " responsible for communicating to available multicast data sources.". + " There can only be one upstream interface.");?> + </span> + <br /> + <span class="vexpl"> + <b><?=gettext("Downstream"); ?></b> + <?=gettext("network interfaces are the distribution interfaces to the". + " destination networks, where multicast clients can join groups and". + " receive multicast data. One or more downstream interfaces must be configured.");?> + </span> + </td> + </tr> + <tr> + <td valign="top" class="vncell"><?=gettext("Threshold");?></td> + <td class="vtable"> + <input name="threshold" class="formfld unknown" id="threshold" value="<?php echo htmlspecialchars($pconfig['threshold']);?>" /> + <br /> + <span class="vexpl"> + <?=gettext("Defines the TTL threshold for the network interface. Packets". + " with a lower TTL than the threshold value will be ignored. This". + " setting is optional, and by default the threshold is 1.");?> + </span> + </td> + </tr> + <tr> + <td width="22%" valign="top" class="vncellreq"><div id="addressnetworkport"><?=gettext("Network (s)");?></div></td> + <td width="78%" class="vtable"> + <table id="maintable"> + <tbody> + <tr> + <td><div id="onecolumn"><?=gettext("Network");?></div></td> + <td><div id="twocolumn"><?=gettext("CIDR");?></div></td> + </tr> + +<?php $counter = 0; $address = $pconfig['address']; if ($address <> "") { $item = explode(" ", $address); - foreach($item as $ww) { + foreach ($item as $ww) { $address = $item[$counter]; $address_subnet = ""; $item2 = explode("/", $address); - foreach($item2 as $current) { - if($item2[1] <> "") { + foreach ($item2 as $current) { + if ($item2[1] <> "") { $address = $item2[0]; $address_subnet = $item2[1]; } } $item4 = $item3[$counter]; $tracker = $counter; - ?> - <tr> - <td> - <input name="address<?php echo $tracker; ?>" type="text" class="formfld unknown" id="address<?php echo $tracker; ?>" size="30" value="<?=htmlspecialchars($address);?>" /> - </td> - <td> - <select name="address_subnet<?php echo $tracker; ?>" class="formselect" id="address_subnet<?php echo $tracker; ?>"> - <option></option> - <?php for ($i = 32; $i >= 1; $i--): ?> - <option value="<?=$i;?>" <?php if ($i == $address_subnet) echo "selected=\"selected\""; ?>><?=$i;?></option> - <?php endfor; ?> - </select> - </td> - <td> - <a onclick="removeRow(this); return false;" href="#"><img border="0" src="/themes/<?echo $g['theme'];?>/images/icons/icon_x.gif" alt="delete" /></a> - </td> - </tr> +?> + <tr> + <td> + <input name="address<?php echo $tracker; ?>" type="text" class="formfld unknown" id="address<?php echo $tracker; ?>" size="30" value="<?=htmlspecialchars($address);?>" /> + </td> + <td> + <select name="address_subnet<?php echo $tracker; ?>" class="formselect" id="address_subnet<?php echo $tracker; ?>"> + <option></option> + <?php for ($i = 32; $i >= 1; $i--): ?> + <option value="<?=$i;?>" <?php if ($i == $address_subnet) echo "selected=\"selected\""; ?>><?=$i;?></option> + <?php endfor; ?> + </select> + </td> + <td> + <a onclick="removeRow(this); return false;" href="#"><img border="0" src="/themes/<?echo $g['theme'];?>/images/icons/icon_x.gif" alt="delete" /></a> + </td> + </tr> <?php - $counter++; + $counter++; - } // end foreach + } // end foreach } // end if ?> - </tbody> - </table> - <a onclick="javascript:addRowTo('maintable'); return false;" href="#"> - <img border="0" src="/themes/<?= $g['theme']; ?>/images/icons/icon_plus.gif" alt="add" title="<?=gettext("add another entry");?>" /> - </a> - </td> - </tr> - <tr> - <td width="22%" valign="top"> </td> - <td width="78%"> - <input id="submit" name="submit" type="submit" class="formbtn" value="<?=gettext("Save");?>" /> - <a href="services_igmpproxy.php"><input id="cancelbutton" name="cancelbutton" type="button" class="formbtn" value="<?=gettext("Cancel");?>" /></a> - <?php if (isset($id) && $a_igmpproxy[$id]): ?> - <input name="id" type="hidden" value="<?=htmlspecialchars($id);?>" /> - <?php endif; ?> - </td> - </tr> -</table> + </tbody> + </table> + <a onclick="javascript:addRowTo('maintable'); return false;" href="#"> + <img border="0" src="/themes/<?= $g['theme']; ?>/images/icons/icon_plus.gif" alt="add" title="<?=gettext("add another entry");?>" /> + </a> + </td> + </tr> + <tr> + <td width="22%" valign="top"> </td> + <td width="78%"> + <input id="submit" name="submit" type="submit" class="formbtn" value="<?=gettext("Save");?>" /> + <a href="services_igmpproxy.php"><input id="cancelbutton" name="cancelbutton" type="button" class="formbtn" value="<?=gettext("Cancel");?>" /></a> + <?php if (isset($id) && $a_igmpproxy[$id]): ?> + <input name="id" type="hidden" value="<?=htmlspecialchars($id);?>" /> + <?php endif; ?> + </td> + </tr> + </table> </form> <script type="text/javascript"> |