call destroyed hook only once

web/static/src/ts/core/Widget.ts

@@ -21,6 +21,7 @@ interface Meta<T extends WEnv> {
   vnode: VNode | null;
   isStarted: boolean;
   isMounted: boolean;
+  isDestroyed: boolean;
   parent: Widget<T> | null;
   children: Widget<T>[];
 }
@@ -57,6 +58,7 @@ export class Widget<T extends WEnv> {
       vnode: null,
       isStarted: false,
       isMounted: false,
+      isDestroyed: false,
       parent: p,
       children: []
     };
@@ -91,8 +93,13 @@ export class Widget<T extends WEnv> {
   }
 
   destroy() {
-    if (this.el) {
+    if (!this._.isDestroyed) {
-      this.el.remove();
+      if (this.el) {
+        this.el.remove();
+        delete this._.vnode;
+      }
+      this._.isDestroyed = true;
+      this.destroyed();
     }
   }
 

web/static/tests/widget.test.ts

@@ -249,6 +249,21 @@ describe("destroy method", () => {
     widget.destroy();
     expect(document.contains(widget.el)).toBe(false);
   });
+
+  test("destroying a widget twice only call destroyed once", async () => {
+    let count = 0;
+    class TestWidget extends Widget<WEnv> {
+      destroyed() {
+        count++;
+      }
+    }
+    const widget = new TestWidget(env);
+    await widget.mount(fixture);
+    widget.destroy();
+    expect(count).toBe(1);
+    widget.destroy();
+    expect(count).toBe(1);
+  });
 });
 
 describe("composition", () => {