Skip to content

Commit 77a4aca

Browse files
authored
Merge pull request carlosrocha#39 from chhsiao90/fixed-filter-#38
Resolved carlosrocha#38: display data on page 0 after filter
2 parents 3d092bd + 33cf61a commit 77a4aca

File tree

2 files changed

+27
-1
lines changed

2 files changed

+27
-1
lines changed

src/__tests__/dataReducer-test.js

Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -132,4 +132,30 @@ describe('dataReducer', () => {
132132
expect(dataReducer(state, action)).toEqual(expected);
133133
});
134134

135+
it('filters on different page', () => {
136+
const data = [ ['carlos', 'r'], [3, 4] ];
137+
const state = {
138+
data,
139+
initialData: data,
140+
page: data.slice(0, 1),
141+
pageSize: 1,
142+
totalPages: 2,
143+
};
144+
const initState = {
145+
...state,
146+
pageNumber: 1,
147+
};
148+
const action = dataFilter('globalSearch', 'c', filters);
149+
const expected = {
150+
...state,
151+
pageNumber: 0,
152+
filterValues: { globalSearch: 'c' },
153+
data: [ ['carlos', 'r'] ],
154+
page: [ ['carlos', 'r'] ],
155+
totalPages: 1,
156+
};
157+
158+
expect(dataReducer(state, action)).toEqual(expected);
159+
});
160+
135161
});

src/dataReducer.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ function dataFilter(state, {value: {key, value, filters}}) {
7373

7474
return {
7575
...state,
76-
...calculatePage(data, state.pageSize, state.pageNumber),
76+
...calculatePage(data, state.pageSize, 0),
7777
data,
7878
filterValues: newFilterValues,
7979
pageNumber: 0,

0 commit comments

Comments
 (0)