Use custom event target instead of window

WordPress · Mar 12, 2024 · f8694b8 · f8694b8
1 parent f447ca9
commit f8694b8
Showing 1 changed file with 10 additions and 14 deletions.
diff --git a/packages/php-wasm/logger/src/logger.ts b/packages/php-wasm/logger/src/logger.ts
@@ -15,7 +15,7 @@ export type LogPrefix = 'Playground' | 'PHP-WASM';
 /**
  * A logger for Playground.
  */
-export class Logger {
+export class Logger extends EventTarget {
 	public readonly fatalErrorEvent = 'playground-fatal-error';
 
 	/**
@@ -39,6 +39,7 @@ export class Logger {
 	private errorLogPath = '/wordpress/wp-content/debug.log';
 
 	constructor(errorLogPath?: string) {
+		super();
 		if (errorLogPath) {
 			this.errorLogPath = errorLogPath;
 		}
@@ -79,15 +80,13 @@ export class Logger {
 	}
 
 	public maybeDispatchFatalErrorEvent() {
-		if (typeof window !== 'undefined') {
-			window.dispatchEvent(
-				new CustomEvent(this.fatalErrorEvent, {
-					detail: {
-						logs: this.getLogs(),
-					},
-				})
-			);
-		}
+		this.dispatchEvent(
+			new CustomEvent(this.fatalErrorEvent, {
+				detail: {
+					logs: this.getLogs(),
+				},
+			})
+		);
 	}
 
 	/**
@@ -255,8 +254,5 @@ export function addFatalErrorListener(
 	loggerInstance: Logger,
 	callback: EventListenerOrEventListenerObject
 ) {
-	if (typeof window === 'undefined') {
-		return;
-	}
-	window.addEventListener(loggerInstance.fatalErrorEvent, callback);
+	loggerInstance.addEventListener(loggerInstance.fatalErrorEvent, callback);
 }