fix(SystemsTable): Do not spread current params on reset

RedHatInsights · Sep 25, 2023 · 210e50e · 210e50e
1 parent eb9c80e
commit 210e50e
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/src/Helpers/TableToolbarHelper.js b/src/Helpers/TableToolbarHelper.js
@@ -102,7 +102,7 @@ export const buildActiveFilters = (currentFilters, filterRuleValues = []) => {
 
 export const removeFilters = (chips, apply, reset = false, defaultFilters = {}) => {
     if (reset) {
-        apply({ ...defaultFilters, page: 1 });
+        apply({ ...defaultFilters, page: 1, reset });
         return;
     }
 

diff --git a/src/Store/Reducers/SystemsPageStore.js b/src/Store/Reducers/SystemsPageStore.js
@@ -25,13 +25,17 @@ export const initialState = {
 export const SystemsPageStore = (state = initialState, action) => {
     let newState = { ...state };
     switch (action.type) {
-        case ActionTypes.CHANGE_SYSTEMS_PARAMS:
+        case ActionTypes.CHANGE_SYSTEMS_PARAMS: {
+            const { reset, ...params } = action.payload;
             newState.params = {
-                ...newState.params,
-                ...action.payload,
+                ...!reset ? newState.params : {},
+                ...params,
                 page_size: action.payload.page_size || newState.params.page_size
             };
+
             return newState;
+        }
+
         case ActionTypes.CHANGE_COLUMNS_SYSTEM_LIST:
             return {
                 ...newState,