From bc798b182dab1dd71cf257b8e8fe50c650796096 Mon Sep 17 00:00:00 2001 From: Brian Ford Date: Tue, 16 Jun 2015 09:46:11 -0700 Subject: [PATCH] fix(router): return promise with error handler See https://github.com/angular/angular/pull/2528\#discussion_r32493195 --- modules/angular2/src/router/router.ts | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/modules/angular2/src/router/router.ts b/modules/angular2/src/router/router.ts index a624fa3bfd..9cf204ac42 100644 --- a/modules/angular2/src/router/router.ts +++ b/modules/angular2/src/router/router.ts @@ -128,12 +128,10 @@ export class Router { ObservableWrapper.callNext(this._subject, matchedInstruction.accumulatedUrl); }); - PromiseWrapper.catchError(result, (err) => { + return PromiseWrapper.catchError(result, (err) => { this._finishNavigating(); - return err; + throw err; }); - - return result; }); }