From 536430d0eeecf5c75651ff625b97c9fa4fc646b4 Mon Sep 17 00:00:00 2001
From: Jo-Philipp Wich <jow@openwrt.org>
Date: Thu, 15 Oct 2009 16:37:29 +0000
Subject: [PATCH] libs/cbi: change widgets using model.network to new api

---
 .../cbi/luasrc/view/cbi/firewall_zonelist.htm |  2 +-
 .../cbi/luasrc/view/cbi/network_ifacelist.htm | 22 ++++++++++++++-----
 libs/cbi/luasrc/view/cbi/network_netlist.htm  |  2 +-
 3 files changed, 18 insertions(+), 8 deletions(-)

diff --git a/libs/cbi/luasrc/view/cbi/firewall_zonelist.htm b/libs/cbi/luasrc/view/cbi/firewall_zonelist.htm
index 26c46cb54..199d9ac9a 100644
--- a/libs/cbi/luasrc/view/cbi/firewall_zonelist.htm
+++ b/libs/cbi/luasrc/view/cbi/firewall_zonelist.htm
@@ -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>
diff --git a/libs/cbi/luasrc/view/cbi/network_ifacelist.htm b/libs/cbi/luasrc/view/cbi/network_ifacelist.htm
index 00bb83a2e..2f9821817 100644
--- a/libs/cbi/luasrc/view/cbi/network_ifacelist.htm
+++ b/libs/cbi/luasrc/view/cbi/network_ifacelist.htm
@@ -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 %>
diff --git a/libs/cbi/luasrc/view/cbi/network_netlist.htm b/libs/cbi/luasrc/view/cbi/network_netlist.htm
index efff02432..df6e24d5a 100644
--- a/libs/cbi/luasrc/view/cbi/network_netlist.htm
+++ b/libs/cbi/luasrc/view/cbi/network_netlist.htm
@@ -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>
-- 
2.25.1