Refactor JSON-API and cleanups

assets/webconfig/i18n/en.json

@@ -186,6 +186,7 @@
     "conf_network_json_intro": "The JSON-RPC-Port of all Hyperion instances, used for remote control.",
     "conf_network_net_intro": "Network related settings which are applied to all network services.",
     "conf_network_proto_intro": "The PROTO-Port of all Hyperion instances, used for picture streams (HyperionScreenCap, Kodi Addon, Android Hyperion Grabber, ...)",
+    "conf_network_tok_idhead": "ID",
     "conf_network_tok_cidhead": "Description",
     "conf_network_tok_comment_title": "Token description",
     "conf_network_tok_desc": "Tokens grant other applications access to the Hyperion API, an application can request a token where you need to accept it or you create them on your own below. These tokens are just required when \"API Authorization\" is enabled in network settings.",
@@ -500,19 +501,16 @@
     "edt_conf_log_level_expl": "Depending on loglevel you see less or more messages in your log.",
     "edt_conf_log_level_title": "Log-Level",
     "edt_conf_net_apiAuth_expl": "Enforce all applications that use the Hyperion API to authenticate themself against Hyperion (Exception: see \"Local API Authentication\"). Higher security, as you control the access and revoke it at any time.",
-    "edt_conf_net_apiAuth_title": "API Authentication",
     "edt_conf_net_heading_title": "Network",
-    "edt_conf_net_internetAccessAPI_expl": "Allow access to the Hyperion API/Webinterface from the internet. Disable for higher security.",
+    "edt_conf_net_internetAccessAPI_expl": "Allow access to the Hyperion API/Web Interface from the Internet. Disable for increased security.",
     "edt_conf_net_internetAccessAPI_title": "Internet API Access",
-    "edt_conf_net_ipWhitelist_expl": "You can whitelist IP addresses instead allowing all connections from internet to connect to the Hyperion API/Webinterface.",
-    "edt_conf_net_ipWhitelist_title": "Whitelisted IP's",
+    "edt_conf_net_ipWhitelist_expl": "Define whitelisted IP addresses from which API requests from the Internet are allowed. All other external connections will be denied.",
+    "edt_conf_net_ipWhitelist_title": "Whitelisted IP addresses",
     "edt_conf_net_ip_itemtitle": "IP",
-    "edt_conf_net_localAdminAuth_expl": "When enabled, administration access from your local network needs a password.",
-    "edt_conf_net_localAdminAuth_title": "Local Admin API Authentication",
-    "edt_conf_net_localApiAuth_expl": "When enabled, connections from your home network needs to authenticate themselves against Hyperion with a token.",
+    "edt_conf_net_localApiAuth_expl": "When disabled, API authorisation via password or token is not required for local connections. The exception is administrative commands.",
     "edt_conf_net_localApiAuth_title": "Local API Authentication",
-    "edt_conf_net_restirctedInternetAccessAPI_expl": "You can restrict the access to the API through the internet to certain IP's.",
-    "edt_conf_net_restirctedInternetAccessAPI_title": "Restrict to IP's",
+    "edt_conf_net_restirctedInternetAccessAPI_expl": "You can restrict API requests over the Internet to only those IP addresses on the whitelist.",
+    "edt_conf_net_restirctedInternetAccessAPI_title": "Restrict to IP addresses",
     "edt_conf_os_events_lockEnable_title": "Listen to lock events",
     "edt_conf_os_events_lockEnable_expl": "Listen to screen lock/unlock events",
     "edt_conf_os_events_suspendEnable_title": "Listen to suspend events",

assets/webconfig/js/content_index.js

