diff options
| author | Phil Davis <phil.davis@inf.org> | 2016-06-27 14:30:14 +0930 |
|---|---|---|
| committer | Chris Buechler <cmb@pfsense.org> | 2016-06-27 14:16:13 -0500 |
| commit | aceaf18c1ad93c808b77e0bbf324949d5d2eede0 (patch) | |
| tree | cc34e086600b4a481586e35eb6142a1b46799679 /src/etc/inc/upgrade_config.inc | |
| parent | 9d265a8db874249ac16d4d02ce68463416f1aee2 (diff) | |
| download | pfsense-aceaf18c1ad93c808b77e0bbf324949d5d2eede0.zip pfsense-aceaf18c1ad93c808b77e0bbf324949d5d2eede0.tar.gz | |
Always use require_once
The usage of require() and require_once() throughout the system is
inconsistent, and "bugs" come up now and then when the order of
"requires" is a bit different and some require() happens after the
include file is already included/required.
It seems to me that there is no harm at all in always using
require_once().
Diffstat (limited to 'src/etc/inc/upgrade_config.inc')
| -rw-r--r-- | src/etc/inc/upgrade_config.inc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/etc/inc/upgrade_config.inc b/src/etc/inc/upgrade_config.inc index 1c2b818..c126e3e 100644 --- a/src/etc/inc/upgrade_config.inc +++ b/src/etc/inc/upgrade_config.inc @@ -54,10 +54,10 @@ */ if (!function_exists("dump_rrd_to_xml")) { - require("rrd.inc"); + require_once("rrd.inc"); } if (!function_exists("read_altq_config")) { - require("shaper.inc"); + require_once("shaper.inc"); } /* Upgrade functions must be named: |
