fix(redirect): do not use Promise.reject on redirect

Closes #480
aurelia · May 12, 2018 · 487c33d · 487c33d
1 parent c08a318
commit 487c33d
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/navigation-plan.js b/src/navigation-plan.js
@@ -13,6 +13,9 @@ export class BuildNavigationPlanStep {
   run(navigationInstruction: NavigationInstruction, next: Function) {
     return _buildNavigationPlan(navigationInstruction)
       .then(plan => {
+        if (plan instanceof Redirect) {
+          return next.cancel(plan);
+        }
         navigationInstruction.plan = plan;
         return next();
       }).catch(next.cancel);
@@ -33,7 +36,7 @@ export function _buildNavigationPlan(instruction: NavigationInstruction, forceLi
         redirectLocation += '?' + instruction.queryString;
       }
 
-      return Promise.reject(new Redirect(redirectLocation));
+      return Promise.resolve(new Redirect(redirectLocation));
     });
   }