luci-mod-network: improve static lease ip validation logic
authorPaul Donald <newtwen@gmail.com>
Thu, 25 Jan 2024 00:05:15 +0000 (01:05 +0100)
committerPaul Donald <newtwen@gmail.com>
Thu, 25 Jan 2024 00:09:12 +0000 (01:09 +0100)
m (mac) is an array, and is zero length or greater.

Signed-off-by: Paul Donald <newtwen@gmail.com>
modules/luci-mod-network/htdocs/luci-static/resources/view/network/dhcp.js

index 2c141d307850d7843c2a94ac4f87e626d9319439..2967e99b4735c1da19a7be4a3a20272d85294f1a 100644 (file)
@@ -942,10 +942,10 @@ return view.extend({
                        var m = this.section.formvalue(section, 'mac'),
                            n = this.section.formvalue(section, 'name');
 
-                       if ((m == null || m == '') && (n == null || n == ''))
+                       if ((m && !m.length > 0) && !n)
                                return _('One of hostname or MAC address must be specified!');
 
-                       if (value == null || value == '' || value == 'ignore')
+                       if (!value || value == 'ignore')
                                return true;
 
                        var leases = uci.sections('dhcp', 'host');