nils/gluon
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

gluon-web-wifi-config: rename and simplify filter_active_interfaces()

Browse Source 
- Return early
- Rename to has_active_interfaces(), as it returns a boolean
This commit is contained in:
Matthias Schiffer 2020-06-06 16:48:26 +02:00
parent 600ab99f80
commit 53c33acaf9
No known key found for this signature in database
GPG Key ID: 16EF3F64CB201D9C
1 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
package/gluon-web-wifi-config/luasrc/lib/gluon/config-mode/model/admin/wifi-config.lua
View File

@ -71,14 +71,13 @@ uci:foreach('wireless', 'wifi-device', function(config)
return out
end
local function filter_active_interfaces(interfaces)
local out = false
local function has_active_interfaces(interfaces)
for _, interface in ipairs(interfaces) do
if not uci:get_bool('wireless', interface .. '_' .. radio, 'disabled') then
out = true
return true
end
end
return out
return false
end
local function vif_option(name, interfaces, msg)
@ -89,7 +88,7 @@ uci:foreach('wireless', 'wifi-device', function(config)
end
local o = p:option(Flag, radio .. '_' .. name .. '_enabled', msg)
o.default = filter_active_interfaces(existing_interfaces)
o.default = has_active_interfaces(existing_interfaces)
function o:write(data)
for _, interface in ipairs(existing_interfaces) do