luci-mod-network: handle multiple mac for static lease

The mac section for the static lease doesn't correctly handle when multiple mac are set for a rule.
Fixes: #4291
Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com>

diff --git a/modules/luci-mod-network/htdocs/luci-static/resources/view/network/dhcp.js b/modules/luci-mod-network/htdocs/luci-static/resources/view/network/dhcp.js
index 37cc29f32a25b11f6bab0b3c04b2a1be6369c1a3..7f46740da43e55ee70df243d416941597edd2360 100644 (file)
--- a/modules/luci-mod-network/htdocs/luci-static/resources/view/network/dhcp.js
+++ b/modules/luci-mod-network/htdocs/luci-static/resources/view/network/dhcp.js
@@ -432,12 +432,9 @@ return view.extend({
                so.datatype = 'list(unique(macaddr))';
                so.rmempty  = true;
                so.cfgvalue = function(section) {
-                       var macs = uci.get('dhcp', section, 'mac'),
+                       var macs = L.toArray(uci.get('dhcp', section, 'mac')),
                            result = [];
 
-                       if (!Array.isArray(macs))
-                               macs = (macs != null && macs != '') ? macs.split(/\ss+/) : [];
-
                        for (var i = 0, mac; (mac = macs[i]) != null; i++)
                                if (/^([0-9a-fA-F]{1,2}):([0-9a-fA-F]{1,2}):([0-9a-fA-F]{1,2}):([0-9a-fA-F]{1,2}):([0-9a-fA-F]{1,2}):([0-9a-fA-F]{1,2})$/.test(mac))
                                        result.push('%02X:%02X:%02X:%02X:%02X:%02X'.format(