aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorlukehamburg <luke.hamburg@gmail.com>2015-06-28 18:27:01 -0400
committerlukehamburg <luke.hamburg@gmail.com>2015-06-28 18:27:01 -0400
commitbe8c33396a4cf7fe504ddf2802724cc8d7df1ed0 (patch)
tree8fbb91191e92f2c51be17a60b7ac123b20c65468
parentefcb133964c38a8e3db7e93a90205b75948c5e05 (diff)
downloadpfsense-packages-be8c33396a4cf7fe504ddf2802724cc8d7df1ed0.tar.gz
pfsense-packages-be8c33396a4cf7fe504ddf2802724cc8d7df1ed0.tar.bz2
pfsense-packages-be8c33396a4cf7fe504ddf2802724cc8d7df1ed0.zip
Fix bug in status_mail_report_add_cmd.php
Existing code was referencing the wrong variable ($a_graphs instead of $a_cmds), which caused the IF test to fail, which then caused duplicate entries to be created each time the "edit command" form was submitted. This edit fixes the issue, tested on pfSense 2.1.5 nanobsd.
-rw-r--r--config/mailreport/status_mail_report_add_cmd.php2
1 files changed, 1 insertions, 1 deletions
diff --git a/config/mailreport/status_mail_report_add_cmd.php b/config/mailreport/status_mail_report_add_cmd.php
index 6a924142..b60f9a80 100644
--- a/config/mailreport/status_mail_report_add_cmd.php
+++ b/config/mailreport/status_mail_report_add_cmd.php
@@ -130,7 +130,7 @@ include("head.inc");
<input name="Submit" type="submit" class="formbtn" value="<?=gettext("Save");?>">
<a href="status_mail_report_edit.php?id=<?php echo $reportid;?>"><input name="cancel" type="button" class="formbtn" value="<?=gettext("Cancel");?>"></a>
<input name="reportid" type="hidden" value="<?=htmlspecialchars($reportid);?>">
- <?php if (isset($id) && $a_graphs[$id]): ?>
+ <?php if (isset($id) && $a_cmds[$id]): ?>
<input name="id" type="hidden" value="<?=htmlspecialchars($id);?>">
<?php endif; ?>
</td>