add: logging and error handling (#4493)

eclipse-dirigible · Dec 4, 2024 · a10f9ae · a10f9ae
1 parent e1f9f3f
commit a10f9ae
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 4 deletions.
diff --git a/...src/main/resources/META-INF/dirigible/portal/api/NavigationExtension/NavigationService.js b/...src/main/resources/META-INF/dirigible/portal/api/NavigationExtension/NavigationService.js
@@ -15,7 +15,7 @@ for (let i = 0; i < navigationExtensions.length; i++) {
         const navigation = getNavigation();
         navigationList.push(navigation);
     } catch (err) {
-        console.error(err)
+        console.error(`Failed to load a navigation group in NavigationService: ${err}\npath: ${path}`);
     }
 }
 

diff --git a/...ources/META-INF/dirigible/portal/api/NavigationGroupsExtension/NavigationGroupsService.js b/...ources/META-INF/dirigible/portal/api/NavigationGroupsExtension/NavigationGroupsService.js
@@ -15,7 +15,7 @@ for (let i = 0; i < groupExtensions.length; i++) {
         const group = getGroup();
         groupList.push(group);
     } catch (err) {
-        console.error(err)
+        console.error(`Failed to load a widget in WidgetService: ${err}\npath: ${path}`);
     }
 }
 

diff --git a/...resources-portal/src/main/resources/META-INF/dirigible/portal/js/navigation-controller.js b/...resources-portal/src/main/resources/META-INF/dirigible/portal/js/navigation-controller.js
@@ -9,8 +9,17 @@ navigation.controller("LaunchpadViewController", ["$scope", "messageHub", "$http
     function loadNavigationGroups() {
         return $http.get("/services/js/portal/api/NavigationGroupsExtension/NavigationGroupsService.js")
             .then(function (response) {
+                for (itemData of response.data) {
+                    if (!itemData || !itemData.icon || !itemData.order || !itemData.label) {
+                        console.error(`Invalid navigation group data: ${JSON.stringify(itemData)}. Missing one of the properties: icon, order, label`);
+                        return;
+                    }
+                }
+
                 $scope.groups = response.data;
 
+                $scope.groups.sort((a, b) => a.order - b.order)
+
                 response.data.forEach(elem => {
                     $scope.groupItems[elem.label.toLowerCase()] = [];
                 });
@@ -41,7 +50,7 @@ navigation.controller("LaunchpadViewController", ["$scope", "messageHub", "$http
 
     function addNavigationItem(itemData) {
         if (!itemData || !itemData.label || !itemData.group || !itemData.order || !itemData.link) {
-            console.error('Invalid item data:', itemData);
+            console.error(`Invalid item data: ${JSON.stringify(itemData)} Missing one of the properties: label, group, order, link`);
             return;
         }
 
@@ -54,7 +63,8 @@ navigation.controller("LaunchpadViewController", ["$scope", "messageHub", "$http
         $scope.groupItems[groupKey].push({
             id: itemData.id,
             label: itemData.label,
-            link: itemData.link
+            link: itemData.link,
+            order: itemData.order
         });
     }