simplify v-model assignment gen + support adding reactive field when using dynamic segments

yyx990803 · yyx990803 · commit d333a498e93a · 2017-05-25T18:15:15.000+08:00
diff --git a/src/compiler/directives/model.js b/src/compiler/directives/model.js
@@ -41,10 +41,7 @@ export function genAssignmentCode (
   if (modelRs.idx === null) {
     return `${value}=${assignment}`
   } else {
-    return `var $$exp = ${modelRs.exp}, $$idx = ${modelRs.idx};` +
-      `if (!Array.isArray($$exp)){` +
-        `${value}=${assignment}}` +
-      `else{$$exp.splice($$idx, 1, ${assignment})}`
+    return `$set(${modelRs.exp}, ${modelRs.idx}, ${assignment})`
   }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -41,10 +41,7 @@ export function genAssignmentCode (`
`41`	`41`	`if (modelRs.idx === null) {`
`42`	`42`	return `${value}=${assignment}`
`43`	`43`	`} else {`
`44`		- return `var $$exp = ${modelRs.exp}, $$idx = ${modelRs.idx};` +
`45`		- `if (!Array.isArray($$exp)){` +
`46`		- `${value}=${assignment}}` +
`47`		- `else{$$exp.splice($$idx, 1, ${assignment})}`
	`44`	+ return `$set(${modelRs.exp}, ${modelRs.idx}, ${assignment})`
`48`	`45`	`}`
`49`	`46`	`}`
`50`	`47`