Skip to content

Commit

Permalink
Fix bug in status_mail_report_add_cmd.php
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
luckman212 committed Jun 28, 2015
1 parent efcb133 commit be8c333
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion config/mailreport/status_mail_report_add_cmd.php
Expand Up @@ -130,7 +130,7 @@
<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>
Expand Down

0 comments on commit be8c333

Please sign in to comment.