diff options
author | NOYB <Al_Stu@Frontier.com> | 2016-02-15 00:42:35 -0800 |
---|---|---|
committer | NOYB <Al_Stu@Frontier.com> | 2016-02-15 00:42:35 -0800 |
commit | f2066870061a3b0875fbf181090ca2517f69ea35 (patch) | |
tree | ac0d0dc10f2847e9f6bd90c663bd0e36a3948e3e /src/usr/local/www/firewall_rules.php | |
parent | f96d306e122951011352cf0cf13c0f099e5278e2 (diff) | |
download | pfsense-f2066870061a3b0875fbf181090ca2517f69ea35.zip pfsense-f2066870061a3b0875fbf181090ca2517f69ea35.tar.gz |
Firewall / Rules - for/foreach Vars
Replace for $i var with foreach $filteri var.
Diffstat (limited to 'src/usr/local/www/firewall_rules.php')
-rw-r--r-- | src/usr/local/www/firewall_rules.php | 18 |
1 files changed, 8 insertions, 10 deletions
diff --git a/src/usr/local/www/firewall_rules.php b/src/usr/local/www/firewall_rules.php index c6c9206..0891dcf 100644 --- a/src/usr/local/www/firewall_rules.php +++ b/src/usr/local/www/firewall_rules.php @@ -486,8 +486,6 @@ $seprows = separator_rows($separators); foreach ($a_filter as $filteri => $filterent): - $i = $filteri; - if (($filterent['interface'] == $if && !isset($filterent['floating'])) || (isset($filterent['floating']) && "FloatingRules" == $if)) { // Display separator(s) for section beginning at rule n @@ -495,9 +493,9 @@ foreach ($a_filter as $filteri => $filterent): display_separator($separators, $nrules, $columns_in_table); } ?> - <tr id="fr<?=$nrules;?>" onClick="fr_toggle(<?=$nrules;?>)" ondblclick="document.location='firewall_rules_edit.php?id=<?=$i;?>';" <?=(isset($filterent['disabled']) ? ' class="disabled"' : '')?>> + <tr id="fr<?=$nrules;?>" onClick="fr_toggle(<?=$nrules;?>)" ondblclick="document.location='firewall_rules_edit.php?id=<?=$filteri;?>';" <?=(isset($filterent['disabled']) ? ' class="disabled"' : '')?>> <td> - <input type="checkbox" id="frc<?=$nrules;?>" onClick="fr_toggle(<?=$nrules;?>)" name="rule[]" value="<?=$i;?>"/> + <input type="checkbox" id="frc<?=$nrules;?>" onClick="fr_toggle(<?=$nrules;?>)" name="rule[]" value="<?=$filteri;?>"/> </td> <?php @@ -516,7 +514,7 @@ foreach ($a_filter as $filteri => $filterent): } ?> <td title="<?=$title_text?>"> - <a href="?if=<?=htmlspecialchars($if);?>&act=toggle&id=<?=$i;?>"> + <a href="?if=<?=htmlspecialchars($if);?>&act=toggle&id=<?=$filteri;?>"> <i class="fa fa-<?=$iconfn?>" title="<?=gettext("click to toggle enabled/disabled status");?>"></i> </a> <?php @@ -784,17 +782,17 @@ foreach ($a_filter as $filteri => $filterent): </td> <td class="action-icons"> <!-- <?=(isset($filterent['disabled']) ? 'enable' : 'disable')?> --> - <a href="firewall_rules_edit.php?id=<?=$i;?>" class="fa fa-pencil" title="<?=gettext('Edit')?>"></a> - <a href="firewall_rules_edit.php?dup=<?=$i;?>" class="fa fa-clone" title="<?=gettext('Copy')?>"></a> + <a href="firewall_rules_edit.php?id=<?=$filteri;?>" class="fa fa-pencil" title="<?=gettext('Edit')?>"></a> + <a href="firewall_rules_edit.php?dup=<?=$filteri;?>" class="fa fa-clone" title="<?=gettext('Copy')?>"></a> <?php if (isset($filterent['disabled'])) { ?> - <a href="?act=toggle&if=<?=htmlspecialchars($if);?>&id=<?=$i;?>" class="fa fa-check-square-o" title="<?=gettext('Enable')?>"></a> + <a href="?act=toggle&if=<?=htmlspecialchars($if);?>&id=<?=$filteri;?>" class="fa fa-check-square-o" title="<?=gettext('Enable')?>"></a> <?php } else { ?> - <a href="?act=toggle&if=<?=htmlspecialchars($if);?>&id=<?=$i;?>" class="fa fa-ban" title="<?=gettext('Disable')?>"></a> + <a href="?act=toggle&if=<?=htmlspecialchars($if);?>&id=<?=$filteri;?>" class="fa fa-ban" title="<?=gettext('Disable')?>"></a> <?php } ?> - <a href="?act=del&if=<?=htmlspecialchars($if);?>&id=<?=$i;?>" class="fa fa-trash" title="<?=gettext('Delete this rule')?>"></a> + <a href="?act=del&if=<?=htmlspecialchars($if);?>&id=<?=$filteri;?>" class="fa fa-trash" title="<?=gettext('Delete this rule')?>"></a> </td> </tr> <?php |