More UI polish, easymesh.sh actually gets run now on apply

luasrc/model/cbi/easymesh.lua

@@ -13,34 +13,28 @@ m = Map("easymesh",
 
 
 function detect_Node()
-	local data = {}
+    local data = {}
-	local lps = luci.util.execi(" batctl n 2>/dev/null | tail +2 | sed 's/^[ ][ ]*//g' | sed 's/[ ][ ]*/ /g' | sed 's/$/ /g' ")
+    -- Streamline the command to retrieve node information
-	for value in lps do
+    local lps = luci.util.execi("batctl n 2>/dev/null | tail +2 | sed 's/^[ ]*//;s/[ ]*/ /g'")
-		local row = {}
+
-		local pos = string.find(value, " ")
+    for line in lps do
-		local IFA = string.sub(value, 1, pos - 1)
+        -- Split the line into parts using pattern matching
-		local value = string.sub(value, pos + 1, string.len(value))
+        local ifa, neighbor, lastseen = string.match(line, "(%S+) (%S+) (%S+)")
-		pos = string.find(value, " ")
+        
-		local pos = string.find(value, " ")
+        if ifa and neighbor and lastseen then
-		local Neighbora = string.sub(value, 1, pos - 1)
+            -- Construct the row and add it to the data table
-		local value = string.sub(value, pos + 1, string.len(value))
+            local row = {["IF"] = ifa, ["Neighbor"] = neighbor, ["lastseen"] = lastseen}
-		pos = string.find(value, " ")
+            table.insert(data, row)
-		local pos = string.find(value, " ")
+        end
-		local lastseena = string.sub(value, 1, pos - 1)
+    end
-		local value = string.sub(value, pos + 1, string.len(value))
+    return data
-		pos = string.find(value, " ")
-		row["IF"] = IFA
-		row["Neighbor"] = Neighbora
-		row["lastseen"] = lastseena
-		table.insert(data, row)
-	end
-	return data
 end
 
+
 function get_verbose_hw_info(iface)
     local type = iwinfo.type(iface)
     if not type then
-        return "Generic Wifi Device"
+        return "Generic"
     end
 
     local driver = iwinfo[type]
@@ -68,17 +62,16 @@ function get_verbose_hw_info(iface)
     else
         supported_modes_str = table.concat(supported_modes, "/")
     end
-
     local verbose_info = hw_name .. " (" .. supported_modes_str .. ")"
     return verbose_info
 end
 
 local Nodes = luci.sys.exec("batctl n 2>/dev/null| tail +3 | wc -l")
 local Node = detect_Node()
-v = m:section(Table, Node, "" ,"<b>" .. translate("Active node") .. "" .. Nodes .. "</b>")
+v = m:section(Table, Node, translate("Active Nodes") ,"<b>" .. translate("Number of Active Nodes: ") .. Nodes .. "</b>")
-v:option(DummyValue, "IF", translate("IF"))
+v:option(DummyValue, "IF", translate("Interface"))
-v:option(DummyValue, "Neighbor", translate("Neighbor"))
+v:option(DummyValue, "Neighbor", translate("Neighbor Nodes"))
-v:option(DummyValue, "lastseen", translate("lastseen"))
+v:option(DummyValue, "lastseen", translate("Last Seen Timestamp"))
 
 -- Basic
 s = m:section(TypedSection, "easymesh", translate("Mesh Gateway & Node WiFi Settings"), translate("Choose Mesh Gateway or Mesh Node WiFi Settings: Begin by completing this section on your mesh server and nodes. Ensure each radio's WiFi network name is consistent. For example: easymesh_AC, easymesh_N."))
@@ -226,17 +219,12 @@ function btnAPMode.write()
 end
 btnAPMode:depends("role", "off")
 
--- MESH Node Control: apply mesh settings
-ctrl = m:section(TypedSection, "easymesh", "Click Save Then Enable or Disable Your Mesh WiFi settings")
-ctrl.anonymous = true
-ctrl.addremove = false
-
 m.on_before_apply = function(self)
-    local enabled = m:formvalue("cbid.easymesh.easymesh.enabled")
+    local enabled = m:formvalue("cbid.easymesh.config.enabled")
     if enabled and enabled == "1" then
-        luci.sys.call("/easymesh/easymesh.sh &")
+        luci.sys.call("/easymesh/easymesh.sh enable &")
     else
-        luci.sys.call("/easymesh/easymesh.sh &")
+        luci.sys.call("/easymesh/easymesh.sh disable &")
     end
 end
 

root/easymesh/easymesh.sh

@@ -410,14 +410,17 @@ disable_easymesh() {
     uci del easymesh.config.running
 }
 
-if [ "$1" = "dumbap" ]; then
+sleep 5
+if [ "$1" = "enable" ]; then
+    enable_easymesh
+    exit 0
+elif [ "$1" = "disable" ]; then
+    disable_easymesh
+    exit 0
+elif [ "$1" = "dumbap" ]; then
     set_apmode
     exit 0
-fi
-
-# Check if enabled
-if [ "$(uci -q get easymesh.config.enabled)" = 1 ]; then
-    enable_easymesh
 else
-    disable_easymesh
+    echo "Invalid argument. Please use 'start', 'stop' or 'dumbap'"
+    exit 1
 fi