OSDN Git Service

luci-app-mentohust: code alignment
authorBeginner-Go <70857188+Beginner-Go@users.noreply.github.com>
Mon, 16 May 2022 17:27:28 +0000 (17:27 +0000)
committerTianling Shen <cnsztl@gmail.com>
Wed, 18 May 2022 08:26:58 +0000 (16:26 +0800)
applications/luci-app-mentohust/Makefile
applications/luci-app-mentohust/luasrc/controller/mentohust.lua
applications/luci-app-mentohust/luasrc/model/cbi/mentohust/general.lua
applications/luci-app-mentohust/luasrc/model/cbi/mentohust/log.lua
applications/luci-app-mentohust/root/etc/uci-defaults/luci-mentohust [new file with mode: 0755]

index 744f1af..ddb04e9 100644 (file)
@@ -1,7 +1,6 @@
 include $(TOPDIR)/rules.mk
 
 LUCI_TITLE:=MentoHUST 802.1X Client for LuCI
-LUCI_DESCRIPTION:=This package contains LuCI configuration pages for 8021xclient.
 LUCI_DEPENDS:=+mentohust
 LUCI_PKGARCH:=all
 
@@ -12,4 +11,3 @@ PKG_RELEASE:=4
 include ../../luci.mk
 
 # call BuildPackage - OpenWrt buildroot signature
-
index 522116b..7bd05f1 100644 (file)
@@ -1,18 +1,19 @@
 module("luci.controller.mentohust", package.seeall)
 
 function index()
-    if not nixio.fs.access("/etc/config/mentohust") then
-        return
-    end
-    if luci.sys.call("command -v mentohust >/dev/null") ~= 0 then
-        return
-    end
-    local page = entry({"admin", "services", "mentohust"},
-        alias("admin", "services", "mentohust", "general"),
-        _("MentoHUST"))
-    page.order = 10
-    page.dependent = true
-    page.acl_depends = { "luci-app-mentohust" }
-    entry({"admin", "services", "mentohust", "general"}, cbi("mentohust/general"), _("MentoHUST Settings"), 10).leaf = true
-    entry({"admin", "services", "mentohust", "log"}, cbi("mentohust/log"), _("MentoHUST LOG"), 20).leaf = true
+       if not nixio.fs.access("/etc/config/mentohust") then
+               return
+       end
+
+       if luci.sys.call("command -v mentohust >/dev/null") ~= 0 then
+               return
+       end
+
+       local page = entry({"admin", "services", "mentohust"}, alias("admin", "services", "mentohust", "general"), _("MentoHUST"))
+       page.order = 10
+       page.dependent = true
+       page.acl_depends = { "luci-app-mentohust" }
+
+       entry({"admin", "services", "mentohust", "general"}, cbi("mentohust/general"), _("MentoHUST Settings"), 10).leaf = true
+       entry({"admin", "services", "mentohust", "log"}, cbi("mentohust/log"), _("MentoHUST LOG"), 20).leaf = true
 end
index 0bf6153..c0fb8c6 100644 (file)
@@ -1,33 +1,35 @@
 local function is_running(name)
-    if luci.sys.call("pidof %s >/dev/null" %{name}) == 0 then
-        return translate("RUNNING")
-    else
-        return translate("NOT RUNNING")
-    end
+       if luci.sys.call("pidof %s >/dev/null" %{name}) == 0 then
+               return translate("RUNNING")
+       else
+               return translate("NOT RUNNING")
+       end
 end
 
 local function is_online(ipaddr)
-    if ipaddr == "0.0.0.0" then 
-        return translate("Pinghost not set")
-    end
-    if luci.sys.call("ping -c1 -w1 %s >/dev/null 2>&1" %{ipaddr}) == 0 then
-        return translate("ONLINE")
-    else
-        return translate("NOT ONLINE")
-    end
+       if ipaddr == "0.0.0.0" then 
+               return translate("Pinghost not set")
+       end
+       if luci.sys.call("ping -c1 -w1 %s >/dev/null 2>&1" %{ipaddr}) == 0 then
+               return translate("ONLINE")
+       else
+               return translate("NOT ONLINE")
+       end
 end
 
 require("luci.sys")
 
