fix(runtime-core): scheduler should allow intentional self triggering effects

yyx990803 · yyx990803 · commit c27dfe1d0994 · 2020-07-28T12:29:43.000-04:00
fix #1727
diff --git a/packages/runtime-core/src/scheduler.ts b/packages/runtime-core/src/scheduler.ts
@@ -26,7 +26,7 @@ export function nextTick(fn?: () => void): Promise<void> {
 }
 
 export function queueJob(job: Job) {
-  if (!queue.includes(job, flushIndex)) {
+  if (!queue.includes(job, flushIndex + 1)) {
     queue.push(job)
     queueFlush()
   }
@@ -43,7 +43,7 @@ export function queuePostFlushCb(cb: Function | Function[]) {
   if (!isArray(cb)) {
     if (
       !pendingPostFlushCbs ||
-      !pendingPostFlushCbs.includes(cb, pendingPostFlushIndex)
+      !pendingPostFlushCbs.includes(cb, pendingPostFlushIndex + 1)
     ) {
       postFlushCbs.push(cb)
     }

Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ export function nextTick(fn?: () => void): Promise<void> {`
`26`	`26`	`}`
`27`	`27`
`28`	`28`	`export function queueJob(job: Job) {`
`29`		`- if (!queue.includes(job, flushIndex)) {`
	`29`	`+ if (!queue.includes(job, flushIndex + 1)) {`
`30`	`30`	`queue.push(job)`
`31`	`31`	`queueFlush()`
`32`	`32`	`}`
`@@ -43,7 +43,7 @@ export function queuePostFlushCb(cb: Function \| Function[]) {`
`43`	`43`	`if (!isArray(cb)) {`
`44`	`44`	`if (`
`45`	`45`	`!pendingPostFlushCbs \|\|`
`46`		`- !pendingPostFlushCbs.includes(cb, pendingPostFlushIndex)`
	`46`	`+ !pendingPostFlushCbs.includes(cb, pendingPostFlushIndex + 1)`
`47`	`47`	`) {`
`48`	`48`	`postFlushCbs.push(cb)`
`49`	`49`	`}`