summaryrefslogtreecommitdiffstats
path: root/src/usr/local/www/services_captiveportal_ip_edit.php
diff options
context:
space:
mode:
authorPhil Davis <phil.davis@inf.org>2016-06-27 14:30:14 +0930
committerChris Buechler <cmb@pfsense.org>2016-06-27 14:16:13 -0500
commitaceaf18c1ad93c808b77e0bbf324949d5d2eede0 (patch)
treecc34e086600b4a481586e35eb6142a1b46799679 /src/usr/local/www/services_captiveportal_ip_edit.php
parent9d265a8db874249ac16d4d02ce68463416f1aee2 (diff)
downloadpfsense-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/usr/local/www/services_captiveportal_ip_edit.php')
-rw-r--r--src/usr/local/www/services_captiveportal_ip_edit.php8
1 files changed, 4 insertions, 4 deletions
diff --git a/src/usr/local/www/services_captiveportal_ip_edit.php b/src/usr/local/www/services_captiveportal_ip_edit.php
index e32b82b..39cbe02 100644
--- a/src/usr/local/www/services_captiveportal_ip_edit.php
+++ b/src/usr/local/www/services_captiveportal_ip_edit.php
@@ -74,11 +74,11 @@ function allowedips_sort() {
usort($config['captiveportal'][$cpzone]['allowedip'], "allowedipscmp");
}
-require("guiconfig.inc");
-require("functions.inc");
+require_once("guiconfig.inc");
+require_once("functions.inc");
require_once("filter.inc");
-require("shaper.inc");
-require("captiveportal.inc");
+require_once("shaper.inc");
+require_once("captiveportal.inc");
$cpzone = $_GET['zone'];
if (isset($_POST['zone'])) {
OpenPOWER on IntegriCloud