diff options
author | Jo-Philipp Wich <jow@openwrt.org> | 2009-10-10 14:47:37 +0000 |
---|---|---|
committer | Jo-Philipp Wich <jow@openwrt.org> | 2009-10-10 14:47:37 +0000 |
commit | 988de587ee5e159f4991f8ab855c75b9cfcee847 (patch) | |
tree | 8f6b54bc3c2723d04ca5fef522789d9b8f99fcc5 | |
parent | 0533309ef29e3ff3239d3fa2a8c561178137856e (diff) |
libs/cbi: various widget fixes
-rw-r--r-- | libs/cbi/luasrc/view/cbi/firewall_zonelist.htm | 35 | ||||
-rw-r--r-- | libs/cbi/luasrc/view/cbi/fvalue.htm | 2 | ||||
-rw-r--r-- | libs/cbi/luasrc/view/cbi/network_ifacelist.htm | 1 | ||||
-rw-r--r-- | libs/cbi/luasrc/view/cbi/value.htm | 2 |
4 files changed, 22 insertions, 18 deletions
diff --git a/libs/cbi/luasrc/view/cbi/firewall_zonelist.htm b/libs/cbi/luasrc/view/cbi/firewall_zonelist.htm index 847010b7cb..26c46cb549 100644 --- a/libs/cbi/luasrc/view/cbi/firewall_zonelist.htm +++ b/libs/cbi/luasrc/view/cbi/firewall_zonelist.htm @@ -15,48 +15,51 @@ $Id$ <%- local utl = require "luci.util" - local wba = require "luci.tools.webadmin" - local wbi = require "luci.tools.webadmin_interfaces" local fwm = require "luci.model.firewall" + local nwm = require "luci.model.network" - wbi.init(self.map.uci) - - local zone, iface, ifname + local zone, net, iface local zones = fwm:get_zones() local value = self:cfgvalue(section) or self.default + local selected = false if value and #value == 0 then value = nil end -%> <ul style="margin:0; list-style-type:none"> - <% for _, zone in utl.spairs(zones, function(a,b) return (zones[a]:name() < zones[b]:name()) end) do %> + <% + for _, zone in utl.spairs(zones, function(a,b) return (zones[a]:name() < zones[b]:name()) end) do + selected = selected or (value == zone:name()) + %> <li style="padding:0.5em"> <input class="cbi-input-radio" onclick="cbi_d_update(this.id)" onchange="cbi_d_update(this.id)" type="radio"<%=attr("id", cbid .. "." .. zone:name()) .. attr("name", cbid) .. attr("value", zone:name()) .. ifattr(value == zone:name(), "checked", "checked")%> /> <label<%=attr("for", cbid .. "." .. zone:name())%> style="background-color:<%=zone:get_color()%>; padding:0.5em"> <strong><%=zone:name()%>:</strong> <% local empty = true - for _, iface in ipairs(zone:get_networks()) do - local ifname = wbi.network_get_ifname(iface) - local dev = wbi.iface_get_list()[ifname] + for _, net in ipairs(zone:get_networks()) do + net = nwm:get_network(net) + for _, iface in ipairs(net and net:get_interfaces() or {}) do + if not iface:is_bridgeport() then + empty = false %> - <% if ifname and dev then empty = false %> - <img title="<%=wbi.iface_format_type(dev.type)%>" style="width:16px; height:16px; vertical-align:middle" src="<%=resource%>/icons/<%=dev.type or "ethernet"%><%=dev.flags and dev.flags.up and "" or "_disabled"%>.png" /> - <% if iface == self.iface then %><strong><%=ifname%></strong><% else %><%=ifname%><% end %> - <% end %> - <% end %> + <img title="<%=iface:get_type_i18n()%>" style="width:16px; height:16px; vertical-align:middle" src="<%=resource%>/icons/<%=iface:type()%><%=iface:is_up() and "" or "_disabled"%>.png" /> + <% if iface:name() == self.iface then %><strong><%=iface:name()%></strong><% else %><%=iface:name()%><% end %> + <% end end end %> <% if empty then %><em><%:a_s_ipt_zone_empty (no interfaces attached)%></em><% end %> </label> </li> <% end %> + <% if not self.nocreate then %> <li style="padding:0.5em"> - <input class="cbi-input-radio" onclick="cbi_d_update(this.id)" onchange="cbi_d_update(this.id)" type="radio"<%=attr("id", cbid .. "_new") .. attr("name", cbid) .. attr("value", "-") .. ifattr(not value, "checked", "checked")%> /> + <input class="cbi-input-radio" onclick="cbi_d_update(this.id)" onchange="cbi_d_update(this.id)" type="radio"<%=attr("id", cbid .. "_new") .. attr("name", cbid) .. attr("value", "-") .. ifattr(not selected, "checked", "checked")%> /> <div style="background-color:<%=fwm.zone.get_color()%>; padding:0.5em; display:inline"> <label<%=attr("for", cbid .. "_new")%>><em><%:a_s_ipt_zone_unspec_create unspecified -or- create:%> </em></label> - <input style="width:6em" type="text"<%=attr("name", cbid .. ".newzone")%> onfocus="document.getElementById('<%=cbid%>_new').checked=true" /> + <input style="width:6em" type="text"<%=attr("name", cbid .. ".newzone") .. ifattr(not selected, "value", self.default)%> onfocus="document.getElementById('<%=cbid%>_new').checked=true" /> </div> </li> + <% end %> </ul> <%+cbi/valuefooter%> diff --git a/libs/cbi/luasrc/view/cbi/fvalue.htm b/libs/cbi/luasrc/view/cbi/fvalue.htm index b46817c6f7..35ebac6e1c 100644 --- a/libs/cbi/luasrc/view/cbi/fvalue.htm +++ b/libs/cbi/luasrc/view/cbi/fvalue.htm @@ -13,5 +13,5 @@ $Id$ -%> <%+cbi/valueheader%> - <input class="cbi-input-checkbox" onclick="cbi_d_update(this.id)" onchange="cbi_d_update(this.id)" type="checkbox"<%= attr("id", cbid) .. attr("name", cbid) .. ifattr(self:cfgvalue(section) == self.enabled, "checked", "checked") %> value="1" /> + <input class="cbi-input-checkbox" onclick="cbi_d_update(this.id)" onchange="cbi_d_update(this.id)" type="checkbox"<%= attr("id", cbid) .. attr("name", cbid) .. ifattr((self:cfgvalue(section) or self.default) == self.enabled, "checked", "checked") %> value="1" /> <%+cbi/valuefooter%> diff --git a/libs/cbi/luasrc/view/cbi/network_ifacelist.htm b/libs/cbi/luasrc/view/cbi/network_ifacelist.htm index b3a61fa6ad..00bb83a2e4 100644 --- a/libs/cbi/luasrc/view/cbi/network_ifacelist.htm +++ b/libs/cbi/luasrc/view/cbi/network_ifacelist.htm @@ -22,6 +22,7 @@ $Id$ local value = (self:cfgvalue(section) or self.default or "") local checked = { } + if type(value) == "table" then value = table.concat(value, " ") end for value in value:gmatch("%S+") do checked[value] = true end diff --git a/libs/cbi/luasrc/view/cbi/value.htm b/libs/cbi/luasrc/view/cbi/value.htm index e6a7ab6d9f..de083e62cf 100644 --- a/libs/cbi/luasrc/view/cbi/value.htm +++ b/libs/cbi/luasrc/view/cbi/value.htm @@ -13,7 +13,7 @@ $Id$ -%> <%+cbi/valueheader%> - <input type="<%=self.password and 'password" class="cbi-input-password' or 'text" class="cbi-input-text' %>" onchange="cbi_d_update(this.id)"<%= attr("name", cbid) .. attr("id", cbid) .. attr("value", self:cfgvalue(section)) .. ifattr(self.size, "size")%> /> + <input type="<%=self.password and 'password" class="cbi-input-password' or 'text" class="cbi-input-text' %>" onchange="cbi_d_update(this.id)"<%= attr("name", cbid) .. attr("id", cbid) .. attr("value", self:cfgvalue(section) or self.default) .. ifattr(self.size, "size")%> /> <% if #self.keylist > 0 then -%> <script type="text/javascript"> cbi_combobox_init('<%=cbid%>', { |