From af64dd61523363cc7677a0e6d5813cfa753d2213 Mon Sep 17 00:00:00 2001
From: Ghislain B <gbeaulac@gmail.com>
Date: Wed, 5 Feb 2020 09:26:08 -0500
Subject: [PATCH] fix(pagination): on filter change pagination should reset to
 1st page (#397)

- when using local grid, the DataView should be reset to page 0
---
 .../services/__tests__/pagination.service.spec.ts    | 12 ++++++++++++
 .../angular-slickgrid/services/pagination.service.ts |  4 ++++
 2 files changed, 16 insertions(+)

diff --git a/src/app/modules/angular-slickgrid/services/__tests__/pagination.service.spec.ts b/src/app/modules/angular-slickgrid/services/__tests__/pagination.service.spec.ts
index 65951bf93..3269e1478 100644
--- a/src/app/modules/angular-slickgrid/services/__tests__/pagination.service.spec.ts
+++ b/src/app/modules/angular-slickgrid/services/__tests__/pagination.service.spec.ts
@@ -565,6 +565,18 @@ describe('PaginationService', () => {
 
       expect(spy).toHaveBeenCalledWith(true, false);
     });
+
+    it('should reset the DataView when using local grid by calling "setPagingOptions" with page 0 and also call "refreshPagination" method', () => {
+      const spy = jest.spyOn(service, 'refreshPagination');
+      const setPagingSpy = jest.spyOn(dataviewStub, 'setPagingOptions');
+
+      mockGridOption.backendServiceApi = null;
+      service.init(gridStub, dataviewStub, mockGridOption.pagination, null);
+      service.resetPagination();
+
+      expect(setPagingSpy).toHaveBeenCalledWith({ pageSize: 25, pageNum: 0 });
+      expect(spy).toHaveBeenCalledWith(true, true);
+    });
   });
 
   // processOnItemAddedOrRemoved is private but we can spy on recalculateFromToIndexes
diff --git a/src/app/modules/angular-slickgrid/services/pagination.service.ts b/src/app/modules/angular-slickgrid/services/pagination.service.ts
index 8b049acbc..85f45f97b 100644
--- a/src/app/modules/angular-slickgrid/services/pagination.service.ts
+++ b/src/app/modules/angular-slickgrid/services/pagination.service.ts
@@ -329,6 +329,10 @@ export class PaginationService {
 
   /** Reset the Pagination to first page and recalculate necessary numbers */
   resetPagination(triggerChangedEvent = true) {
+    if (this._isLocalGrid) {
+      // on a local grid we also need to reset the DataView paging to 1st page
+      this.dataView.setPagingOptions({ pageSize: this._itemsPerPage, pageNum: 0 });
+    }
     this.refreshPagination(true, triggerChangedEvent);
   }