Bug fixes (#230)

* Added undefined check on status bar removal
* Navigating to base url no longer breaks check status functionality on subscription page
* There is no longer an empty message when switching to an offline backend while in the information page.

Author: Christoffer-Cortes

src/main/resources/static/js/errorMessages.js

@@ -69,10 +69,12 @@ ko.applyBindings(vm, $("#alertsItem")[0]);
 vm.stopPropagation();
 
 function logMessage(message) {
-    $.jGrowl(message, { sticky: false, theme: 'Error', position: 'center' });
-    vm.addErrorMessage(message);
-    vm.storeErrorMessage(message);
-    vm.stopPropagation();
+    if (message != undefined && message != "") {
+        $.jGrowl(message, { sticky: false, theme: 'Error', position: 'center' });
+        vm.addErrorMessage(message);
+        vm.storeErrorMessage(message);
+        vm.stopPropagation();
+    }
 }
 
 function parseAndLogMessage(message) {

src/main/resources/static/js/status-handler.js

@@ -9,7 +9,8 @@ and reload subscriptions in subscription list depending on the back-end status.
 // Start ## Status Indicator ##
 
 function checkBackendStatus() {
-    const currentUrl = router._lastRouteResolved.url;
+    var currentUrl = router._lastRouteResolved.url;
+    currentUrl = currentUrl.replace(/^\//, '');
     var isWhitelistedPage = !stringContainsSubstring(getWhiteListedPages(), currentUrl);
     if (isWhitelistedPage) {
         return;
@@ -34,7 +35,6 @@ function routeSpecificFunctions(currentUrl) {
     if (currentUrl === "subscriptions") {
         reDrawTable(isStatusChanged);
         updateConnectionButtonsStatus();
-
     }
 
     if (currentUrl !== "subscriptions") {
@@ -115,7 +115,10 @@ function addStatusIndicator(statusType, statusText) {
 }
 
 function removeStatusIndicator() {
-    $($(".content")[0].previousElementSibling).remove();
+    var parentContent = $(".content")[0];
+    if (parentContent != undefined) {
+        $(parentContent.previousElementSibling).remove();
+    }
 }
 // End ## Status Indicator ##