From b995063b43d676ec7bc5c7fc0b3c6dccf87a4557 Mon Sep 17 00:00:00 2001
From: Ziver Koc <ziver@koc.se>
Date: Thu, 28 Jul 2016 23:30:03 +0200
Subject: [PATCH] Progress on ConfigPage

---
 resources/WebContent/page/ConfigPage.tmpl     | 18 +++++++-------
 resources/WebContent/page/index.tmpl          |  4 ++--
 src/wa/server/WAContext.java                  | 17 +++++++++----
 src/wa/server/page/ConfigPage.java            | 24 ++++++++++---------
 src/wa/server/page/ServiceStatusPage.java     |  3 +--
 src/wa/server/page/WAPage.java                |  4 +++-
 src/wa/server/plugin/WAConfiguration.java     | 12 ++++++++--
 .../plugin/apache/ApacheConfigProxy.java      |  2 +-
 .../apache/ApacheConfigVirtualHost.java       |  2 +-
 src/wa/server/plugin/apache/plugin.json       |  3 ++-
 src/wa/server/plugin/hwstatus/HDDStatus.java  |  4 ++--
 src/wa/server/plugin/hwstatus/NetStatus.java  |  4 ++--
 src/wa/server/plugin/hwstatus/NetStatus.tmpl  |  2 +-
 src/wa/server/plugin/nutups/UPSStatus.java    |  2 +-
 14 files changed, 59 insertions(+), 42 deletions(-)

diff --git a/resources/WebContent/page/ConfigPage.tmpl b/resources/WebContent/page/ConfigPage.tmpl
index 1fafc26..182a040 100755
--- a/resources/WebContent/page/ConfigPage.tmpl
+++ b/resources/WebContent/page/ConfigPage.tmpl
@@ -1,10 +1,15 @@
 <div class="col-md-12">
     <div class="panel panel-default">
-        <div class="panel-heading">Service Status</div>
+        <div class="panel-heading">{{name}}</div>
         <div class="panel-body">
-            <table class="table table-hover">
+            <table class="table table-hover table-condensed">
                 <thead><tr>
