Use the `null` value to indicate a missing identifier

diff --git a/ts/WoltLabSuite/Core/Acp/Ui/Page/Menu/Main/Backend.ts b/ts/WoltLabSuite/Core/Acp/Ui/Page/Menu/Main/Backend.ts
index d7268e4964063c2987cdab650e88287e01259a09..bb010448ab38def1316b0a6a451a82e0dbfb3e12 100644 (file)
--- a/ts/WoltLabSuite/Core/Acp/Ui/Page/Menu/Main/Backend.ts
+++ b/ts/WoltLabSuite/Core/Acp/Ui/Page/Menu/Main/Backend.ts
@@ -20,7 +20,7 @@ function getSubMenuItems(subMenu: HTMLElement, menuItem: string): MenuItem[] {
       children,
       counter: 0,
       depth: 1,
-      identifier: false,
+      identifier: null,
       title,
     };
   });
@@ -40,7 +40,7 @@ function getMenuItems(category: HTMLOListElement): MenuItem[] {
       children,
       counter: 0,
       depth: 2,
-      identifier: false,
+      identifier: null,
       link: link.href,
       title: link.textContent!,
     };
@@ -59,7 +59,7 @@ function getMenuItemActions(link: HTMLAnchorElement): MenuItem[] {
       children: [],
       counter: 0,
       depth: 2,
-      identifier: false,
+      identifier: null,
       link: action.href,
       title: action.dataset.tooltip || action.title,
     };
@@ -82,7 +82,7 @@ export class AcpUiPageMenuMainBackend implements PageMenuMainProvider {
           children,
           counter: 0,
           depth: 0,
-          identifier: false,
+          identifier: null,
           title,
         };
       },

diff --git a/ts/WoltLabSuite/Core/Ui/Page/Menu/Main.ts b/ts/WoltLabSuite/Core/Ui/Page/Menu/Main.ts
index d803b10fbdb1763bb77a99c7b01d611a4680dad3..91178d4e975e58f3852c4806145d4abd6682b0f8 100644 (file)
--- a/ts/WoltLabSuite/Core/Ui/Page/Menu/Main.ts
+++ b/ts/WoltLabSuite/Core/Ui/Page/Menu/Main.ts
@@ -221,7 +221,7 @@ export class PageMenuMain implements PageMenuProvider {
         counter.setAttribute("aria-hidden", "true");
         counter.textContent = menuItem.counter.toString();
 
-        if (menuItem.identifier !== false) {
+        if (menuItem.identifier !== null) {
           this.menuItemBadges.set(menuItem.identifier, counter);
         }
 
@@ -325,7 +325,7 @@ export class PageMenuMain implements PageMenuProvider {
   }
 
   private refreshUnreadBage(menuItem: MenuItem): void {
-    if (menuItem.identifier !== false) {
+    if (menuItem.identifier !== null) {
       const counter = this.menuItemBadges.get(menuItem.identifier);
       if (counter) {
         if (menuItem.counter === 0) {

diff --git a/ts/WoltLabSuite/Core/Ui/Page/Menu/Main/Provider.ts b/ts/WoltLabSuite/Core/Ui/Page/Menu/Main/Provider.ts
index 96b153515889e5db1f5f2130b539f220dee2eb6d..bf2954222519efc8a9e6bac844db97574fa24005 100644 (file)
--- a/ts/WoltLabSuite/Core/Ui/Page/Menu/Main/Provider.ts
+++ b/ts/WoltLabSuite/Core/Ui/Page/Menu/Main/Provider.ts
@@ -16,7 +16,7 @@ export type MenuItem = {
   children: MenuItem[];
   counter: number;
   depth: MenuItemDepth;
-  identifier: string | false;
+  identifier: string | null;
   link?: string;
   title: string;
 };

diff --git a/wcfsetup/install/files/js/WoltLabSuite/Core/Acp/Ui/Page/Menu/Main/Backend.js b/wcfsetup/install/files/js/WoltLabSuite/Core/Acp/Ui/Page/Menu/Main/Backend.js
index 1fcc63ac03faf113c462d6db069b3d5d44fc62cc..4536af7e5f927741a33dfba17caf0565cfc64018 100644 (file)
--- a/wcfsetup/install/files/js/WoltLabSuite/Core/Acp/Ui/Page/Menu/Main/Backend.js
+++ b/wcfsetup/install/files/js/WoltLabSuite/Core/Acp/Ui/Page/Menu/Main/Backend.js
@@ -20,7 +20,7 @@ define(["require", "exports"], function (require, exports) {
                 children,
                 counter: 0,
                 depth: 1,
-                identifier: false,
+                identifier: null,
                 title,
             };
         });
@@ -37,7 +37,7 @@ define(["require", "exports"], function (require, exports) {
                 children,
                 counter: 0,
                 depth: 2,
-                identifier: false,
+                identifier: null,
                 link: link.href,
                 title: link.textContent,
             };
@@ -54,7 +54,7 @@ define(["require", "exports"], function (require, exports) {
                 children: [],
                 counter: 0,
                 depth: 2,
-                identifier: false,
+                identifier: null,
                 link: action.href,
                 title: action.dataset.tooltip || action.title,
             };
@@ -73,7 +73,7 @@ define(["require", "exports"], function (require, exports) {
                     children,
                     counter: 0,
                     depth: 0,
-                    identifier: false,
+                    identifier: null,
                     title,
                 };
             });

diff --git a/wcfsetup/install/files/js/WoltLabSuite/Core/Ui/Page/Menu/Main.js b/wcfsetup/install/files/js/WoltLabSuite/Core/Ui/Page/Menu/Main.js
index a1d2ebfc0ed88d1013b110f5230f2d3e4d52a014..a803706fece850cfb409a55abe3a5d9f9055661c 100644 (file)
--- a/wcfsetup/install/files/js/WoltLabSuite/Core/Ui/Page/Menu/Main.js
+++ b/wcfsetup/install/files/js/WoltLabSuite/Core/Ui/Page/Menu/Main.js
@@ -168,7 +168,7 @@ define(["require", "exports", "tslib", "./Container", "../../../Language", "../.
                     counter.classList.add("pageMenuMainItemCounter", "badge", "badgeUpdate");
                     counter.setAttribute("aria-hidden", "true");
                     counter.textContent = menuItem.counter.toString();
-                    if (menuItem.identifier !== false) {
+                    if (menuItem.identifier !== null) {
                         this.menuItemBadges.set(menuItem.identifier, counter);
                     }
                     link.append(counter);
@@ -253,7 +253,7 @@ define(["require", "exports", "tslib", "./Container", "../../../Language", "../.
             menuItems.forEach((menuItem) => this.refreshUnreadBage(menuItem));
         }
         refreshUnreadBage(menuItem) {
-            if (menuItem.identifier !== false) {
+            if (menuItem.identifier !== null) {
                 const counter = this.menuItemBadges.get(menuItem.identifier);
                 if (counter) {
                     if (menuItem.counter === 0) {