libs/cbi: change widgets using model.network to new api
authorJo-Philipp Wich <jow@openwrt.org>
Thu, 15 Oct 2009 16:37:29 +0000 (16:37 +0000)
committerJo-Philipp Wich <jow@openwrt.org>
Thu, 15 Oct 2009 16:37:29 +0000 (16:37 +0000)
libs/cbi/luasrc/view/cbi/firewall_zonelist.htm
libs/cbi/luasrc/view/cbi/network_ifacelist.htm
libs/cbi/luasrc/view/cbi/network_netlist.htm

index 26c46cb549565efd503d1db2f5773ebe8140205e..199d9ac9aeab8e5043bbd2ffe057770ebda9c3e3 100644 (file)
@@ -44,7 +44,7 @@ $Id$
                                                        empty = false
                         %>
                                <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 %> &nbsp;
+                               <% if iface:name() == self.iface then %><strong><%=iface:shortname()%></strong><% else %><%=iface:shortname()%><% end %> &nbsp;
                        <% end end end %>
                        <% if empty then %><em><%:a_s_ipt_zone_empty (no interfaces attached)%></em><% end %>
                </label>
index 00bb83a2e4443dcbd08c53cef186c9e9b14b8143..2f9821817051c3f4c7088ae1ee75cd8f7ccbdd63 100644 (file)
@@ -19,23 +19,33 @@ $Id$
 
        local iface
        local ifaces = net:get_interfaces()
-       local value = (self:cfgvalue(section) or self.default or "")
+       local value = (self:formvalue(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
+       if value and #value > 0 then
+               if type(value) == "table" then value = table.concat(value, " ") end
+               for value in value:gmatch("%S+") do
+                       checked[value] = true
+               end             
+       else
+               local n = self.network and net:get_network(self.network)
+               if n then
+                       local i
+                       for _, i in ipairs(n:get_interfaces()) do
+                               checked[i:name()] = true
+                       end
+               end
        end
 -%>
 
 <ul style="margin:0; list-style-type:none">
-       <% for _, iface in utl.spairs(ifaces, function(a,b) return (ifaces[a]:name() < ifaces[b]:name()) end) do
+       <% for _, iface in utl.spairs(ifaces, function(a,b) return (ifaces[a]:type() < ifaces[b]:type()) end) do
          if not self.nobridges or not iface:is_bridge() then %>
        <li>
                <input class="cbi-input-<%=self.widget or "radio"%>" onclick="cbi_d_update(this.id)" onchange="cbi_d_update(this.id)"<%=attr("type", self.widget or "radio") .. attr("id", cbid .. "." .. iface:name()) .. attr("name", cbid) .. attr("value", iface:name()) .. ifattr(checked[iface:name()], "checked", "checked")%> /> &nbsp;
                <label<%=attr("for", cbid .. "." .. iface:name())%>>
                        <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" />
-                       <%=iface:name()%><% local n = iface:get_network(); if n then %> (<%=n:name()%>)<% end %>
+                       <%=iface:get_i18n()%><% local n = iface:get_network(); if n then %> (<%=n:name()%>)<% end %>
                </label>
        </li>
        <% end end %>
index efff02432e1c8fc3bcc0af06afaaa6ab02384d01..df6e24d5a42b93117d8f497d1c3d5f48b4ca6419 100644 (file)
@@ -37,7 +37,7 @@ $Id$
                                        empty = false
                         %>
                                <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 %> &nbsp;
+                               <% if iface:name() == self.iface then %><strong><%=iface:shortname()%></strong><% else %><%=iface:shortname()%><% end %> &nbsp;
                        <% end end %>
                        <% if empty then %><em><%:a_s_ipt_zone_empty (no interfaces attached)%></em><% end %>
                </label>