-m = Map("mentohust", translate("MentoHUST"), translate("Configure MentoHUST 802.11x."))
+m = Map("mentohust", translate("MentoHUST"))
+m.description = translate("Configure MentoHUST 802.11x.")
 
 s = m:section(TypedSection, "mentohust", translate("Status"))
 s.anonymous = true
+
 status = s:option(DummyValue,"_mentohust_status", "MentoHUST")
 status.value = "<span id=\"_mentohust_status\">%s</span>" %{is_running("mentohust")}
 status.rawhtml = true
 t = io.popen('uci get mentohust.@mentohust[0].pinghost')
-netstat=is_online(tostring(t:read("*line")))
+netstat = is_online(tostring(t:read("*line")))
 t:close()
 if netstat ~= "" then
 netstatus = s:option(DummyValue,"_network_status", translate("Network Status"))
@@ -43,69 +45,88 @@ o:tab("base", translate("Normal Settings"))
 o:tab("advanced", translate("Advanced Settings"))
 
 enable = o:taboption("base", Flag, "enable", translate("Enable"))
-name = o:taboption("base", Value, "username", translate("Username"),translate("The username given to you by your network administrator"))
-pass = o:taboption("base", Value, "password", translate("Password"), translate("The password you set or given to you by your network administrator"))
+
+name = o:taboption("base", Value, "username", translate("Username"))
+name.description = translate("The username given to you by your network administrator")
+
+pass = o:taboption("base", Value, "password", translate("Password"))
+pass.description = translate("The password you set or given to you by your network administrator")
 pass.password = true
 
-ifname = o:taboption("base", ListValue, "ifname", translate("Interfaces"), translate("Physical interface of WAN"))
+ifname = o:taboption("base", ListValue, "ifname", translate("Interfaces"))
+ifname.description = translate("Physical interface of WAN")
 for k, v in ipairs(luci.sys.net.devices()) do
     if v ~= "lo" then
         ifname:value(v)
     end
 end
 
-pinghost = o:taboption("base", Value, "pinghost", translate("PingHost"), translate("Ping host for drop detection, 0.0.0.0 to turn off this feature"))
+pinghost = o:taboption("base", Value, "pinghost", translate("PingHost"))
+pinghost.description = translate("Ping host for drop detection, 0.0.0.0 to turn off this feature")
 pinghost.default = "0.0.0.0"
 
-ipaddr = o:taboption("advanced", Value, "ipaddr", translate("IP Address"), translate("Your IPv4 Address. (DHCP users can set to 0.0.0.0)"))
+ipaddr = o:taboption("advanced", Value, "ipaddr", translate("IP Address"))
+ipaddr.description = translate("Your IPv4 Address. (DHCP users can set to 0.0.0.0)")
 ipaddr.default = "0.0.0.0"
 
-mask = o:taboption("advanced", Value, "mask", translate("NetMask"), translate("NetMask, it doesn't matter"))
+mask = o:taboption("advanced", Value, "mask", translate("NetMask"))
+mask.description = translate("NetMask, it doesn't matter")
 mask.default = "0.0.0.0"
 
-gateway = o:taboption("advanced", Value, "gateway", translate("Gateway"), translate("Gateway, if specified, will monitor gateway ARP information"))
+gateway = o:taboption("advanced", Value, "gateway", translate("Gateway"))
+gateway.description = translate("Gateway, if specified, will monitor gateway ARP information")
 gateway.default = "0.0.0.0"
 
-dnsserver = o:taboption("advanced", Value, "dns", translate("DNS server"), translate("DNS server, it doesn't matter"))
+dnsserver = o:taboption("advanced", Value, "dns", translate("DNS server"))
+dnsserver.description = translate("DNS server, it doesn't matter")
 dnsserver.default = "0.0.0.0"
 
-timeout = o:taboption("advanced", Value, "timeout", translate("Timeout"), translate("Each timeout of the package (seconds)"))
+timeout = o:taboption("advanced", Value, "timeout", translate("Timeout"))
+timeout.description = translate("Each timeout of the package (seconds)")
 timeout.default = "8"
 
