observable.js: lazy init of logger to go around bidirectional dependency issues.

Dierk Koenig · Dierk Koenig · commit d05210d7689b · 2024-12-08T21:12:45.000+01:00
diff --git a/docs/src/kolibri/observable.js b/docs/src/kolibri/observable.js
@@ -4,11 +4,13 @@ import { LOG_CONTEXT_KOLIBRI_BASE } from "./logger/logConstants.js";
 
 export {Observable, ObservableList}
 
-const { warn } = LoggerFactory(LOG_CONTEXT_KOLIBRI_BASE + ".observable");
-
+let warn = undefined;
 /** @private */
 function checkWarning(list) {
     if (list.length > 100) {
+        if (!warn) {
+            warn = LoggerFactory(LOG_CONTEXT_KOLIBRI_BASE + ".observable").warn;
+        }
         warn(`Beware of memory leak. ${list.length} listeners.`);
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -4,11 +4,13 @@ import { LOG_CONTEXT_KOLIBRI_BASE } from "./logger/logConstants.js";`
`4`	`4`
`5`	`5`	`export {Observable, ObservableList}`
`6`	`6`
`7`		`-const { warn } = LoggerFactory(LOG_CONTEXT_KOLIBRI_BASE + ".observable");`
`8`		`-`
	`7`	`+let warn = undefined;`
`9`	`8`	`/** @private */`
`10`	`9`	`function checkWarning(list) {`
`11`	`10`	`if (list.length > 100) {`
	`11`	`+ if (!warn) {`
	`12`	`+ warn = LoggerFactory(LOG_CONTEXT_KOLIBRI_BASE + ".observable").warn;`
	`13`	`+ }`
`12`	`14`	warn(`Beware of memory leak. ${list.length} listeners.`);
`13`	`15`	`}`
`14`	`16`	`}`