Merge pull request #143 from lionel-bijaoui/lb_fix_checklist_value_onChange

Change `onChange` trigger event from `change` to `input` for `checklist`

Author: lionel-bijaoui

src/fields/core/fieldChecklist.vue

@@ -3,7 +3,7 @@
 		.listbox.form-control(v-if="schema.listBox", :disabled="disabled")
 			.list-row(v-for="item in items")
 				label
-					input(type="checkbox", :checked="getItemIsChecked(item)", :disabled="disabled", @change="onChanged($event, item)")
+					input(type="checkbox", :checked="getItemIsChecked(item)", :disabled="disabled", @input="onChanged($event, item)")
 					| {{ getItemName(item) }}
 
 		.combobox.form-control(v-if="!schema.listBox", :disabled="disabled")
@@ -14,7 +14,7 @@
 			.dropList
 				.list-row(v-if="comboExpanded", v-for="item in items")
 					label
-						input(type="checkbox", :checked="getItemIsChecked(item)", :disabled="disabled", @change="onChanged($event, item)")
+						input(type="checkbox", :checked="getItemIsChecked(item)", :disabled="disabled", @input="onChanged($event, item)")
 						| {{ getItemName(item) }}
 </template>
 
@@ -69,7 +69,7 @@
 
 			onChanged(event, item) {
 				if (isNil(this.value))
-					this.value = [];
+					this.$set(this, "value", []);
 
 				if (event.target.checked)
 					this.value.push(this.getItemID(item));

test/unit/specs/fields/fieldChecklist.spec.js

@@ -78,7 +78,7 @@ describe("fieldChecklist.vue", function() {
 
 			it("model value should be the listbox value if changed", (done) => {
 				checkboxes[0].checked = true;
-				trigger(checkboxes[0], "change");
+				trigger(checkboxes[0], "input");
 
 				vm.$nextTick( () => {
 					expect(model.skills).to.be.deep.equal(["ReactJS", "HTML5"]);
@@ -142,7 +142,7 @@ describe("fieldChecklist.vue", function() {
 
 			it("model value should be the listbox value if changed", (done) => {
 				checkboxes[0].checked = true;
-				trigger(checkboxes[0], "change");
+				trigger(checkboxes[0], "input");
 
 				vm.$nextTick( () => {
 					expect(model.skills).to.be.deep.equal([3, 1]);
@@ -208,7 +208,7 @@ describe("fieldChecklist.vue", function() {
 
 			it("model value should be the listbox value if changed", (done) => {
 				checkboxes[0].checked = true;
-				trigger(checkboxes[0], "change");
+				trigger(checkboxes[0], "input");
 
 				vm.$nextTick( () => {
 					expect(model.skills).to.be.deep.equal([3, 1]);
@@ -305,7 +305,7 @@ describe("fieldChecklist.vue", function() {
 
 			it("model value should be the dropList value if changed (add)", (done) => {
 				checkboxes[0].checked = true;
-				trigger(checkboxes[0], "change");
+				trigger(checkboxes[0], "input");
 
 				vm.$nextTick( () => {
 					expect(model.skills).to.be.deep.equal(["ReactJS", "HTML5"]);
@@ -316,7 +316,7 @@ describe("fieldChecklist.vue", function() {
 
 			it("model value should be the checklist value if changed (remove)", (done) => {
 				checkboxes[0].checked = false;
-				trigger(checkboxes[0], "change");
+				trigger(checkboxes[0], "input");
 
 				vm.$nextTick( () => {
 					expect(model.skills).to.be.deep.equal(["ReactJS"]);
@@ -328,7 +328,7 @@ describe("fieldChecklist.vue", function() {
 			it("model value should be the dropList value if changed (null)", (done) => {
 				model.skills = null;
 				checkboxes[0].checked = true;
-				trigger(checkboxes[0], "change");
+				trigger(checkboxes[0], "input");
 
 				vm.$nextTick( () => {
 					expect(model.skills).to.be.deep.equal(["HTML5"]);