fix: update initial after save (#309)

* use set intead of setItem after a save request

After a save request, the initial store should
be updated too.

* fix setAllMutation

in some cases the method is called with just one
item as payload.
In this case we just have to update that item within the collection.

* add changelog entry

* linting

Author: salisdemos

Changelog.md

@@ -1,6 +1,7 @@
 # next version
 
 - fix listRelationships for "to one"-relationships 
+- fix storing data on save in collections
 
 # v0.0.35
 

src/module/actions/saveAction.js

@@ -70,15 +70,15 @@ export function saveAction (api, isCollection, moduleName, defaultQuery = {}) {
         const { data, status } = response
 
         if (status === 204) {
-          vuexFns.commit('setItem', getExpectedResponse(currentItemState))
+          vuexFns.commit('set', getExpectedResponse(currentItemState))
         }
 
         if (status === 200) {
           const isEmptyArray = Array.isArray(data) === true && data.length === 0
           const isEmptyObject = typeof data === 'object' && data !== null && Object.keys(data).length === 0
           const isNull = data === null
           if (isEmptyArray || isEmptyObject || isNull) {
-            vuexFns.commit('setItem', getExpectedResponse(currentItemState))
+            vuexFns.commit('set', getExpectedResponse(currentItemState))
           }
         }
 

src/module/mutations/setAllMutation.js

@@ -1,3 +1,6 @@
+import { hasOwn } from '../../shared/utils'
+import Vue from 'vue'
+
 /**
  * Proxy for setting Resource Objects on a collection module
  *
@@ -12,8 +15,13 @@ export function setAllMutation (resourceBuilder) {
     apply (target, thisArg, [state, payload]) {
       const settablePayload = resourceBuilder.build(payload)
 
-      state.items = { ...state.items, ...settablePayload }
-      state.initial = { ...state.initial, ...settablePayload }
+      if (hasOwn(settablePayload, 'id')) {
+        Vue.set(state.items, settablePayload.id, settablePayload)
+        Vue.set(state.initial, settablePayload.id, settablePayload)
+      } else {
+        state.items = { ...state.items, ...settablePayload }
+        state.initial = { ...state.initial, ...settablePayload }
+      }
     }
   })
 }