-                    {{#params}}<th>{{.getNiceName()}}</th>{{/params}}
+                    {{#params}}
+                    <th>{{.getNiceName()}}</th>
+                    {{/params}}
+                    <th>
+                        <button type="button" class="btn btn-default pull-left" data-toggle="modal" data-target="#configureModal">+</button>
+                    </th>
                 </tr></thead>
                 {{#data}}
                 <tr {{#.disabled}}class="active"{{/.disabled}}>
@@ -39,7 +44,6 @@
             </table>
         </div>
         <div class="panel-footer clearfix">
-            <button type="button" class="btn btn-default pull-left" data-toggle="modal" data-target="#configureModal">+</button>
             <button type="button" class="btn btn-primary pull-right">Apply Configuration</button>
         </div>
     </div>
@@ -68,12 +72,6 @@
                             {{#.isTypeBoolean()}}
                             <input type="checkbox" class="form-control" id="{{.getName()}}">
                             {{/.isTypeBoolean()}}
-                            <!--
-                            <select class="form-control" id="sselect" placeholder="">
-                                <option value="volvo">-- Chose an Option --</option>
-                                <option value="saab">Saab</option>
-                            </select>
-                            -->
                         </div>
                     </div>
                     {{/params}}
diff --git a/resources/WebContent/page/index.tmpl b/resources/WebContent/page/index.tmpl
index 11a3886..d9cbc60 100755
--- a/resources/WebContent/page/index.tmpl
+++ b/resources/WebContent/page/index.tmpl
@@ -106,9 +106,9 @@
 				<a data-toggle="collapse" data-parent="#side-bar" href="#{{.getName()}}_collapse">
 					{{.getName()}} <b class="caret"></b>
 				</a>
-				<ul id="{{.name}}_collapse" class="side-sub-menu collapse nav nav-pills nav-stacked ">
+				<ul id="{{.getName()}}_collapse" class="side-sub-menu collapse nav nav-pills nav-stacked ">
 					{{#.getSubNavs()}}
-					<li><a href="{{.url}}">{{.getName()}}</a></li>
+					<li><a href="{{.getURL()}}">{{.getName()}}</a></li>
 					{{/.getSubNavs()}}
 				</ul>
 			</li>
diff --git a/src/wa/server/WAContext.java b/src/wa/server/WAContext.java
index a999ee2..f1d6684 100755
--- a/src/wa/server/WAContext.java
+++ b/src/wa/server/WAContext.java
@@ -36,6 +36,7 @@ import zutil.plugin.PluginManager;
 import zutil.ui.Navigation;
 
 import java.io.File;
+import java.nio.file.NoSuchFileException;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Map;
@@ -85,13 +86,11 @@ public class WAContext {
 
     protected static void initialize(){
         try {
-            pluginManager = new PluginManager();
-            rootNav = Navigation.createRootNav();
-
-
             // Setup DB
             File dbFile = WAConstants.getConfigFile(WAConstants.DB_FILE);
             File defDbFile = WAConstants.getConfigFile(WAConstants.DB_DEFAULT_FILE);
+            if (!defDbFile.exists())
+                throw new NoSuchFileException("Deafult DB missing: "+defDbFile.getAbsolutePath());
             db = new DBConnection(DBConnection.DBMS.SQLite, dbFile.getAbsolutePath());
             DBConnection defaultDB = new DBConnection(DBConnection.DBMS.SQLite, defDbFile.getAbsolutePath());
             int defaultDBVersion = Integer.parseInt(defaultDB.exec(
@@ -116,6 +115,11 @@ public class WAContext {
                 handler.upgrade();
             }
 
+            // Setup fields
+            pluginManager = new PluginManager();
+            rootNav = Navigation.createRootNav();
+
+            // Http Server setup
             httpServer = new HttpServer(80);
             httpServer.setDefaultPage(new HttpFilePage(FileUtil.find("WebContent/")));
             httpServer.start();
@@ -132,7 +136,10 @@ public class WAContext {
         return httpServer;
     }
     public static void registerWaPage(WAPage page){
-        httpServer.setPage(page.getPageName(), page);
+        if (page.getPageName() != null) {
+            logger.info("Registering new WAPage: "+ page.getPageName());
+            httpServer.setPage(page.getPageName(), page);
+        }
     }
     public static Navigation getRootNav(){
         return rootNav;
diff --git a/src/wa/server/page/ConfigPage.java b/src/wa/server/page/ConfigPage.java
index ead5211..c827580 100755
--- a/src/wa/server/page/ConfigPage.java
+++ b/src/wa/server/page/ConfigPage.java
@@ -32,7 +32,6 @@ import zutil.parser.Templator;
 import zutil.ui.Configurator;
 
 import java.io.IOException;
-import java.util.ArrayList;
 import java.util.Iterator;
 import java.util.List;
 import java.util.Map;
@@ -51,22 +50,24 @@ public class ConfigPage extends WAPage {
 
 
     private Class<? extends WAConfiguration> configClass;
+    private String name;
 
 
     public ConfigPage(Class<? extends WAConfiguration> configClass) {
         super(getPageName(configClass));
         this.configClass = configClass;
+
+        WAConfiguration.WAConfig params = configClass.getAnnotation(WAConfiguration.WAConfig.class);
+        WAContext.getRootNav()
+                .createSubNav(WAServicePage.NAVIGATION_NAME)
+                .createSubNav(params.serviceName())
+                .createSubNav(this.getPageName(), params.name());
+        this.name = params.name();
+
     }
     private static String getPageName(Class<? extends WAConfiguration> configClass){
         WAConfiguration.WAConfig params = configClass.getAnnotation(WAConfiguration.WAConfig.class);
-        if (params != null) {
-            WAContext.getRootNav()
-                    .createSubNav(WAServicePage.PAGE_NAME)
-                    .createSubNav(params.service())
-                    .createSubNav(params.id(), params.name());
-            return WAServicePage.PAGE_NAME +"/"+ params.service() +"/"+ params.id();
-        }
-        return null;
+        return WAServicePage.PAGE_NAME +"/"+ params.servicePage() +"/"+ params.pageName();
     }
 
 
@@ -84,8 +85,8 @@ public class ConfigPage extends WAPage {
             // Actions
             if (request.containsKey("action")) {
                 int id = -1;
-                if (request.containsKey("id"))
-                    id = Integer.parseInt(request.get("id"));
+                if (request.containsKey("pageName"))
+                    id = Integer.parseInt(request.get("pageName"));
                 WAConfiguration target = getConfig(id);
                     switch (request.get("action")) {
                         case "create":
@@ -109,6 +110,7 @@ public class ConfigPage extends WAPage {
 
             // Prepare Output
             Templator tmpl = new Templator(FileUtil.find(TEMPLATE));
+            tmpl.set("name", name);
             tmpl.set("params", Configurator.getConfiguration(configClass));
             tmpl.set("data", objList);
             return tmpl;
diff --git a/src/wa/server/page/ServiceStatusPage.java b/src/wa/server/page/ServiceStatusPage.java
index f5923be..defaecf 100755
--- a/src/wa/server/page/ServiceStatusPage.java
+++ b/src/wa/server/page/ServiceStatusPage.java
@@ -52,14 +52,13 @@ public class ServiceStatusPage extends WAPage {
 
 
     public ServiceStatusPage(){
-        super(WAServicePage.NAVIGATION_NAME +"/"+ PAGE_NAME);
+        super(WAServicePage.PAGE_NAME);
         this.services = WAContext.getPluginManager().toArray(WAServiceStatus.class);
 
         Navigation nav = WAContext.getRootNav().createSubNav(WAServicePage.NAVIGATION_NAME)
                 .createSubNav(getPageName(), NAVIGATION_NAME).setWeight(-100);
     }
     public ServiceStatusPage(WAServiceStatus ss){
-        super(WAServicePage.NAVIGATION_NAME);
         services = new ArrayList<>();
         services.add(ss);
     }
diff --git a/src/wa/server/page/WAPage.java b/src/wa/server/page/WAPage.java
index 0a104ff..a174c0a 100755
--- a/src/wa/server/page/WAPage.java
+++ b/src/wa/server/page/WAPage.java
@@ -50,7 +50,9 @@ public abstract class WAPage implements HttpPage{
     private String pageName;
     private Templator tmpl;
 
-
+    public WAPage() {
+        this(null);
+    }
     public WAPage(String pageName) {
         this.pageName = pageName;
         try {
diff --git a/src/wa/server/plugin/WAConfiguration.java b/src/wa/server/plugin/WAConfiguration.java
index 325e1f8..e4f7ffa 100755
--- a/src/wa/server/plugin/WAConfiguration.java
+++ b/src/wa/server/plugin/WAConfiguration.java
@@ -1,15 +1,23 @@
 package wa.server.plugin;
 
 
+import java.lang.annotation.ElementType;
+import java.lang.annotation.Retention;
+import java.lang.annotation.RetentionPolicy;
+import java.lang.annotation.Target;
+
 /**
  * Created by Ziver on 2016-07-27.
  */
 public abstract class WAConfiguration{
 
+    @Retention(RetentionPolicy.RUNTIME)
+    @Target(ElementType.TYPE)
     public @interface WAConfig{
-        String id();
+        String pageName();
         String name();
-        String service();
+        String servicePage(); // TODO: this is horrible design!
+        String serviceName(); // TODO: this is horrible design!
     }
 
 
diff --git a/src/wa/server/plugin/apache/ApacheConfigProxy.java b/src/wa/server/plugin/apache/ApacheConfigProxy.java
index 01d9d7f..4c6d722 100755
--- a/src/wa/server/plugin/apache/ApacheConfigProxy.java
+++ b/src/wa/server/plugin/apache/ApacheConfigProxy.java
@@ -6,7 +6,7 @@ import zutil.ui.Configurator;
 import java.io.PrintStream;
 
 
-@WAConfiguration.WAConfig(service=ApacheService.PAGE_NAME, id="proxy", name="Proxy")
+@WAConfiguration.WAConfig(servicePage=ApacheService.SERVICE_NAME, serviceName=ApacheService.NAVIGATION_NAME, pageName ="proxy", name="Proxy")
 public class ApacheConfigProxy extends ApacheAbstractConfig {
 
     @Configurator.Configurable("Proxy Target")
diff --git a/src/wa/server/plugin/apache/ApacheConfigVirtualHost.java b/src/wa/server/plugin/apache/ApacheConfigVirtualHost.java
index aadd9ee..b07a0bd 100755
--- a/src/wa/server/plugin/apache/ApacheConfigVirtualHost.java
+++ b/src/wa/server/plugin/apache/ApacheConfigVirtualHost.java
@@ -6,7 +6,7 @@ import zutil.ui.Configurator;
 import java.io.PrintStream;
 
 
-@WAConfiguration.WAConfig(service=ApacheService.PAGE_NAME, id="vhost", name="VirtualHost")
+@WAConfiguration.WAConfig(servicePage=ApacheService.SERVICE_NAME, serviceName=ApacheService.NAVIGATION_NAME, pageName ="vhost", name="VirtualHost")
 public class ApacheConfigVirtualHost extends ApacheAbstractConfig {
 
     @Configurator.Configurable("Document Root")
diff --git a/src/wa/server/plugin/apache/plugin.json b/src/wa/server/plugin/apache/plugin.json
index 13e6a63..800cddc 100755
--- a/src/wa/server/plugin/apache/plugin.json
+++ b/src/wa/server/plugin/apache/plugin.json
@@ -4,6 +4,7 @@
 	"interfaces": [
 		{"wa.server.page.WAPage": "wa.server.plugin.apache.ApacheService"},
 		{"wa.server.plugin.WAServiceStatus": "wa.server.plugin.apache.ApacheStatus"},
-		{"wa.server.page.WAPage": "wa.server.plugin.apache.ApacheConfigVirtualHost"}
+		{"wa.server.plugin.WAConfiguration": "wa.server.plugin.apache.ApacheConfigProxy"},
+		{"wa.server.plugin.WAConfiguration": "wa.server.plugin.apache.ApacheConfigVirtualHost"}
 	]
 }
\ No newline at end of file
diff --git a/src/wa/server/plugin/hwstatus/HDDStatus.java b/src/wa/server/plugin/hwstatus/HDDStatus.java
index 83f3ba5..f91139b 100755
--- a/src/wa/server/plugin/hwstatus/HDDStatus.java
+++ b/src/wa/server/plugin/hwstatus/HDDStatus.java
@@ -84,10 +84,10 @@ public class HDDStatus extends WAStatusPage {
                     node.set("mount", hdd.getDirName());
 
                     if (idMap.containsKey(device))
-                        node.set("id", idMap.get(device));
+                        node.set("pageName", idMap.get(device));
                     else {
                         idMap.put(device, nextId);
-                        node.set("id", nextId++);
+                        node.set("pageName", nextId++);
                     }
 
                     FileSystemUsage hdd_use = sigar.getFileSystemUsage(hdd.getDirName());
diff --git a/src/wa/server/plugin/hwstatus/NetStatus.java b/src/wa/server/plugin/hwstatus/NetStatus.java
index ec322f7..4e41770 100755
--- a/src/wa/server/plugin/hwstatus/NetStatus.java
+++ b/src/wa/server/plugin/hwstatus/NetStatus.java
@@ -76,10 +76,10 @@ public class NetStatus extends WAStatusPage {
                 for( String intfName : intfNameList ){
                     DataNode intfNode = new DataNode(DataNode.DataType.Map);
                     if(idMap.containsKey(intfName))
-                        intfNode.set("id", idMap.get(intfName));
+                        intfNode.set("pageName", idMap.get(intfName));
                     else{
                         idMap.put(intfName, nextId);
-                        intfNode.set("id", nextId++);
+                        intfNode.set("pageName", nextId++);
                     }
 
                     NetInterfaceStat net_stat = sigar.getNetInterfaceStat(intfName);
diff --git a/src/wa/server/plugin/hwstatus/NetStatus.tmpl b/src/wa/server/plugin/hwstatus/NetStatus.tmpl
index fd8cfb1..0b53dd1 100755
--- a/src/wa/server/plugin/hwstatus/NetStatus.tmpl
+++ b/src/wa/server/plugin/hwstatus/NetStatus.tmpl
@@ -139,7 +139,7 @@ function updateRoutingTable(){
                     <tr><th>Flags</th><td class="net-flags"></td></tr>
                 </table>
             </div>
-            <!--<div><canvas id="hdd-io" height="400"></canvas></div>-->
+            <!--<div><canvas pageName="hdd-io" height="400"></canvas></div>-->
         </div>
     </div></div>
 </div>
\ No newline at end of file
diff --git a/src/wa/server/plugin/nutups/UPSStatus.java b/src/wa/server/plugin/nutups/UPSStatus.java
index 4815a1a..ad407dc 100755
--- a/src/wa/server/plugin/nutups/UPSStatus.java
+++ b/src/wa/server/plugin/nutups/UPSStatus.java
@@ -46,7 +46,7 @@ public class UPSStatus extends WAStatusPage {
             DataNode upsRoot = new DataNode(DataNode.DataType.List);
             for (NutUPSClient.UPSDevice ups : nutClient.getUPSList()){
                 DataNode upsNode = new DataNode(DataNode.DataType.Map);
-                upsNode.set("id", ups.getId());
+                upsNode.set("pageName", ups.getId());
                 upsNode.set("model", ups.getModelName());
                 upsNode.set("desc", ups.getDescription());
                 upsNode.set("charge", ups.getBatteryCharge());