Fix catalog entity context menu duplicate entries (#3035)

Signed-off-by: Jari Kolehmainen <jari.kolehmainen@gmail.com>

src/renderer/components/+catalog/catalog.tsx

@@ -79,7 +79,7 @@ export class Catalog extends React.Component<Props> {
 
   async componentDidMount() {
     this.contextMenu = {
-      menuItems: [],
+      menuItems: observable.array([]),
       navigate: (url: string) => navigate(url)
     };
     this.catalogEntityStore = new CatalogEntityStore();
@@ -169,8 +169,14 @@ export class Catalog extends React.Component<Props> {
   }
 
   renderItemMenu = (item: CatalogEntityItem) => {
+    const onOpen = () => {
+      this.contextMenu.menuItems = [];
+
+      item.onContextMenuOpen(this.contextMenu);
+    };
+
     return (
-      <MenuActions onOpen={() => item.onContextMenuOpen(this.contextMenu)}>
+      <MenuActions onOpen={onOpen}>
         {
           this.contextMenu.menuItems.map((menuItem, index) => (
             <MenuItem key={index} onClick={() => this.onMenuItemClick(menuItem)}>

src/renderer/components/hotbar/hotbar-entity-icon.tsx

@@ -23,7 +23,7 @@ import React, { DOMAttributes } from "react";
 import { makeObservable, observable } from "mobx";
 import { observer } from "mobx-react";
 
-import type { CatalogEntity, CatalogEntityContextMenuContext } from "../../../common/catalog";
+import type { CatalogEntity, CatalogEntityContextMenu, CatalogEntityContextMenuContext } from "../../../common/catalog";
 import { catalogCategoryRegistry } from "../../api/catalog-category-registry";
 import { catalogEntityRegistry } from "../../api/catalog-entity-registry";
 import { navigate } from "../../navigation";
@@ -101,12 +101,10 @@ export class HotbarEntityIcon extends React.Component<Props> {
       active: this.isActive(entity),
       disabled: !entity
     });
-    const onOpen = async () => {
+
-      await entity.onContextMenuOpen(this.contextMenu);
-    };
-    const isActive = this.isActive(entity);
     const isPersisted = this.isPersisted(entity);
-    const menuItems = this.contextMenu?.menuItems ?? [];
+    const onOpen = async () => {
+      const menuItems: CatalogEntityContextMenu[] = [];
 
       if (!isPersisted) {
         menuItems.unshift({
@@ -120,6 +118,12 @@ export class HotbarEntityIcon extends React.Component<Props> {
         });
       }
 
+      this.contextMenu.menuItems = menuItems;
+
+      await entity.onContextMenuOpen(this.contextMenu);
+    };
+    const isActive = this.isActive(entity);
+
     return (
       <HotbarIcon
         uid={entity.metadata.uid}
@@ -128,7 +132,7 @@ export class HotbarEntityIcon extends React.Component<Props> {
         className={className}
         active={isActive}
         onMenuOpen={onOpen}
-        menuItems={menuItems}
+        menuItems={this.contextMenu.menuItems}
         {...elemProps}
       >
         { this.ledIcon }