luci-mod-network: made expandAndFormatMAC return arrays

instead of returning a space separated string

follow-up to 5ab0cb1f15bcb31015ae8345d302c6d023ba51e2

Closes #7209

Signed-off-by: Paul Donald <newtwen+github@gmail.com>

1 files changed, 7 insertions, 3 deletions

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 4f6fe79385..ef466c115f 100644
--- 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
@@ -221,7 +221,7 @@ function expandAndFormatMAC(macs) {
 		}
 	});
 
-	return result.length ? result.join(' ') : null;
+	return result.length ? result : null;
 }
 
 function isValidMAC(sid, s) {
@@ -1039,8 +1039,12 @@ return view.extend({
 		//As a special case, in DHCPv4, it is possible to include more than one hardware address. eg: --dhcp-host=11:22:33:44:55:66,12:34:56:78:90:12,192.168.0.2 This allows an IP address to be associated with multiple hardware addresses, and gives dnsmasq permission to abandon a DHCP lease to one of the hardware addresses when another one asks for a lease
 		so.rmempty  = true;
 		so.cfgvalue = function(section) {
-			var macs = L.toArray(uci.get('dhcp', section, 'mac'));
-			return expandAndFormatMAC(macs);
+			var macs = uci.get('dhcp', section, 'mac');
+			if(!Array.isArray(macs)){
+				return expandAndFormatMAC(L.toArray(macs));
+			} else {
+				return expandAndFormatMAC(macs);
+			}
 		};
 		//removed jows renderwidget function which hindered multi-mac entry
 		so.validate = validateMACAddr.bind(so, pools);