Bug #1090
closedclean up interfaces mess in setup wizard
100%
Description
Revision 389c778ec29d929a17843139f8cddf337b8fa9ad by gnhb added an interface drop down to the WAN configuration page in the setup wizard, which breaks the WAN assignment when it's anything but a physical interface (example: VLAN) when the wizard is run or re-run. That box just needs to go away, there is no need to change the interface assignment at that point.
Jim also noted bad usage of a temp variable under $config in the <stepbeforeformdisplay>
Updated by Marcus Brown almost 14 years ago
- Status changed from New to Feedback
- % Done changed from 0 to 100
Applied in changeset 841c4125a6b4488c16a0c69e7642779f294fa449.
Updated by Marcus Brown almost 14 years ago
Committed the removal of the dropdown interfaces select box.
About the temp variable, I couldn't really figure out how to store temporary data in the wizard code, so I stored it under $config because that's the only place I could tell that the wizard is designed to put data that is entered on the php form, and I didn't feel qualified or authorized to re-write the structure of the wizard. If anyone knows a better way, please fix it.
Updated by Erik Fonnesbeck almost 14 years ago
As far as I could tell, it doesn't end up writing that to the config between those steps, but it would probably be better to at least put that temporary item in a different section just in case, because nothing other than wan, lan, and opt# sections are supposed to be in interfaces.
Updated by Marcus Brown almost 14 years ago
This is probably obvious, but the Setup Wizard will silently fail to create a valid PPPoE or PPTP link if the WAN is not already assigned to the correct parent interface for the PPPoE/PPTP link. That's why I added the interface select list in the first place.
Updated by Erik Fonnesbeck almost 14 years ago
It is assumed that the port for your WAN connection is already assigned to WAN before the setup wizard is even started, either by having plugged the cable into the correct port or by assigning the correct port during interface assignment on the console (or interface assignment in the web gui if running the setup wizard again later).
Updated by Ermal Luçi almost 14 years ago
- Status changed from Feedback to Resolved