From 8a655557b782ba24df0cbd6d8b967c6e08a824c5 Mon Sep 17 00:00:00 2001
From: kb-light <freifunk@kb-light.de>
Date: Wed, 27 Apr 2016 23:52:06 +0200
Subject: [PATCH] gluon-node-info: iterate through a list of options

---
 .../gluon/upgrade/520-node-info-whitespace-fix | 18 ++++++------------
 1 file changed, 6 insertions(+), 12 deletions(-)

diff --git a/package/gluon-node-info/files/lib/gluon/upgrade/520-node-info-whitespace-fix b/package/gluon-node-info/files/lib/gluon/upgrade/520-node-info-whitespace-fix
index 6597598b..49e7053d 100755
--- a/package/gluon-node-info/files/lib/gluon/upgrade/520-node-info-whitespace-fix
+++ b/package/gluon-node-info/files/lib/gluon/upgrade/520-node-info-whitespace-fix
@@ -1,20 +1,14 @@
 #!/usr/bin/lua
-
 local uci = require('luci.model.uci').cursor()
 
 local sname = uci:get_first('gluon-node-info', 'location')
 if sname then
-  local latitude = uci:get('gluon-node-info', sname, 'latitude')
-  if latitude then
-    uci:set('gluon-node-info', sname, 'latitude', latitude:trim())
-  end
-  local longitude = uci:get('gluon-node-info', sname, 'longitude')
-  if longitude then
-    uci:set('gluon-node-info', sname, 'longitude', longitude:trim())
-  end
-  local altitude = uci:get('gluon-node-info', sname, 'altitude')
-  if altitude then
-    uci:set('gluon-node-info', sname, 'altitude', altitude:trim())
+  local options = {'longitude', 'latitude', 'altitude'}
+  for _, option in ipairs(options) do
+    local value = uci:get('gluon-node-info', sname, option)
+    if value then
+      uci:set('gluon-node-info', sname, option, value:trim())
+    end
   end
   uci:save('gluon-node-info')
 end