From 2b25a8b5ad97eccb5b2b5bdd46d6aa8462b025bb Mon Sep 17 00:00:00 2001 From: Jo-Philipp Wich Date: Fri, 20 May 2011 11:49:09 +0000 Subject: [PATCH] modules/admin-full: throw more work-arounds, exceptions, hacks and detections at switch config page --- .../luasrc/model/cbi/admin_network/vlan.lua | 55 +++++++++++++------ 1 file changed, 38 insertions(+), 17 deletions(-) diff --git a/modules/admin-full/luasrc/model/cbi/admin_network/vlan.lua b/modules/admin-full/luasrc/model/cbi/admin_network/vlan.lua index 911245740..b4ad864c6 100644 --- a/modules/admin-full/luasrc/model/cbi/admin_network/vlan.lua +++ b/modules/admin-full/luasrc/model/cbi/admin_network/vlan.lua @@ -2,7 +2,7 @@ LuCI - Lua Configuration Interface Copyright 2008 Steven Barth -Copyright 2010 Jo-Philipp Wich +Copyright 2010-2011 Jo-Philipp Wich Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with the License. @@ -18,13 +18,16 @@ m = Map("network", translate("Switch"), translate("The network ports on your rou m.uci:foreach("network", "switch", function(x) local switch_name = x.name or x['.name'] + local has_enable = "enable" + local has_reset = nil + local has_vlan = nil + local has_learn = nil local has_vlan4k = nil local has_ptpvid = nil - local has_jumbo3 = nil local min_vid = 0 local max_vid = 16 local num_vlans = 16 - local num_ports = 5 + local num_ports = 6 local cpu_port = 5 local enable_vlan4k = false @@ -51,20 +54,28 @@ m.uci:foreach("network", "switch", num_ports, cpu_port, num_vlans = line:match("ports: (%d+) %(cpu @ (%d+)%), vlans: (%d+)") - num_ports = tonumber(num_ports) or 5 - num_vlans = tonumber(num_vlans) or 16 - cpu_port = tonumber(cpu_port) or 5 - min_vid = 1 + num_ports = tonumber(num_ports or 6) + num_vlans = tonumber(num_vlans or 16) + cpu_port = tonumber(cpu_port or 5) + min_vid = 1 + has_enable = nil elseif line:match(": pvid") or line:match(": tag") or line:match(": vid") then if is_vlan_attr then has_vlan4k = line:match(": (%w+)") end if is_port_attr then has_ptpvid = line:match(": (%w+)") end + elseif line:match(": enable_vlan") then + has_vlan = "enable_vlan" + + elseif line:match(": enable_learning") then + has_learn = "enable_learning" + + elseif line:match(": reset") then + has_reset = "reset" + elseif line:match(": enable_vlan4k") then enable_vlan4k = true - elseif line:match(": max_length") then - has_jumbo3 = true end end @@ -103,20 +114,30 @@ m.uci:foreach("network", "switch", s = m:section(NamedSection, x['.name'], "switch", translatef("Switch %q", switch_name)) s.addremove = false - s:option(Flag, "enable", translate("Enable this switch")).default = "1" - s:option(Flag, "enable_vlan", translate("Enable VLAN functionality")).default = "1" + if has_enable then + x = s:option(Flag, has_enable, translate("Enable this switch")) + x.default = x.enabled + end + + if has_vlan then + x = s:option(Flag, has_vlan, translate("Enable VLAN functionality")) + x.default = x.enabled + end if enable_vlan4k then s:option(Flag, "enable_vlan4k", translate("Enable 4K VLANs")) end - if has_jumbo3 then - j = s:option(Flag, "max_length", translate("Enable Jumbo Frame passthrough")) - j.enabled = "3" - j.rmempty = true + if has_learn then + x = s:option(Flag, has_learn, translate("Enable learning and aging")) + x.default = x.enabled + end + + if has_reset then + x = s:option(Flag, has_reset, translate("Reset switch during setup")) + x.default = x.enabled end - s:option(Flag, "reset", translate("Reset switch during setup")).default = "1" -- VLAN table s = m:section(TypedSection, "switch_vlan", translatef("VLANs on %q", switch_name)) @@ -202,7 +223,7 @@ m.uci:foreach("network", "switch", if value == "u" then if not untagged[self.option] then untagged[self.option] = true - else + elseif min_vid > 0 or tonumber(self.option) ~= cpu_port then -- enable multiple untagged cpu ports due to weird broadcom default setup return nil, translatef("Port %d is untagged in multiple VLANs!", tonumber(self.option) + 1) end -- 2.25.1