summaryrefslogtreecommitdiffstats
path: root/src/usr/local/www/widgets
diff options
context:
space:
mode:
authorPhil Davis <phil.davis@inf.org>2015-12-17 12:23:24 +0545
committerPhil Davis <phil.davis@inf.org>2015-12-17 12:23:24 +0545
commitf402c3bcaffe67a9d73aa7cda741c2daca01685f (patch)
treef273729850a82ff9f4501d36b640e4a60e420b58 /src/usr/local/www/widgets
parent0cf1b4f4f29f2ad63691d598e83a9327ae9867ff (diff)
downloadpfsense-f402c3bcaffe67a9d73aa7cda741c2daca01685f.zip
pfsense-f402c3bcaffe67a9d73aa7cda741c2daca01685f.tar.gz
dyn_dns_status widget avoid var naming issue
This use of var $i was conflicting and interfering with a use of it in index.php Use a much more unique name for now to avoid future risk. Maybe a proper final solution for var naming across dashboard and widget is needed - but that is a design issue to be discussed.
Diffstat (limited to 'src/usr/local/www/widgets')
-rw-r--r--src/usr/local/www/widgets/widgets/dyn_dns_status.widget.php8
1 files changed, 4 insertions, 4 deletions
diff --git a/src/usr/local/www/widgets/widgets/dyn_dns_status.widget.php b/src/usr/local/www/widgets/widgets/dyn_dns_status.widget.php
index b450915..2eb1c84 100644
--- a/src/usr/local/www/widgets/widgets/dyn_dns_status.widget.php
+++ b/src/usr/local/www/widgets/widgets/dyn_dns_status.widget.php
@@ -105,8 +105,8 @@ if ($_REQUEST['getdyndnsstatus']) {
<td width="20%" class="listhdrr"><?=gettext("Hostname");?></td>
<td width="20%" class="listhdrr"><?=gettext("Cached IP");?></td>
</tr>
- <?php $i = 0; foreach ($a_dyndns as $dyndns): ?>
- <tr ondblclick="document.location='services_dyndns_edit.php?id=<?=$i;?>'">
+ <?php $dyndnsid = 0; foreach ($a_dyndns as $dyndns): ?>
+ <tr ondblclick="document.location='services_dyndns_edit.php?id=<?=$dyndnsid;?>'">
<td class="listlr">
<?php $iflist = get_configured_interface_with_descr();
foreach ($iflist as $if => $ifdesc) {
@@ -158,10 +158,10 @@ if ($_REQUEST['getdyndnsstatus']) {
?>
</td>
<td class="listr">
- <div id='dyndnsstatus<?= $i;?>'><?= gettext("Checking ...");?></div>
+ <div id='dyndnsstatus<?= $dyndnsid;?>'><?= gettext("Checking ...");?></div>
</td>
</tr>
- <?php $i++; endforeach;?>
+ <?php $dyndnsid++; endforeach;?>
</table>
</div>
<script type="text/javascript">
OpenPOWER on IntegriCloud