-echointerval = o:taboption("advanced", Value, "echointerval", translate("EchoInterval"), translate("Interval for sending Echo packets (seconds)"))
+echointerval = o:taboption("advanced", Value, "echointerval", translate("EchoInterval"))
+echointerval.description = translate("Interval for sending Echo packets (seconds)")
 echointerval.default = "30"
 
-restartwait = o:taboption("advanced", Value, "restartwait", translate("RestartWait"), translate("Failed Wait (seconds) Wait for seconds after authentication failed or restart authentication after server request"))
+restartwait = o:taboption("advanced", Value, "restartwait", translate("RestartWait"))
+restartwait.description = translate("Failed Wait (seconds) Wait for seconds after authentication failed or restart authentication after server request")
 restartwait.default = "15"
 
-startmode = o:taboption("advanced", ListValue, "startmode", translate("StartMode"), translate("Multicast address type when searching for servers"))
+startmode = o:taboption("advanced", ListValue, "startmode", translate("StartMode"))
+startmode.description = translate("Multicast address type when searching for servers")
 startmode:value(0, translate("Standard"))
 startmode:value(1, translate("Ruijie"))
 startmode:value(2, translate("Uses MentoHUST for Xaar certification"))
 startmode.default = "0"
 
-dhcpmode = o:taboption("advanced", ListValue, "dhcpmode", translate("DhcpMode"), translate("DHCP method"))
+dhcpmode = o:taboption("advanced", ListValue, "dhcpmode", translate("DhcpMode"))
+dhcpmode.description = translate("DHCP method")
 dhcpmode:value(0, translate("None"))
 dhcpmode:value(1, translate("secondary authentication"))
 dhcpmode:value(2, translate("after certification"))
 dhcpmode:value(3, translate("before certification"))
 dhcpmode.default = "2"
 
-shownotify = o:taboption("advanced", Value, "shownotify", translate("ShowNotify"), translate("Whether to display notifications 0 (no) 1 to 20 (yes)"))
+shownotify = o:taboption("advanced", Value, "shownotify", translate("ShowNotify"))
+shownotify.description = translate("Whether to display notifications 0 (no) 1 to 20 (yes)")
 shownotify.default = "5"
 
-version = o:taboption("advanced", Value, "version", translate("Client Version"), translate("Client version number. If client verification is not enabled but the version number is required, it can be specified here. The format is 3.30."))
+version = o:taboption("advanced", Value, "version", translate("Client Version"))
+version.description = translate("Client version number. If client verification is not enabled but the version number is required, it can be specified here. The format is 3.30.")
 version.default = "0.00"
 
-datafile = o:taboption("advanced", Value, "datafile", translate("DataFile"), translate("Authentication data file, if you need to verify the client, you need to set correctly"))
+datafile = o:taboption("advanced", Value, "datafile", translate("DataFile"))
+datafile.description = translate("Authentication data file, if you need to verify the client, you need to set correctly")
 datafile.default = "/etc/mentohust/"
 
-dhcpscript = o:taboption("advanced", Value, "dhcpscript", translate("DhcpScript"), translate("DHCP script"))
+dhcpscript = o:taboption("advanced", Value, "dhcpscript", translate("DhcpScript"))
+dhcpscript.description = translate("DHCP script")
 dhcpscript.default = "udhcpc -i"
 
 local apply = luci.http.formvalue("cbi.apply")
 if apply then
-    io.popen("/etc/init.d/mentohust restart")
+       io.popen("/etc/init.d/mentohust restart")
 end
 
 return m
diff --git a/applications/luci-app-mentohust/root/etc/uci-defaults/luci-mentohust b/applications/luci-app-mentohust/root/etc/uci-defaults/luci-mentohust
new file mode 100755 (executable)
index 0000000..2ef80be
--- /dev/null
@@ -0,0 +1,11 @@
+#!/bin/sh
+
+uci -q batch <<-EOF >/dev/null
+       delete ucitrack.@mentohust[-1]
+       add ucitrack mentohust
+       set ucitrack.@mentohust[-1].init=mentohust
+       commit ucitrack
+EOF
+
+rm -rf /tmp/luci-*
+exit 0