consider enumerability of fields

Author: gr0uch

CHANGELOG.md

@@ -1,7 +1,7 @@
 # Changelog
 
 
-##### 1.6.8 (2018-10-13)
+##### 1.6.9 (2018-10-13)
 - Fix: do not allow extraneous fields in update to cause silent error.
 
 

lib/index.js

@@ -441,7 +441,7 @@ module.exports = Adapter => class PostgreSQLAdapter extends Adapter {
     const client = this.client
     const typeMap = this.options.typeMap
     const primaryKey = this.keys.primary
-    const recordTypes = this.recordTypes
+    const recordType = this.recordTypes[type]
 
     // This is a little bit wrong, it is only safe to update within a
     // transaction. It's not possible to put it all in one update statement,
@@ -452,7 +452,7 @@ module.exports = Adapter => class PostgreSQLAdapter extends Adapter {
       let set = []
 
       for (const field in update.replace) {
-        if (!(field in recordTypes[type])) continue
+        if (!recordType.propertyIsEnumerable(field)) continue
         const value = update.replace[field]
         index++
         if (Array.isArray(value)) parameters.push(value.map(inputValue))
@@ -461,7 +461,7 @@ module.exports = Adapter => class PostgreSQLAdapter extends Adapter {
       }
 
       for (const field in update.push) {
-        if (!(field in recordTypes[type])) continue
+        if (!recordType.propertyIsEnumerable(field)) continue
         const value = update.push[field]
         index++
 
@@ -476,7 +476,7 @@ module.exports = Adapter => class PostgreSQLAdapter extends Adapter {
       }
 
       for (const field in update.pull) {
-        if (!(field in recordTypes[type])) continue
+        if (!recordType.propertyIsEnumerable(field)) continue
         const value = update.pull[field]
 
         if (Array.isArray(value)) {

package.json

@@ -1,7 +1,7 @@
 {
   "name": "fortune-postgres",
   "description": "Postgres adapter for Fortune.",
-  "version": "1.6.8",
+  "version": "1.6.9",
   "license": "MIT",
   "repository": {
     "type": "git",