@configura/web-utilities 2.2.1 → 2.3.0-alpha.1

package/dist/Observable/Observable.d.ts

@@ -7,7 +7,16 @@ export declare class Observable<T> {
     stopListen: (listener: (v: T) => void) => void;
     stopAllListen: () => void;
     notifyAll: (v: T, context?: unknown) => void;
-    link: (other: Observable<T>) => void;
+    /**
+     * Links the two observables so events raised in one will trigger the other one, and vice versa.
+     * @param other The other observable to link with
+     * @returns A function to unlink the two observables
+     */
+    link: (other: Observable<T>) => (() => void);
+    /**
+     * @deprecated This function does not unlink properly. Use the returned value from {@link link} instead.
+     * @param other The other observable to link with
+     */
     unlink: (other: Observable<T>) => void;
 }
 //# sourceMappingURL=Observable.d.ts.map

package/dist/Observable/Observable.js

@@ -21,13 +21,27 @@ export class Observable {
                 l.l(v);
             });
         };
+        /**
+         * Links the two observables so events raised in one will trigger the other one, and vice versa.
+         * @param other The other observable to link with
+         * @returns A function to unlink the two observables
+         */
         this.link = (other) => {
-            this.listen((v) => other.notifyAll(v, this), other);
-            other.listen((v) => this.notifyAll(v, other), this);
+            const thisListener = (v) => other.notifyAll(v, this);
+            const otherListener = (v) => other.notifyAll(v, other);
+            this.listen(thisListener, other);
+            other.listen(otherListener, this);
+            return () => {
+                this.stopListen(thisListener);
+                other.stopListen(otherListener);
+            };
         };
+        /**
+         * @deprecated This function does not unlink properly. Use the returned value from {@link link} instead.
+         * @param other The other observable to link with
+         */
         this.unlink = (other) => {
-            this.stopListen(other.notifyAll);
-            other.stopListen(this.notifyAll);
+            console.warn("The Observable.unlink() function has been deprecated. Use the returned value from link() instead");
         };
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@configura/web-utilities",
-	"version": "2.2.1",
+	"version": "2.3.0-alpha.1",
 	"license": "Apache-2.0",
 	"type": "module",
 	"main": "dist/index.js",
@@ -22,5 +22,5 @@
 	"publishConfig": {
 		"access": "public"
 	},
-	"gitHead": "f5f47783fde9eddb263baa0739d4fde4ea4b38c2"
+	"gitHead": "1b93d8008aca97111b931a535b9d92b01ec24c4f"
 }