@@ -73,26 +73,30 @@ $(document).ready(function () {
   //End language selection
 
   $(window.hyperion).on("cmd-authorize-tokenRequest cmd-authorize-getPendingTokenRequests", function (event) {
-    var val = event.response.info;
-    if (Array.isArray(event.response.info)) {
-      if (event.response.info.length == 0) {
-        return
-      }
-      val = event.response.info[0]
-      if (val.comment == '')
-        $('#modal_dialog').modal('hide');
-    }
 
-    showInfoDialog("grantToken", $.i18n('conf_network_tok_grantT'), $.i18n('conf_network_tok_grantMsg') + '<br><span style="font-weight:bold">App: ' + val.comment + '</span><br><span style="font-weight:bold">Code: ' + val.id + '</span>')
-    $("#tok_grant_acc").off().on('click', function () {
-      tokenList.push(val)
-      // forward event, in case we need to rebuild the list now
-      $(window.hyperion).trigger({ type: "build-token-list" });
-      requestHandleTokenRequest(val.id, true)
-    });
-    $("#tok_deny_acc").off().on('click', function () {
-      requestHandleTokenRequest(val.id, false)
-    });
+    if (event.response && event.response.info !== undefined) {
+      var val = event.response.info;
+
+      if (Array.isArray(event.response.info)) {
+        if (event.response.info.length == 0) {
+          return
+        }
+        val = event.response.info[0]
+        if (val.comment == '')
+          $('#modal_dialog').modal('hide');
+      }
+
+      showInfoDialog("grantToken", $.i18n('conf_network_tok_grantT'), $.i18n('conf_network_tok_grantMsg') + '<br><span style="font-weight:bold">App: ' + val.comment + '</span><br><span style="font-weight:bold">Code: ' + val.id + '</span>')
+      $("#tok_grant_acc").off().on('click', function () {
+        tokenList.push(val)
+        // forward event, in case we need to rebuild the list now
+        $(window.hyperion).trigger({ type: "build-token-list" });
+        requestHandleTokenRequest(val.id, true)
+      });
+      $("#tok_deny_acc").off().on('click', function () {
+        requestHandleTokenRequest(val.id, false)
+      });
+    }
   });
 
   $(window.hyperion).one("cmd-authorize-getTokenList", function (event) {

assets/webconfig/js/content_logging.js

@@ -3,10 +3,13 @@ var createdCont = false;
 var isScroll = true;
 
 performTranslation();
-requestLoggingStop();
 
 $(document).ready(function () {
 
+  window.addEventListener('hashchange', function(event) {
+    requestLoggingStop();
+  });
+
   requestLoggingStart();
 
   $('#conf_cont').append(createOptPanel('fa-reorder', $.i18n("edt_conf_log_heading_title"), 'editor_container', 'btn_submit'));
@@ -180,7 +183,7 @@ $(document).ready(function () {
 
     $(window.hyperion).on("cmd-logging-update", function (event) {
 
-      var messages = (event.response.result.messages);
+      var messages = (event.response.data.messages);
 
       if (messages.length != 0) {
         if (!createdCont) {

assets/webconfig/js/content_network.js

@@ -213,13 +213,13 @@ $(document).ready(function () {
     for (var key in tokenList) {
       var lastUse = (tokenList[key].last_use) ? tokenList[key].last_use : "-";
       var btn = '<button id="tok' + tokenList[key].id + '" type="button" class="btn btn-danger">' + $.i18n('general_btn_delete') + '</button>';
-      $('.tktbody').append(createTableRow([tokenList[key].comment, lastUse, btn], false, true));
+      $('.tktbody').append(createTableRow([tokenList[key].id, tokenList[key].comment, lastUse, btn], false, true));
       $('#tok' + tokenList[key].id).off().on('click', handleDeleteToken);
     }
   }
 
   createTable('tkthead', 'tktbody', 'tktable');
-  $('.tkthead').html(createTableRow([$.i18n('conf_network_tok_cidhead'), $.i18n('conf_network_tok_lastuse'), $.i18n('general_btn_delete')], true, true));
+  $('.tkthead').html(createTableRow([$.i18n('conf_network_tok_idhead'), $.i18n('conf_network_tok_cidhead'), $.i18n('conf_network_tok_lastuse'), $.i18n('general_btn_delete')], true, true));
   buildTokenList();
 
   function handleDeleteToken(e) {

assets/webconfig/js/ledsim.js

@@ -261,7 +261,7 @@ $(document).ready(function () {
       $("body").get(0).style.setProperty("--background-var", "none");
     }
     else {
-      printLedsToCanvas(event.response.result.leds)
+      printLedsToCanvas(event.response.data.leds)
       $("body").get(0).style.setProperty("--background-var", "url(" + ($('#leds_preview_canv')[0]).toDataURL("image/jpg") + ") no-repeat top left");
     }
   });
@@ -275,7 +275,7 @@ $(document).ready(function () {
       }
     }
     else {
-      var imageData = (event.response.result.image);
+      var imageData = (event.response.data.image);
 
       var image = new Image();
       image.onload = function () {