diff options
author | Ermal LUÇI <eri@pfsense.org> | 2015-01-12 09:27:09 +0100 |
---|---|---|
committer | Ermal LUÇI <eri@pfsense.org> | 2015-01-12 09:27:17 +0100 |
commit | 384deecb5cfec1302dd99f3b81ca8c4f184cd93e (patch) | |
tree | 2993f2e02aa9ecc76120b5824e292476d12fe298 /usr/local/www | |
parent | 1aecc4daf86e642aff23a83ee128a400fa5598b5 (diff) | |
download | pfsense-384deecb5cfec1302dd99f3b81ca8c4f184cd93e.zip pfsense-384deecb5cfec1302dd99f3b81ca8c4f184cd93e.tar.gz |
Fix inherent issues with isset and empty values set as true by our parser. This made the piep configuration to be wrong at least for passthrough entries. Ticket #3932
Diffstat (limited to 'usr/local/www')
-rw-r--r-- | usr/local/www/services_captiveportal.php | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/usr/local/www/services_captiveportal.php b/usr/local/www/services_captiveportal.php index c88f6af..8c1487b 100644 --- a/usr/local/www/services_captiveportal.php +++ b/usr/local/www/services_captiveportal.php @@ -339,8 +339,14 @@ if ($_POST) { $newcp['preauthurl'] = $_POST['preauthurl']; $newcp['blockedmacsurl'] = $_POST['blockedmacsurl']; $newcp['peruserbw'] = $_POST['peruserbw'] ? true : false; - $newcp['bwdefaultdn'] = $_POST['bwdefaultdn']; - $newcp['bwdefaultup'] = $_POST['bwdefaultup']; + if (isset($_POST['bwdefaultdn'])) + $newcp['bwdefaultdn'] = $_POST['bwdefaultdn']; + else + unset($newcp['bwdefaultdn']); + if (isset($_POST['bwdefaultup'])) + $newcp['bwdefaultup'] = $_POST['bwdefaultup']; + else + unset($newcp['bwdefaultup']); $newcp['certref'] = $_POST['certref']; $newcp['nohttpsforwards'] = $_POST['nohttpsforwards'] ? true : false; $newcp['logoutwin_enable'] = $_POST['logoutwin_enable'] ? true : false; |