fix(PagerProcessor): empty array returned on page change

Fix a bug where setting the page number on a disabled PagerProcessor would return an empty array. fixes #29
Colonise · Aug 2, 2018 · 17470c5 · 17470c5
1 parent 5aab48c
commit 17470c5
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/processors/complex.ts b/src/processors/complex.ts
@@ -66,7 +66,9 @@ export abstract class ComplexProcessor<TData> extends BehaviourSubject<TData> im
             return data;
         }
 
-        return this.next(this.processor(data));
+        const processedData = this.processor(data);
+
+        return this.next(processedData);
     }
 
     /**

diff --git a/src/processors/pager.ts b/src/processors/pager.ts
@@ -41,7 +41,9 @@ export class PagerProcessor<TEntry> extends ArrayProcessor<TEntry> implements Pa
 
         this._page = page;
 
-        this.next(this.currentPageEntries());
+        if (this.shouldProcess()) {
+            this.next(this.currentPageEntries());
+        }
     }
 
     // tslint:disable-next-line:variable-name