diff --git a/src/platforms/web/compiler/directives/model.js b/src/platforms/web/compiler/directives/model.js index 8f5951eac56..7c662629b65 100644 --- a/src/platforms/web/compiler/directives/model.js +++ b/src/platforms/web/compiler/directives/model.js @@ -93,7 +93,7 @@ function genCheckboxModel ( 'if(Array.isArray($$a)){' + `var $$v=${number ? '_n(' + valueBinding + ')' : valueBinding},` + '$$i=_i($$a,$$v);' + - `if($$el.checked){$$i<0&&(${value}=$$a.concat($$v))}` + + `if($$el.checked){$$i<0&&(${value}=$$a.concat([$$v]))}` + `else{$$i>-1&&(${value}=$$a.slice(0,$$i).concat($$a.slice($$i+1)))}` + `}else{${genAssignmentCode(value, '$$c')}}`, null, true diff --git a/test/unit/features/directives/model-checkbox.spec.js b/test/unit/features/directives/model-checkbox.spec.js index 2c296ccbf5a..e102af62609 100644 --- a/test/unit/features/directives/model-checkbox.spec.js +++ b/test/unit/features/directives/model-checkbox.spec.js @@ -161,6 +161,34 @@ describe('Directive v-model checkbox', () => { }).then(done) }) + it('bind to Array value with array value bindings (object loose equal)', done => { + const vm = new Vue({ + data: { + test: [{ a: 1 }] + }, + template: ` +
+ + +
+ ` + }).$mount() + document.body.appendChild(vm.$el) + expect(vm.$el.children[0].checked).toBe(true) + expect(vm.$el.children[1].checked).toBe(false) + vm.$el.children[0].click() + expect(vm.test.length).toBe(0) + vm.$el.children[1].click() + expect(vm.test).toEqual([[2]]) + vm.$el.children[0].click() + expect(vm.test).toEqual([[2], { a: 1 }]) + vm.test = [{ a: 1 }] + waitForUpdate(() => { + expect(vm.$el.children[0].checked).toBe(true) + expect(vm.$el.children[1].checked).toBe(false) + }).then(done) + }) + it('.number modifier', () => { const vm = new Vue({ data: {