Update href trait value. Closes #2411

src/i18n/locale/en.js

@@ -108,7 +108,7 @@ export default {
       // In a trait like select, these are used to translate option names
       options: {
         target: {
-          '': 'This window',
+          false: 'This window',
           _blank: 'New window'
         }
       }

src/i18n/locale/it.js

@@ -94,7 +94,7 @@ export default {
       },
       options: {
         target: {
-          '': 'Questa finestra',
+          false: 'Questa finestra',
           _blank: 'Nuova finestra'
         }
       }

src/trait_manager/config/config.js

@@ -6,5 +6,5 @@ export default {
   appendTo: '',
 
   // Default options for the target input
-  optionsTarget: [{ value: '' }, { value: '_blank' }]
+  optionsTarget: [{ value: false }, { value: '_blank' }]
 };

src/trait_manager/model/Trait.js

@@ -64,12 +64,19 @@ export default Backbone.Model.extend({
     const target = this.target;
     const name = this.get('name');
     if (isUndefined(value)) return;
+    let valueToSet = value;
+
+    if (value === 'false') {
+      valueToSet = false;
+    } else if (value === 'true') {
+      valueToSet = true;
+    }
 
     if (this.get('changeProp')) {
-      target.set(name, value, opts);
+      target.set(name, valueToSet, opts);
     } else {
       const attrs = { ...target.get('attributes') };
-      attrs[name] = value;
+      attrs[name] = valueToSet;
       target.set('attributes', attrs, opts);
     }
   },

src/trait_manager/view/TraitSelectView.js

@@ -54,7 +54,7 @@ export default TraitView.extend({
 
       input += '</select>';
       this.$input = $(input);
-      let val = model.getTargetValue() || model.get('value');
+      const val = model.getTargetValue();
       !isUndefined(val) && this.$input.val(val);
     }