Added TableModel:updateNormalized() override to fix state mutation bug in redux-db

Author: jetako

package.json

@@ -1,6 +1,6 @@
 {
   "name": "redux-db-extras",
-  "version": "1.0.34",
+  "version": "1.0.35",
   "description": "Collections, selectors, and top-level record fields for redux-db",
   "main": "src/index.js",
   "scripts": {
@@ -13,6 +13,7 @@
     "reselect": "^3.0.1"
   },
   "peerDependencies": {
+    "lodash": "^4.17.4",
     "redux-db": "^0.9.11"
   },
   "repository": {

src/TableModel.js

@@ -106,6 +106,43 @@ export default class TableModel extends DefaultTableModel {
     this.dirty = true
   }
 
+  /**
+   * Fix state mutation bug 
+   */
+  updateNormalized(table) {
+    let dirty
+    let { state } = this
+
+    const records = Object.keys(table.byId).map(id => {
+      if (!state.byId[id])
+        throw new Error("Failed to apply update. No \"" + this.schema.name + "\" record with id: " + id + " exists.")
+
+      const oldRecord = state.byId[id]
+      const newRecord = { ...oldRecord, ...table.byId[id] }
+
+      const isModified = this.schema.isModified(oldRecord, newRecord)
+
+      if (isModified) {
+        if (!this.dirty) {
+          state = { ...state, byId: { ...state.byId } }
+          this.dirty = true
+        }
+
+        dirty = true
+        state.byId[id] = newRecord
+      }
+
+      return this.schema.db.factory.newRecordModel(id, this)
+    })
+
+    if (dirty) {
+      this.state = state
+      this._updateIndexes(table)
+    }
+
+    return records
+  }
+
   _cleanIndexes(id, record, indexes) {
     super._cleanIndexes(id, record, indexes)