Vue.js add property to bound object

Question

I have an list like this one

<div class="list-group entity-list">

   <a  v-for="entity in master.data"
       :class="['entity list-group-item', entity.selected ? 'selected' : '']"
       @click="itemselect(entity)"
   >
      @{{entity.name}}
   </a>
</div>

The list is displayed as expected. The Bound entity is added as an prop to an outer compontent like this.

<my-compontent :entityname="customers" :master="{{json_encode($customers->toArray())}}" inline-template>

As you can see it should be a list of customers. To indicate if a customer is selected i want to set an "selected" attribute on click. However it does not seem to work if the default bound object does not have a select property.

methods: {
    itemselect: function(item) {
        this.master.data.forEach(function(el) {
            el.selected = false;
        });

        item.selected = true;
    }
}

The shown method does not seem to work / trigger a change

craig_h · Accepted Answer · 2016-10-24 12:55:21Z

If you want to modify data passed in by a prop then you need to copy it, vue will throw out a warning if you try to directly modify a prop. I would probably handle this by doing something like:

new Vue({
  el: '#app',
  created() {
    // Assume this is passed in by a prop
      var myData = {master: {
        data: [{
          name: 'foo'
        }, {
          name: 'bar'
        }, {
          name: 'baz'
        }]
      }};

    // Copy data from prop
    myData.master.data.forEach(el => {
      var obj = {
        name: el.name,
        selected: false
      };

      this.master.data.push(obj);
    });
  },
  methods: {
    itemselect(item) {
      this.master.data.forEach(el => {
        // if this element is selected set to true      
        el['selected'] = (el.name == item.name);
      });
    }
  },
  data() {
    return {
      master: {
        data: []
      }
    }
  }
});

I just mocked the actual data coming in, but obviously this would add this to a prop and pass it to your component as usual. Here's the jsFiddle: https://jsfiddle.net/xswtfkaf/

cske · Accepted Answer · 2016-10-24 12:04:34Z

Instead of altering your master data, track selection in model, using $index assuming you don't reorder your master.data

<div class="list-group entity-list">
    <a  v-for="entity in master.data"
        :class="eClass[$index]"
        @click="itemselect($index)"
            >
        @{{entity.name}}
    </a>
</div>

methods: {
    itemselect: function(idx) {
        if ( 0 > this.selected.indexOf(idx) ) {
            this.selected.push(idx)
        }
    }
}
computed : {
    eClass : function () {
        var r=[];
        for (var i= 0;c=this.master.data[i];++i) {
            r[i] = {'selected': -1 < this.selected.indexOf(i),'entity list-group-item':true};
        }
        return r;
    }
}

if entity has am unique id

<div class="list-group entity-list">
    <a  v-for="entity in master.data"
        :class="eClass[entity.id]"
        @click="itemselect(entity.id)"
            >
        @{{entity.name}}
    </a>
</div>

methods: {
    itemselect: function(idx) {
        if ( 0 > this.selected.indexOf(idx) ) {
            this.selected.push(idx)
        }
    }
}
computed : {
    eClass : function () {
        var r={};
        for (var i= 0;c=this.master.data[i];++i) {
            r[c.id] = {'selected': -1 < this.selected.indexOf(c.id),'entity list-group-item':true};
        }
        return r;
    }
}

Collectives™ on Stack Overflow

Vue.js add property to bound object

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related