Move checkbox trait value logic to the model

src/trait_manager/model/Trait.ts

@@ -195,7 +195,7 @@ export default class Trait extends Model<TraitProperties> {
   }
 
   setTargetValue(value: any, opts: SetOptions = {}) {
-    const target = this.target;
+    const { target, attributes } = this;
     const name = this.getName();
     if (isUndefined(value)) return;
     let valueToSet = value;
@@ -206,6 +206,18 @@ export default class Trait extends Model<TraitProperties> {
       valueToSet = true;
     }
 
+    if (this.getType() === 'checkbox') {
+      const { valueTrue, valueFalse } = attributes;
+
+      if (valueToSet && !isUndefined(valueTrue)) {
+        valueToSet = valueTrue;
+      }
+
+      if (!valueToSet && !isUndefined(valueFalse)) {
+        valueToSet = valueFalse;
+      }
+    }
+
     if (this.get('changeProp')) {
       target.set(name, valueToSet, opts);
     } else {

src/trait_manager/view/TraitCheckboxView.ts

@@ -16,23 +16,7 @@ export default class TraitCheckboxView extends TraitView {
    * @private
    */
   onChange() {
-    const value = this.getInputElem().checked;
-    this.model.set('value', this.getCheckedValue(value));
-  }
-
-  getCheckedValue(checked: boolean) {
-    let result: boolean | string = checked;
-    const { valueTrue, valueFalse } = this.model.attributes;
-
-    if (result && !isUndefined(valueTrue)) {
-      result = valueTrue;
-    }
-
-    if (!result && !isUndefined(valueFalse)) {
-      result = valueFalse;
-    }
-
-    return result;
+    this.model.set('value', this.getInputElem().checked);
   }
 
   /**