Skip to content

Commit

Permalink
fix(nav): it is not allowed to pop all the views
Browse files Browse the repository at this point in the history
  • Loading branch information
manucorporat committed Nov 4, 2016
1 parent 5f1a862 commit 0ab990c
Showing 1 changed file with 14 additions and 3 deletions.
17 changes: 14 additions & 3 deletions src/navigation/nav-controller-base.ts
Expand Up @@ -248,14 +248,15 @@ export class NavControllerBase extends Ion implements NavController {
if (!ti) {
return false;
}
// set that this nav is actively transitioning
this.setTransitioning(true);

// Get entering and leaving views
const leavingView = this.getActive();
const enteringView = this._getEnteringView(ti, leavingView);

assert(leavingView || enteringView, 'Both leavingView and enteringView are null');
assert(leavingView || enteringView, 'both leavingView and enteringView are null');

// set that this nav is actively transitioning
this.setTransitioning(true);

// Initialize enteringView
if (enteringView && isBlank(enteringView._state)) {
Expand Down Expand Up @@ -328,6 +329,16 @@ export class NavControllerBase extends Ion implements NavController {
}

_postViewInit(enteringView: ViewController, leavingView: ViewController, ti: TransitionInstruction, resolve: TransitionResolveFn) {
assert(leavingView || enteringView, 'Both leavingView and enteringView are null');

if (!enteringView && !this._isPortal) {
console.warn(`You can't remove all the pages in the navigation stack. nav.pop() is probably called too many times.`,
this, this.getNativeElement());

ti.reject && ti.reject('navigation stack needs at least one root page');
return false;
}

const opts = ti.opts || {};
const insertViews = ti.insertViews;
const removeStart = ti.removeStart;
Expand Down

0 comments on commit 0ab990c

Please sign in to comment.