@sequent-org/moodboard 1.0.12 → 1.0.14

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sequent-org/moodboard",
-  "version": "1.0.12",
+  "version": "1.0.14",
   "type": "module",
   "description": "Interactive moodboard",
   "main": "./src/index.js",

package/src/moodboard/MoodBoard.js

@@ -346,6 +346,25 @@ export class MoodBoard {
         }
     }
     
+    /**
+     * Безопасное уничтожение объекта с проверкой наличия метода destroy
+     * @param {Object} obj - объект для уничтожения
+     * @param {string} name - имя объекта для логирования
+     */
+    _safeDestroy(obj, name) {
+        if (obj) {
+            try {
+                if (typeof obj.destroy === 'function') {
+                    obj.destroy();
+                } else {
+                    console.warn(`Объект ${name} не имеет метода destroy()`);
+                }
+            } catch (error) {
+                console.error(`Ошибка при уничтожении ${name}:`, error);
+            }
+        }
+    }
+
     /**
      * Очистка ресурсов
      */
@@ -359,58 +378,44 @@ export class MoodBoard {
         // Устанавливаем флаг уничтожения
         this.destroyed = true;
         
-        // Уничтожаем UI компоненты
-        if (this.toolbar) {
-            this.toolbar.destroy();
-            this.toolbar = null;
-        }
+        // Уничтожаем UI компоненты с безопасными проверками
+        this._safeDestroy(this.toolbar, 'toolbar');
+        this.toolbar = null;
         
-        if (this.saveStatus) {
-            this.saveStatus.destroy();
-            this.saveStatus = null;
-        }
+        this._safeDestroy(this.saveStatus, 'saveStatus');
+        this.saveStatus = null;
         
-        if (this.textPropertiesPanel) {
-            this.textPropertiesPanel.destroy();
-            this.textPropertiesPanel = null;
-        }
+        this._safeDestroy(this.textPropertiesPanel, 'textPropertiesPanel');
+        this.textPropertiesPanel = null;
 
-        if (this.framePropertiesPanel) {
-            this.framePropertiesPanel.destroy();
-            this.framePropertiesPanel = null;
-        }
+        this._safeDestroy(this.framePropertiesPanel, 'framePropertiesPanel');
+        this.framePropertiesPanel = null;
         
-        if (this.notePropertiesPanel) {
-            this.notePropertiesPanel.destroy();
-            this.notePropertiesPanel = null;
-        }
+        this._safeDestroy(this.notePropertiesPanel, 'notePropertiesPanel');
+        this.notePropertiesPanel = null;
         
-        if (this.alignmentGuides) {
-            this.alignmentGuides.destroy();
-            this.alignmentGuides = null;
-        }
+        this._safeDestroy(this.alignmentGuides, 'alignmentGuides');
+        this.alignmentGuides = null;
         
-        if (this.commentPopover) {
-            this.commentPopover.destroy();
-            this.commentPopover = null;
-        }
+        this._safeDestroy(this.commentPopover, 'commentPopover');
+        this.commentPopover = null;
         
-        if (this.contextMenu) {
-            this.contextMenu.destroy();
-            this.contextMenu = null;
-        }
+        this._safeDestroy(this.contextMenu, 'contextMenu');
+        this.contextMenu = null;
+        
+        this._safeDestroy(this.zoombar, 'zoombar');
+        this.zoombar = null;
+        
+        this._safeDestroy(this.mapbar, 'mapbar');
+        this.mapbar = null;
         
         // Уничтожаем ядро
-        if (this.coreMoodboard) {
-            this.coreMoodboard.destroy();
-            this.coreMoodboard = null;
-        }
+        this._safeDestroy(this.coreMoodboard, 'coreMoodboard');
+        this.coreMoodboard = null;
         
         // Уничтожаем workspace
-        if (this.workspaceManager) {
-            this.workspaceManager.destroy();
-            this.workspaceManager = null;
-        }
+        this._safeDestroy(this.workspaceManager, 'workspaceManager');
+        this.workspaceManager = null;
         
         // Очищаем ссылки на менеджеры
         this.dataManager = null;

package/src/tools/object-tools/SelectTool.js

@@ -308,6 +308,11 @@ export class SelectTool extends BaseTool {
      * Перемещение мыши
      */
 		onMouseMove(event) {
+        // Проверяем, что инструмент не уничтожен
+        if (this.destroyed) {
+            return;
+        }
+        
         super.onMouseMove(event);
         
         // Обновляем текущие координаты мыши
@@ -468,6 +473,11 @@ export class SelectTool extends BaseTool {
      * Тестирование попадания курсора
      */
     hitTest(x, y) {
+        // Проверяем, что инструмент не уничтожен
+        if (this.destroyed) {
+            return { type: 'empty' };
+        }
+        
         // Сначала проверяем ручки изменения размера (они имеют приоритет)
         if (this.resizeHandles) {
             const pixiObjectAtPoint = this.getPixiObjectAt(x, y);
@@ -509,7 +519,7 @@ export class SelectTool extends BaseTool {
             return null;
         }
         
-        if (!this.resizeHandles || !this.resizeHandles.app) return null;
+        if (!this.resizeHandles || !this.resizeHandles.app || !this.resizeHandles.container) return null;
         
         const point = new PIXI.Point(x, y);
         
@@ -1051,6 +1061,11 @@ export class SelectTool extends BaseTool {
      * Обновление курсора
      */
     updateCursor(event) {
+        // Проверяем, что инструмент не уничтожен
+        if (this.destroyed) {
+            return;
+        }
+        
         const hitResult = this.hitTest(event.x, event.y);
         
         switch (hitResult.type) {

package/src/ui/ContextMenu.js

@@ -9,6 +9,9 @@ export class ContextMenu {
         this.lastX = 0;
         this.lastY = 0;
         this.currentGridType = 'line';
+        
+        // Флаг состояния объекта
+        this.destroyed = false;
 
         this.createElement();
         this.attachEvents();
@@ -335,6 +338,30 @@ export class ContextMenu {
         // По умолчанию — пусто
         this.element.innerHTML = '<div style="padding:8px 12px; color:#888;">(пусто)</div>';
     }
+
+    /**
+     * Уничтожение контекстного меню
+     */
+    destroy() {
+        if (this.destroyed) {
+            return;
+        }
+        
+        this.destroyed = true;
+        
+        // Скрываем меню
+        this.hide();
+        
+        // Удаляем DOM элемент
+        if (this.element && this.element.parentNode) {
+            this.element.parentNode.removeChild(this.element);
+            this.element = null;
+        }
+        
+        // Очищаем ссылки
+        this.container = null;
+        this.eventBus = null;
+    }
 }