fix wait-for with keep-alive (fix vuejs#1150)

yyx990803 · yyx990803 · commit 8600d8aeefdf · 2015-08-12T13:14:32.000-04:00
diff --git a/src/directives/component.js b/src/directives/component.js
@@ -129,8 +129,9 @@ module.exports = {
             }
           }
         }
-        var newComponent = this.build(options)
-        if (!waitFor) {
+        var cached = this.getCached()
+        var newComponent = cached || this.build(options)
+        if (!waitFor || cached) {
           this.transition(newComponent, cb)
         }
       }, this))
@@ -174,11 +175,9 @@ module.exports = {
    */
 
   build: function (extraOptions) {
-    if (this.keepAlive) {
-      var cached = this.cache[this.Component.cid]
-      if (cached) {
-        return cached
-      }
+    var cached = this.getCached()
+    if (cached) {
+      return cached
     }
     if (this.Component) {
       // default options
@@ -205,6 +204,16 @@ module.exports = {
     }
   },
 
+  /**
+   * Try to get a cached instance of the current component.
+   *
+   * @return {Vue|undefined}
+   */
+
+  getCached: function () {
+    return this.keepAlive && this.cache[this.Component.cid]
+  },
+
   /**
    * Teardown the current child, but defers cleanup so
    * that we can separate the destroy and removal steps.
