diff options
| author | lukehamburg <luke.hamburg@gmail.com> | 2015-06-28 18:27:01 -0400 |
|---|---|---|
| committer | lukehamburg <luke.hamburg@gmail.com> | 2015-06-28 18:27:01 -0400 |
| commit | be8c33396a4cf7fe504ddf2802724cc8d7df1ed0 (patch) | |
| tree | 8fbb91191e92f2c51be17a60b7ac123b20c65468 | |
| parent | efcb133964c38a8e3db7e93a90205b75948c5e05 (diff) | |
| download | pfsense-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.php | 2 |
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> |