From 71ee67ec182690406a7983fcdd1df6f36011cd77 Mon Sep 17 00:00:00 2001 From: lemoer Date: Thu, 12 Aug 2021 03:05:04 +0200 Subject: [PATCH] gluon-config-api-geo-location: simplify set --- .../gluon/config-api/parts/geo-location.lua | 22 +++++-------------- 1 file changed, 6 insertions(+), 16 deletions(-) diff --git a/package/gluon-config-api-geo-location/luasrc/lib/gluon/config-api/parts/geo-location.lua b/package/gluon-config-api-geo-location/luasrc/lib/gluon/config-api/parts/geo-location.lua index 3ff5e97d..811d725b 100644 --- a/package/gluon-config-api-geo-location/luasrc/lib/gluon/config-api/parts/geo-location.lua +++ b/package/gluon-config-api-geo-location/luasrc/lib/gluon/config-api/parts/geo-location.lua @@ -30,23 +30,13 @@ end function M.set(config, uci) local location = uci:get_first("gluon-node-info", "location") - local config_location = config.wizard.location + local config_location = config.wizard.location or {} - if config_location then - uci:set("gluon-node-info", location, "share_location", config_location.share_location) - uci:set("gluon-node-info", location, "latitude", config_location.lat) - uci:set("gluon-node-info", location, "longitude", config_location.lon) - if config_location.altitude then -- altitude is optional - uci:set("gluon-node-info", location, "altitude", config_location.altitude) -- TODO: check if the "if" is necessary - else - uci:delete("gluon-node-info", location, "altitude") - end - else - uci:set("gluon-node-info", location, "share_location", false) - uci:delete("gluon-node-info", location, "latitude") - uci:delete("gluon-node-info", location, "longitude") - uci:delete("gluon-node-info", location, "altitude") - end + uci:set("gluon-node-info", location, "share_location", + config_location.share_location or false) + uci:set("gluon-node-info", location, "latitude", config_location.lat) + uci:set("gluon-node-info", location, "longitude", config_location.lon) + uci:set("gluon-node-info", location, "altitude", config_location.altitude) uci:save("gluon-node-info") end