From 3c86c6562ed870bb476349857ca59e4bb14947e2 Mon Sep 17 00:00:00 2001
From: Florian Eckert <fe@dev.tdt.de>
Date: Tue, 27 Feb 2018 15:00:50 +0100
Subject: luci-app-mwan3: fix stack trace if no tacking_ip is set on interface
 add

Signed-off-by: Florian Eckert <fe@dev.tdt.de>
---
 applications/luci-app-mwan3/luasrc/model/cbi/mwan/interface.lua | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/applications/luci-app-mwan3/luasrc/model/cbi/mwan/interface.lua b/applications/luci-app-mwan3/luasrc/model/cbi/mwan/interface.lua
index 920dc6afb..4b1a070d1 100644
--- a/applications/luci-app-mwan3/luasrc/model/cbi/mwan/interface.lua
+++ b/applications/luci-app-mwan3/luasrc/model/cbi/mwan/interface.lua
@@ -93,7 +93,7 @@ function configCheck()
 
 			local trackingNumber = uci:get("mwan3", iface, "track_ip")
 			overview[iface]["tracking"] = 0
-			if #trackingNumber > 0 then
+			if trackingNumber and #trackingNumber > 0 then
 				overview[iface]["tracking"] = #trackingNumber
 				overview[iface]["reliability"] = false
 				local reliabilityNumber = tonumber(uci:get("mwan3", iface, "reliability"))
-- 
cgit v1.2.3


From a82e588c068c407b52c9d329375d22ef35c57bb5 Mon Sep 17 00:00:00 2001
From: Florian Eckert <fe@dev.tdt.de>
Date: Thu, 1 Mar 2018 08:51:51 +0100
Subject: luci-app-mwan3: remove css styling from overview_status_interface
 page

Signed-off-by: Florian Eckert <fe@dev.tdt.de>
---
 .../luasrc/view/mwan/overview_status_interface.htm | 30 ----------------------
 1 file changed, 30 deletions(-)

diff --git a/applications/luci-app-mwan3/luasrc/view/mwan/overview_status_interface.htm b/applications/luci-app-mwan3/luasrc/view/mwan/overview_status_interface.htm
index 6dc3d1274..bb7aaca7b 100644
--- a/applications/luci-app-mwan3/luasrc/view/mwan/overview_status_interface.htm
+++ b/applications/luci-app-mwan3/luasrc/view/mwan/overview_status_interface.htm
@@ -56,33 +56,3 @@ XHR.poll(5, '<%=luci.dispatcher.build_url("admin", "status", "mwan", "interface_
 	<legend><%:MWAN Interface Live Status%></legend>
 	<div id="mwan_status_text"><img src="<%=resource%>/icons/loading.gif" alt="<%:Loading%>" style="vertical-align:middle" /><%:Collecting data...%></div>
 </fieldset>
-
-<style type="text/css">
-  .container {	/* container for entire page. fixes bootstrap theme's ridiculously small page width */
-	max-width: 1044px;
-  }
-  #mwan_status_text {
-	display: table;
-	font-size: 14px;
-	margin: auto;
-	max-width: 1044px;
-	min-width: 246px;
-	width: 100%;
-  }
-  .wanon {
-	background-color: rgb(144, 240, 144);
-  }
-  .wanoff {
-	background-color: rgb(240, 144, 144);
-  }
-  .wanon, .wanoff {
-	border-radius: 60px;
-	box-shadow: 0px 2px 5px -3px;
-	float: left;
-	margin: 8px 3px 0px 3px;
-	min-height: 30px;
-	min-width: 235px;
-	padding: 5px 10px 8px 10px;
-	text-align: center;
-  }
-</style>
-- 
cgit v1.2.3


From 1ff26f380f5791d7ee08b7a98a62452c834bc81c Mon Sep 17 00:00:00 2001
From: Florian Eckert <fe@dev.tdt.de>
Date: Thu, 1 Mar 2018 08:52:06 +0100
Subject: luci-app-mwan3: remove css styling from status_interface page

Signed-off-by: Florian Eckert <fe@dev.tdt.de>
---
 .../luasrc/view/mwan/status_interface.htm          | 31 ----------------------
 1 file changed, 31 deletions(-)

diff --git a/applications/luci-app-mwan3/luasrc/view/mwan/status_interface.htm b/applications/luci-app-mwan3/luasrc/view/mwan/status_interface.htm
index 21e59a801..ae28df10e 100644
--- a/applications/luci-app-mwan3/luasrc/view/mwan/status_interface.htm
+++ b/applications/luci-app-mwan3/luasrc/view/mwan/status_interface.htm
@@ -77,35 +77,4 @@
 	</fieldset>
 </div>
 
