avoid v-model debounce on blur (fix vuejs#2028)

yyx990803 · yyx990803 · commit 81d38aa1d00b · 2015-12-15T11:35:55.000-05:00
diff --git a/src/directive.js b/src/directive.js
@@ -109,6 +109,7 @@ Directive.prototype._bind = function () {
   if (this.bind) {
     this.bind()
   }
+  this._bound = true
 
   if (this.literal) {
     this.update && this.update(descriptor.raw)
@@ -156,7 +157,6 @@ Directive.prototype._bind = function () {
       this.update(watcher.value)
     }
   }
-  this._bound = true
 }
 
 /**
diff --git a/src/directives/public/model/text.js b/src/directives/public/model/text.js
@@ -53,13 +53,13 @@ export default {
         self.focused = false
         // do not sync value after fragment removal (#2017)
         if (!self._frag || self._frag.inserted) {
-          self.listener()
+          self.rawListener()
         }
       })
     }
 
     // Now attach the main listener
-    this.listener = function () {
+    this.listener = this.rawListener = function () {
       if (composing || !self._bound) {
         return
       }
diff --git a/test/unit/specs/directives/public/model_spec.js b/test/unit/specs/directives/public/model_spec.js
@@ -709,8 +709,19 @@ describe('v-model', function () {
     }, 30)
     setTimeout(function () {
       expect(spy.calls.count()).toBe(1)
+      expect(spy).toHaveBeenCalledWith('d', 'a')
       expect(vm.test).toBe('d')
-      done()
+    }, 150)
+    setTimeout(function () {
+      el.firstChild.value = 'e'
+      // blur should trigger change instantly without debounce
+      trigger(el.firstChild, 'blur')
+      _.nextTick(function () {
+        expect(spy.calls.count()).toBe(2)
+        expect(spy).toHaveBeenCalledWith('e', 'd')
+        expect(vm.test).toBe('e')
+        done()
+      })
     }, 200)
   })
 

Original file line number	Diff line number	Diff line change
`@@ -109,6 +109,7 @@ Directive.prototype._bind = function () {`
`109`	`109`	`if (this.bind) {`
`110`	`110`	`this.bind()`
`111`	`111`	`}`
	`112`	`+ this._bound = true`
`112`	`113`
`113`	`114`	`if (this.literal) {`
`114`	`115`	`this.update && this.update(descriptor.raw)`
`@@ -156,7 +157,6 @@ Directive.prototype._bind = function () {`
`156`	`157`	`this.update(watcher.value)`
`157`	`158`	`}`
`158`	`159`	`}`
`159`		`- this._bound = true`
`160`	`160`	`}`
`161`	`161`
`162`	`162`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -53,13 +53,13 @@ export default {`
`53`	`53`	`self.focused = false`
`54`	`54`	`// do not sync value after fragment removal (#2017)`
`55`	`55`	`if (!self._frag \|\| self._frag.inserted) {`
`56`		`- self.listener()`
	`56`	`+ self.rawListener()`
`57`	`57`	`}`
`58`	`58`	`})`
`59`	`59`	`}`
`60`	`60`
`61`	`61`	`// Now attach the main listener`
`62`		`- this.listener = function () {`
	`62`	`+ this.listener = this.rawListener = function () {`
`63`	`63`	`if (composing \|\| !self._bound) {`
`64`	`64`	`return`
`65`	`65`	`}`