luci-mod-admin-full: automatically set dns=1 for static leases with hostname
authorJo-Philipp Wich <jo@mein.io>
Thu, 1 Jun 2017 23:00:12 +0000 (01:00 +0200)
committerJo-Philipp Wich <jo@mein.io>
Thu, 1 Jun 2017 23:02:42 +0000 (01:02 +0200)
Most users likely want the hostname of a static DHCP lease to be resolveable,
even if the corresponding host is currently offline.

Adjust the form logic to automatically set 'option dns 1' whenever a name is
specified in the static lease entry.

FIxes LEDE FS#815.

Signed-off-by: Jo-Philipp Wich <jo@mein.io>
modules/luci-mod-admin-full/luasrc/model/cbi/admin_network/dhcp.lua

index f418ecb40f050db00a895a50d98b1bbac96c0b07..c7d330e68dd463fe12495d7f112afb3507ebeefa 100644 (file)
@@ -276,6 +276,16 @@ name = s:option(Value, "name", translate("Hostname"))
 name.datatype = "hostname"
 name.rmempty  = true
 
+function name.write(self, section, value)
+       Value.write(self, section, value)
+       m:set(section, "dns", "1")
+end
+
+function name.remove(self, section)
+       Value.remove(self, section)
+       m:del(section, "dns")
+end
+
 mac = s:option(Value, "mac", translate("<abbr title=\"Media Access Control\">MAC</abbr>-Address"))
 mac.datatype = "list(macaddr)"
 mac.rmempty  = true