Difuse/luci
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

luci-app-adblock: minor fixes

Browse source 
* use new XHR default on report page
* fix wrong string.find syntax

Signed-off-by: Dirk Brenken <dev@brenken.org>
This commit is contained in:
Dirk Brenken 2018-12-22 18:37:45 +01:00
parent 5aa2d3a77f
commit 0299b13292
2 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
applications/luci-app-adblock/luasrc/controller/adblock.lua
View file

@ -54,7 +54,7 @@ function adb_action(name, domain)
local file = uci:get("adblock", "blacklist", "adb_src") or "/etc/adblock/adblock.blacklist"
if nixio.fs.access(file) then
local blacklist = nixio.fs.readfile(file)
if not string.find(blacklist, domain, 1, plain)
if not string.find(blacklist, domain, 1, true)
then
nixio.fs.writefile(file, blacklist.. domain.. "\n")
end
@ -63,7 +63,7 @@ function adb_action(name, domain)
local file = uci:get("adblock", "global", "adb_whitelist") or "/etc/adblock/adblock.whitelist"
if nixio.fs.access(file) then
local whitelist = nixio.fs.readfile(file)
if not string.find(whitelist, domain, 1, plain)
if not string.find(whitelist, domain, 1, true)
then
nixio.fs.writefile(file, whitelist.. domain.. "\n")
end

4
applications/luci-app-adblock/luasrc/view/adblock/report.htm
View file

@ -166,7 +166,7 @@ This is free software, licensed under the Apache License, Version 2.0
report_json(json_info);
});
XHR.poll(5, '<%=luci.dispatcher.build_url("admin", "services", "adblock", "report_json")%>', null,
XHR.poll(-1, '<%=luci.dispatcher.build_url("admin", "services", "adblock", "report_json")%>', null,
function(x, json_info)
{
if (!x || !json_info || !json_info.data)
@ -188,7 +188,7 @@ This is free software, licensed under the Apache License, Version 2.0
report_text(x.responseText);
});
XHR.poll(5, '<%=luci.dispatcher.build_url("admin", "services", "adblock", "report_text")%>', null,
XHR.poll(-1, '<%=luci.dispatcher.build_url("admin", "services", "adblock", "report_text")%>', null,
function(x)
{
if (!x || !x.responseText)