From d67055f23aacedf8aab6504b57dbf44a70847db3 Mon Sep 17 00:00:00 2001 From: Oliver Foster Date: Mon, 14 Oct 2024 09:27:18 +0100 Subject: [PATCH] Fix: locked attributes with conflicting default value --- js/models/lockingModel.js | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/js/models/lockingModel.js b/js/models/lockingModel.js index 81f434c1..866e916f 100644 --- a/js/models/lockingModel.js +++ b/js/models/lockingModel.js @@ -32,11 +32,8 @@ export default class LockingModel extends Backbone.Model { const defaults = _.result(this, 'defaults'); const isDefault = (defaults[attrName] !== undefined); const isInitialDefault = (isDefault && !this.changed); - if (isInitialDefault) { - this._lockedAttributes[attrName] = !defaults[attrName]; - } - const isSettingValueForSpecificPlugin = options?.pluginName; + if (!isSettingValueForSpecificPlugin) { if (!isInitialDefault) { console.error('Must supply a pluginName to change a locked attribute'); @@ -65,7 +62,7 @@ export default class LockingModel extends Backbone.Model { } } - if (!Object.keys(newValues)) return this; + if (!Object.keys(newValues).length) return this; super.set(newValues, options);