fix v-for range edge cases (fix vuejs#2265)

yyx990803 · yyx990803 · commit 7cab5298c08a · 2016-01-29T10:17:41.000-08:00
diff --git a/src/directives/public/for.js b/src/directives/public/for.js
@@ -526,7 +526,7 @@ const vFor = {
       }
       return res
     } else {
-      if (typeof value === 'number') {
+      if (typeof value === 'number' && !isNaN(value)) {
         value = range(value)
       }
       return value || []
@@ -609,7 +609,7 @@ function findVmFromFrag (frag) {
 
 function range (n) {
   var i = -1
-  var ret = new Array(n)
+  var ret = new Array(Math.floor(n))
   while (++i < n) {
     ret[i] = i
   }

Original file line number	Diff line number	Diff line change
`@@ -526,7 +526,7 @@ const vFor = {`
`526`	`526`	`}`
`527`	`527`	`return res`
`528`	`528`	`} else {`
`529`		`- if (typeof value === 'number') {`
	`529`	`+ if (typeof value === 'number' && !isNaN(value)) {`
`530`	`530`	`value = range(value)`
`531`	`531`	`}`
`532`	`532`	`return value \|\| []`
`@@ -609,7 +609,7 @@ function findVmFromFrag (frag) {`
`609`	`609`
`610`	`610`	`function range (n) {`
`611`	`611`	`var i = -1`
`612`		`- var ret = new Array(n)`
	`612`	`+ var ret = new Array(Math.floor(n))`
`613`	`613`	`while (++i < n) {`
`614`	`614`	`ret[i] = i`
`615`	`615`	`}`