luci-app-simple-adblock: bugfix for service start/stop
[project/luci.git] / applications / luci-app-simple-adblock / luasrc / model / cbi / simpleadblock.lua
index 214f298..2267212 100644 (file)
@@ -2,19 +2,11 @@ m = Map("simple-adblock", translate("Simple AdBlock Settings"))
 s = m:section(NamedSection, "config", "simple-adblock")
 
 -- General options
-e = s:option(Flag, "enabled", translate("Enable/start service"))
+e = s:option(Flag, "enabled", translate("Start Simple Adblock service"))
 e.rmempty  = false
-
-function e.cfgvalue(self, section)
-       return self.map:get(section, "enabled") == "1" and luci.sys.init.enabled("simple-adblock") and self.enabled or self.disabled
-end
-
 function e.write(self, section, value)
-       if value == "1" then
-               luci.sys.call("/etc/init.d/simple-adblock enable >/dev/null")
-               luci.sys.call("/etc/init.d/simple-adblock start >/dev/null")
-       else
-               luci.sys.call("/etc/init.d/simple-adblock stop >/dev/null")
+       if value ~= "1" then
+               luci.sys.init.stop("simple-adblock")
        end
        return Flag.write(self, section, value)
 end
@@ -32,15 +24,14 @@ o3:value("1", translate("Force Router DNS server to all local devices"))
 o3.rmempty = false
 o3.default = 1
 
-
 local sysfs_path = "/sys/class/leds/"
 local leds = {}
 if nixio.fs.access(sysfs_path) then
        leds = nixio.util.consume((nixio.fs.dir(sysfs_path)))
 end
 if #leds ~= 0 then
-       o3 = s:option(Value, "led", translate("LED to indicate status"), translate("Pick the LED not already used in ")
-               .. [[<a href="]] .. luci.dispatcher.build_url("admin/system/leds") .. [[">]]
+       o3 = s:option(Value, "led", translate("LED to indicate status"), translate("Pick the LED not already used in")
+               .. [[ <a href="]] .. luci.dispatcher.build_url("admin/system/leds") .. [[">]]
                .. translate("System LED Configuration") .. [[</a>]])
        o3.rmempty = true
        o3:value("", translate("none"))
@@ -49,7 +40,6 @@ if #leds ~= 0 then
        end
 end
 
-
 s2 = m:section(NamedSection, "config", "simple-adblock")
 -- Whitelisted Domains
 d1 = s2:option(DynamicList, "whitelist_domain", translate("Whitelisted Domains"), translate("Individual domains to be whitelisted"))