Commit 64d21efd authored by Ad Schellevis's avatar Ad Schellevis

fix for https://github.com/opnsense/core/issues/115 , the process flow remains...

fix for https://github.com/opnsense/core/issues/115 , the process flow remains messy, but shouldn't flush the config data anymore
parent a98e94b3
...@@ -40,16 +40,6 @@ $if = $_GET['if']; ...@@ -40,16 +40,6 @@ $if = $_GET['if'];
if ($_POST['if']) if ($_POST['if'])
$if = $_POST['if']; $if = $_POST['if'];
/* if OLSRD is enabled, allow WAN to house DHCP. */
if($config['installedpackages']['olsrd']) {
foreach($config['installedpackages']['olsrd']['config'] as $olsrd) {
if($olsrd['enable']) {
$is_olsr_enabled = true;
break;
}
}
}
if (!$_GET['if']) if (!$_GET['if'])
$savemsg = gettext("The DHCPv6 Server can only be enabled on interfaces configured with static IP addresses") . $savemsg = gettext("The DHCPv6 Server can only be enabled on interfaces configured with static IP addresses") .
gettext("Only interfaces configured with a static IP will be shown") . "."; gettext("Only interfaces configured with a static IP will be shown") . ".";
...@@ -128,11 +118,15 @@ if ($_POST) { ...@@ -128,11 +118,15 @@ if ($_POST) {
unset($input_errors); unset($input_errors);
$old_dhcpdv6_enable = ($pconfig['enable'] == true); if ($_POST['apply'] != 'Apply changes') {
$new_dhcpdv6_enable = ($_POST['enable'] ? true : false); $old_dhcpdv6_enable = ($pconfig['enable'] == true);
$dhcpdv6_enable_changed = ($old_dhcpdv6_enable != $new_dhcpdv6_enable); $new_dhcpdv6_enable = ($_POST['enable'] ? true : false);
$dhcpdv6_enable_changed = ($old_dhcpdv6_enable != $new_dhcpdv6_enable);
$pconfig = $_POST;
} else {
$dhcpdv6_enable_changed = false;
}
$pconfig = $_POST;
$numberoptions = array(); $numberoptions = array();
for($x=0; $x<99; $x++) { for($x=0; $x<99; $x++) {
...@@ -250,7 +244,7 @@ if ($_POST) { ...@@ -250,7 +244,7 @@ if ($_POST) {
} }
} }
if (!$input_errors) { if (!$input_errors && $_POST['apply'] != 'Apply changes') {
if (!is_array($config['dhcpdv6'][$if])) if (!is_array($config['dhcpdv6'][$if]))
$config['dhcpdv6'][$if] = array(); $config['dhcpdv6'][$if] = array();
if (!is_array($config['dhcpdv6'][$if]['range'])) if (!is_array($config['dhcpdv6'][$if]['range']))
...@@ -303,10 +297,9 @@ if ($_POST) { ...@@ -303,10 +297,9 @@ if ($_POST) {
$config['dhcpdv6'][$if]['numberoptions'] = $numberoptions; $config['dhcpdv6'][$if]['numberoptions'] = $numberoptions;
write_config(); write_config();
}
$retval = 0;
$retvaldhcp = 0; if (!$input_errors || $_POST['apply'] == 'Apply changes') {
$retvaldns = 0;
/* Stop DHCPv6 so we can cleanup leases */ /* Stop DHCPv6 so we can cleanup leases */
killbypid("{$g['dhcpd_chroot_path']}{$g['varrun_path']}/dhcpdv6.pid"); killbypid("{$g['dhcpd_chroot_path']}{$g['varrun_path']}/dhcpdv6.pid");
// dhcp_clean_leases(); // dhcp_clean_leases();
...@@ -332,7 +325,7 @@ if ($_POST) { ...@@ -332,7 +325,7 @@ if ($_POST) {
if($retvaldhcp == 1 || $retvaldns == 1 || $retvalfc == 1) if($retvaldhcp == 1 || $retvaldns == 1 || $retvalfc == 1)
$retval = 1; $retval = 1;
$savemsg = get_std_save_message($retval); $savemsg = get_std_save_message($retval);
} }
} }
if ($_GET['act'] == "del") { if ($_GET['act'] == "del") {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment