Bug #1090
closed
clean up interfaces mess in setup wizard
Added by Chris Buechler almost 14 years ago.
Updated almost 14 years ago.
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>
- Status changed from New to Feedback
- % Done changed from 0 to 100
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.
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.
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.
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).
- Status changed from Feedback to Resolved
Also available in: Atom
PDF