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

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;