fix: onDispose should return dispose function if no scope

maverick-js · Nov 30, 2022 · ca04b24 · ca04b24
1 parent 3e70c2e
commit ca04b24
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/observables.ts b/src/observables.ts
@@ -374,7 +374,7 @@ export function onError<T = Error>(handler: (error: T) => void): void {
  * @see {@link https://github.com/maverick-js/observables#ondispose}
  */
 export function onDispose(dispose: MaybeDispose): Dispose {
-  if (!dispose || !currentScope) return NOOP;
+  if (!dispose || !currentScope) return dispose || NOOP;
   (currentScope[DISPOSAL] ??= new Set()).add(dispose);
   return () => {
     (dispose as Dispose)();