Fix CRUDReducer bug when retrieving item that already exists

2019-04-03 23:51:38 +08:00 · 2019-04-03 23:51:38 +08:00 · 363b84699f
commit 363b84699f
parent a81a68b6e2
1 changed files with 5 additions and 1 deletions
--- a/packages/client/src/crud/CRUDReducer.ts
+++ b/packages/client/src/crud/CRUDReducer.ts
@ -115,10 +115,14 @@ export class CRUDReducer<
  }

  handleFindOne = (state: ICRUDState<T>, payload: T): ICRUDState<T> => {
+    const ids = !state.byId[payload.id]
+      ? [...state.ids, payload.id]
+      : state.ids
    return {
      ...state,
-      ids: [...state.ids, payload.id],
+      ids,
      byId: {
+        ...state.byId,
        [payload.id]: payload,
      },
      status: {