fix: [#1527] Fixes problem where it is not possible to observe again …

…after disconnecting in MutationObserver (#1525)
capricorn86 · Aug 31, 2024 · 20b520a · 20b520a
1 parent 07a424a
commit 20b520a
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/packages/happy-dom/src/mutation-observer/MutationObserver.ts b/packages/happy-dom/src/mutation-observer/MutationObserver.ts
@@ -133,8 +133,6 @@ export default class MutationObserver {
 	 * Disconnects.
 	 */
 	public disconnect(): void {
-		this.#destroyed = true;
-
 		if (this.#listeners.length === 0) {
 			return;
 		}
@@ -166,4 +164,12 @@ export default class MutationObserver {
 		}
 		return records;
 	}
+
+	/**
+	 *
+	 */
+	public [PropertySymbol.destroy](): void {
+		this.#destroyed = true;
+		this.disconnect();
+	}
 }
diff --git a/packages/happy-dom/src/window/BrowserWindow.ts b/packages/happy-dom/src/window/BrowserWindow.ts
@@ -1462,7 +1462,9 @@ export default class BrowserWindow extends EventTarget implements INodeJSGlobal
 		const mutationObservers = this[PropertySymbol.mutationObservers];
 
 		for (const mutationObserver of mutationObservers) {
-			mutationObserver.disconnect();
+			if (mutationObserver[PropertySymbol.destroy]) {
+				mutationObserver[PropertySymbol.destroy]();
+			}
 		}
 
 		this[PropertySymbol.mutationObservers] = [];