[luci] [PATCH] ddns: make 'service_name' a ListValue

Benoît Knecht benoit.knecht at gmail.com
Tue Mar 30 14:43:18 CEST 2010


Make 'service_name' a ListValue again, and add the empty string as a
possible value as this option is not required.
---

The 'service_name' option used to be a ListValue, but r5989 turned it
into a Value. This was presumably a mistake, so this patch reverts it
back.

Benoît Knecht
---

diff --git a/applications/luci-ddns/luasrc/model/cbi/ddns/ddns.lua b/applications/luci-ddns/luasrc/model/cbi/ddns/ddns.lua
index ada0ee1..59db5b4 100644
--- a/applications/luci-ddns/luasrc/model/cbi/ddns/ddns.lua
+++ b/applications/luci-ddns/luasrc/model/cbi/ddns/ddns.lua
@@ -24,10 +24,10 @@ s.anonymous = false
 
 s:option(Flag, "enabled", translate("enable"))
 
-svc = s:option(Value, "service_name", translate("Service"))
+svc = s:option(ListValue, "service_name", translate("Service"))
 svc.rmempty = true
 
-local services = { }
+local services = {""}
 local fd = io.open("/usr/lib/ddns/services", "r")
 if fd then
 	local ln
-- 
1.7.0.3



More information about the luci mailing list