-<style type="text/css">
-  #mwan_status_text {
-	display: table;
-	font-size: 14px;
-	margin: auto;
-	max-width: 1044px;
-	min-width: 246px;
-	width: 100%;
-  }
-  .wanon {
-	background-color: rgb(144, 240, 144);
-  }
-  .wanoff {
-	background-color: rgb(240, 144, 144);
-  }
-  .wanon, .wanoff {
-	border-radius: 60px;
-	box-shadow: 0px 2px 5px -3px;
-	float: left;
-	margin: 8px 3px 0px 3px;
-	min-height: 30px;
-	min-width: 235px;
-	padding: 5px 10px 8px 10px;
-	text-align: center;
-  }
-  #mwan_statuslog_text {
-	padding: 20px;
-	text-align: left;
-  }
-</style>
-
 <%+footer%>
-- 
cgit v1.2.3


From deba02e0968e2026334d16606ca24032ec055a76 Mon Sep 17 00:00:00 2001
From: Florian Eckert <fe@dev.tdt.de>
Date: Thu, 1 Mar 2018 08:53:17 +0100
Subject: luci-app-mwan3: fix source styling

Signed-off-by: Florian Eckert <fe@dev.tdt.de>
---
 .../luci-app-mwan3/luasrc/view/mwan/overview_status_interface.htm     | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/applications/luci-app-mwan3/luasrc/view/mwan/overview_status_interface.htm b/applications/luci-app-mwan3/luasrc/view/mwan/overview_status_interface.htm
