From f9f78968b15896a496624e12e4609750cd2421f5 Mon Sep 17 00:00:00 2001 From: Ben Gotow Date: Fri, 15 Jul 2016 19:01:50 -0700 Subject: [PATCH] Use defaultValue as a template, fix critical sync issue --- packages/nylas-core/database-types.js | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/packages/nylas-core/database-types.js b/packages/nylas-core/database-types.js index 8216e8d0f..7a59c5577 100644 --- a/packages/nylas-core/database-types.js +++ b/packages/nylas-core/database-types.js @@ -5,18 +5,22 @@ module.exports = { type: Sequelize.TEXT, get: function get() { const val = this.getDataValue(fieldName); - if (!val) { return defaultValue } + if (!val) { + return defaultValue ? Object.assign({}, defaultValue) : null; + } return JSON.parse(val); }, set: function set(val) { this.setDataValue(fieldName, JSON.stringify(val)); }, }), - JSONARRAYType: (fieldName, {defaultValue = []} = {}) => ({ + JSONARRAYType: (fieldName) => ({ type: Sequelize.TEXT, get: function get() { const val = this.getDataValue(fieldName); - if (!val) { return defaultValue } + if (!val) { + return []; + } return JSON.parse(val); }, set: function set(val) {