modules/admin-full: remove redundant code
authorJo-Philipp Wich <jow@openwrt.org>
Sat, 24 Sep 2011 03:14:48 +0000 (03:14 +0000)
committerJo-Philipp Wich <jow@openwrt.org>
Sat, 24 Sep 2011 03:14:48 +0000 (03:14 +0000)
modules/admin-full/luasrc/view/admin_network/iface_overview.htm
modules/admin-full/luasrc/view/admin_network/iface_status.htm

index 67d5a377f138f8c5f384ecdae342f4716e6bbe12..edfed7d7d2384a530254166fc7fbd00a37d59856 100644 (file)
@@ -77,28 +77,8 @@ $Id$
                                        for (var i = 0; i < ifcs.length; i++)
                                        {
                                                var ifc = ifcs[i];
-                                               var rxb = ifc.rx_bytes || 0;
-                                               var txb = ifc.tx_bytes || 0;
-                                               var rxp = ifc.rx_packets || 0;
-                                               var txp = ifc.tx_packets || 0;
-                                               var mac = ifc.macaddr;
-                                               var upt = null;
-
                                                var html = '';
 
-                                               var icon;
-                                               if (ifc.is_up)
-                                               {
-                                                       if (ifc.uptime)
-                                                               upt = String.format('%t', ifc.uptime);
-
-                                                       icon = "<%=resource%>/icons/%s.png";
-                                               }
-                                               else
-                                               {
-                                                       icon = "<%=resource%>/icons/%s_disabled.png";
-                                               }
-
                                                var s = document.getElementById(ifc.id + '-ifc-devices');
                                                if (s)
                                                {
@@ -138,20 +118,21 @@ $Id$
                                                var d = document.getElementById(ifc.id + '-ifc-description');
                                                if (d && ifc.ifname)
                                                {
-                                                       if (upt)
+                                                       if (ifc.is_up)
                                                        {
-                                                               html += String.format('<strong><%:Uptime%>:</strong> %s<br />', upt);
+                                                               html += String.format('<strong><%:Uptime%>:</strong> %t<br />', ifc.uptime);
                                                        }
 
                                                        if (ifc.type != 'tunnel')
                                                        {
-                                                               html += String.format('<strong><%:MAC Address%>:</strong> %s<br />', mac);
+                                                               html += String.format('<strong><%:MAC Address%>:</strong> %s<br />', ifc.macaddr);
                                                        }
 
                                                        html += String.format(
                                                                '<strong><%:RX%></strong>: %.2mB (%d <%:Pkts.%>)<br />' +
                                                                '<strong><%:TX%></strong>: %.2mB (%d <%:Pkts.%>)<br />',
-                                                                       rxb, rxp, txb, txp
+                                                                       ifc.rx_bytes, ifc.rx_packets,
+                                                                       ifc.tx_bytes, ifc.tx_packets
                                                        );
 
                                                        if (ifc.ipaddrs && ifc.ipaddrs.length)
index c554df88d3fb3f994c8173c87be8deeab130332c..523a780fabba923378d50f723748de336d8f66cf 100644 (file)
@@ -4,7 +4,7 @@
        var iwxhr = new XHR();
 
        function iface_shutdown(id, reconnect) {
-               if (!reconnect && !confirm(String.format('<%_Really shutdown network ?\nYou might loose access to this router if you are connected via this interface.%>', id)))
+               if (!reconnect && !confirm(String.format('<%_Really shutdown network ?\nYou might lose access to this router if you are connected via this interface.%>', id)))
                        return;
 
                var a = document.getElementById(id + '-ifc-addrs');
                        {
                                if (ifc && (ifc = ifc[0]))
                                {
-                                       var rxb = ifc.rx_bytes || 0;
-                                       var txb = ifc.tx_bytes || 0;
-                                       var rxp = ifc.rx_packets || 0;
-                                       var txp = ifc.tx_packets || 0;
-                                       var mac = ifc.macaddr;
-
                                        var html = '';
-                                       var upt, icon;
-
-                                       if (ifc.is_up)
-                                       {
-                                               if (ifc.uptime)
-                                                       upt = String.format('%t', ifc.uptime);
-
-                                               icon = "<%=resource%>/icons/%s.png";
-                                       }
-                                       else
-                                       {
-                                               icon = "<%=resource%>/icons/%s_disabled.png";
-                                       }
 
                                        var s = document.getElementById('<%=self.option%>-ifc-signal');
                                        if (s)
                                                );
 
                                        var d = document.getElementById('<%=self.option%>-ifc-description');
-                                       if (d && ifc.ifname && mac != '00:00:00:00:00:00')
+                                       if (d && ifc.ifname && ifc.macaddr != '00:00:00:00:00:00')
                                        {
-                                               if (upt)
+                                               if (ifc.is_up)
                                                {
-                                                       html += String.format('<strong><%:Uptime%>:</strong> %s<br />', upt);
+                                                       html += String.format('<strong><%:Uptime%>:</strong> %t<br />', ifc.uptime);
                                                }
 
                                                if (ifc.type != 'tunnel')
                                                {
-                                                       html += String.format('<strong><%:MAC Address%>:</strong> %s<br />', mac);
+                                                       html += String.format('<strong><%:MAC Address%>:</strong> %s<br />', ifc.macaddr);
                                                }
 
                                                html += String.format(
                                                        '<strong><%:RX%></strong>: %.2mB (%d <%:Pkts.%>)<br />' +
                                                        '<strong><%:TX%></strong>: %.2mB (%d <%:Pkts.%>)<br />',
-                                                               rxb, rxp, txb, txp
+                                                               ifc.rx_bytes, ifc.rx_packets,
+                                                               ifc.tx_bytes, ifc.tx_packets
                                                );
 
                                                if (ifc.ipaddrs && ifc.ipaddrs.length)