Remove the parent interface reconfiguration functions they are only bogusly/erroneously called here:

* On bootup without this change the parent interface will be configured twice.
* Even when the system is up a vlan does not have to mock up with the parent apart guaranteeing that it is up.

The only thing that does not work is makeing a VLAN on dynamic interfaces like pppoe/pptp.
Afaik it has never worked and i do not think that many people would use vlans over pppoe,
if a feature request for such things comes up it will be on my TODO around 2.x of pfSense.

1 files changed, 0 insertions, 7 deletions

diff --git a/usr/local/www/interfaces_vlan_edit.php b/usr/local/www/interfaces_vlan_edit.php
index 82318a2..c7e7de6 100755
--- a/usr/local/www/interfaces_vlan_edit.php
+++ b/usr/local/www/interfaces_vlan_edit.php
@@ -92,13 +92,6 @@ if ($_POST) {
 
                         write_config();
 
-                        if ($vlan['if'] == "lan")
-                                interfaces_lan_configure();
-                        else if ($vlan['if'] == "wan")
-                                interfaces_wan_configure();
-                        else
-                                interfaces_optional_configure_if($vlan['if']);
-
 			header("Location: interfaces_vlan.php");
 			exit;
 		}