index bb7aaca7b..d95cc8406 100644
--- a/applications/luci-app-mwan3/luasrc/view/mwan/overview_status_interface.htm
+++ b/applications/luci-app-mwan3/luasrc/view/mwan/overview_status_interface.htm
@@ -54,5 +54,7 @@ XHR.poll(5, '<%=luci.dispatcher.build_url("admin", "status", "mwan", "interface_
 
 <fieldset id="interface_field" class="cbi-section">
 	<legend><%:MWAN Interface Live Status%></legend>
-	<div id="mwan_status_text"><img src="<%=resource%>/icons/loading.gif" alt="<%:Loading%>" style="vertical-align:middle" /><%:Collecting data...%></div>
+	<div id="mwan_status_text">
+		<img src="<%=resource%>/icons/loading.gif" alt="<%:Loading%>" style="vertical-align:middle" /><%:Collecting data...%>
+	</div>
 </fieldset>
-- 
cgit v1.2.3


From a73687022d222e61573a11b6b43bc76d246a86c3 Mon Sep 17 00:00:00 2001
From: Florian Eckert <fe@dev.tdt.de>
Date: Thu, 1 Mar 2018 08:56:43 +0100
Subject: luci-app-mwan3: remove duplicated code section

Signed-off-by: Florian Eckert <fe@dev.tdt.de>
---
 .../luasrc/view/mwan/status_interface.htm          | 62 +---------------------
 1 file changed, 1 insertion(+), 61 deletions(-)

diff --git a/applications/luci-app-mwan3/luasrc/view/mwan/status_interface.htm b/applications/luci-app-mwan3/luasrc/view/mwan/status_interface.htm
index ae28df10e..cb476967f 100644
--- a/applications/luci-app-mwan3/luasrc/view/mwan/status_interface.htm
+++ b/applications/luci-app-mwan3/luasrc/view/mwan/status_interface.htm
@@ -14,67 +14,7 @@
 </ul>
 
 <script type="text/javascript" src="<%=resource%>/cbi.js"></script>
-<script type="text/javascript">//<![CDATA[
-	XHR.poll(5, '<%=luci.dispatcher.build_url("admin", "status", "mwan", "interface_status")%>', null,
-		function(x, status)
-		{
-			var legend = document.getElementById('diag-rc-legend');
-			var statusDiv = document.getElementById('diag-rc-output');
-			legend.style.display = 'none';
-			if (status.interfaces)
-			{
-				var statusview = '';
-				for ( var iface in status.interfaces)
-				{
-					var state = '';
-					var css = '';
-					switch (status.interfaces[iface].status)
-					{
-						case 'online':
-							state = '<%:Online (tracking active)%>';
-							css = 'wanon';
-							break;
-						case 'notMonitored':
-							state = '<%:Online (tracking off)%>';
-							css = 'wanon';
-							break;
-						case 'offline':
-							state = '<%:Offline%>';
-							css = 'wanoff';
-							break;
-						default:
-							state = '<%:Disabled%>';
-							css = 'wanoff';
-							break;
-					}
-					statusview += String.format(
-						'<span class="%s"><strong>%s</strong><br />%s</span>',
-						css,
-						iface,
-						state
-					);
-				}
-				statusDiv.innerHTML = statusview;
-			}
-			else
-			{
-				statusDiv.innerHTML = '<strong><%:No MWAN interfaces found%></strong>';
-			}
-		}
-	);
-//]]></script>
 
-<div class="cbi-map">
-	<h2 name="content"><%:MWAN Status - Interface%></h2>
-	<%if not require("luci.sys").init.enabled("mwan3") then%>
-	<div><strong><%:INFO: MWAN not running%></strong></div>
-	<%end%>
-	<fieldset class="cbi-section">
-		<legend id="diag-rc-legend"><%:Collecting data...%></legend>
-		<span id="diag-rc-output">
-			<img src="<%=resource%>/icons/loading.gif" alt="<%:Loading%>" style="vertical-align: middle;" />
-		</span>
-	</fieldset>
-</div>
+<%+mwan/overview_status_interface%>
 
 <%+footer%>
-- 
cgit v1.2.3


From 5362f6970650d984b0e35d10232247f07d81892b Mon Sep 17 00:00:00 2001
From: Florian Eckert <fe@dev.tdt.de>
Date: Thu, 1 Mar 2018 08:58:36 +0100
Subject: luci-app-mwan3: Update header name in status pages

Signed-off-by: Florian Eckert <fe@dev.tdt.de>
---
 .../luci-app-mwan3/luasrc/view/mwan/overview_status_interface.htm       | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/applications/luci-app-mwan3/luasrc/view/mwan/overview_status_interface.htm b/applications/luci-app-mwan3/luasrc/view/mwan/overview_status_interface.htm
index d95cc8406..6df3b3f28 100644
--- a/applications/luci-app-mwan3/luasrc/view/mwan/overview_status_interface.htm
+++ b/applications/luci-app-mwan3/luasrc/view/mwan/overview_status_interface.htm
@@ -53,7 +53,7 @@ XHR.poll(5, '<%=luci.dispatcher.build_url("admin", "status", "mwan", "interface_
 //]]></script>
 
 <fieldset id="interface_field" class="cbi-section">
-	<legend><%:MWAN Interface Live Status%></legend>
+	<legend><%:MWAN Interfaces%></legend>
 	<div id="mwan_status_text">
 		<img src="<%=resource%>/icons/loading.gif" alt="<%:Loading%>" style="vertical-align:middle" /><%:Collecting data...%>
 	</div>
-- 
cgit v1.2.3


From dcb041e3965d6cb706094720975267a9d61f74c0 Mon Sep 17 00:00:00 2001
From: Florian Eckert <fe@dev.tdt.de>
Date: Thu, 1 Mar 2018 10:36:03 +0100
Subject: luci-app-mwan3: remove css from javascript in
 overview_status_interface

Signed-off-by: Florian Eckert <fe@dev.tdt.de>
---
 .../luasrc/view/mwan/overview_status_interface.htm          | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/applications/luci-app-mwan3/luasrc/view/mwan/overview_status_interface.htm b/applications/luci-app-mwan3/luasrc/view/mwan/overview_status_interface.htm
index 6df3b3f28..86b5ac696 100644
--- a/applications/luci-app-mwan3/luasrc/view/mwan/overview_status_interface.htm
+++ b/applications/luci-app-mwan3/luasrc/view/mwan/overview_status_interface.htm
@@ -15,30 +15,27 @@ XHR.poll(5, '<%=luci.dispatcher.build_url("admin", "status", "mwan", "interface_
 				for ( var iface in status.interfaces)
 				{
 					var state = '';
-					var css = '';
 					switch (status.interfaces[iface].status)
 					{
 						case 'online':
 							state = '<%:Online (tracking active)%>';
-							css = 'wanon';
 							break;
 						case 'notMonitored':
 							state = '<%:Online (tracking off)%>';
-							css = 'wanon';
 							break;
 						case 'offline':
 							state = '<%:Offline%>';
-							css = 'wanoff';
 							break;
 						default:
 							state = '<%:Disabled%>';
-							css = 'wanoff';
 							break;
 					}
 					statusview += String.format(
-						'<span class="%s"><strong>%s</strong><br />%s</span>',
-						css,
-						iface,
+						'<div><strong>Interface: </strong>%s</div>',
+						iface
+					);
+					statusview += String.format(
+						'<div><strong>Status: </strong>%s</div></br></br>',
 						state
 					);
 				}
-- 
cgit v1.2.3