Angular4 router.navigate doesn't do anything

Question

I have the conditional navigation based on customers role in the system. Something like this:

this.GetQuickStartStatus()
                    .subscribe(data => {
                            if(data.isActive){
                                this.storageService.Store('isQuick', "true");
                            }
                            else{
                                this.storageService.Store('isQuick', "false");
                            }

                            if(this.storageService.Retrieve("Role") === "user"){
                                console.log("package");
                                this.router.navigate(['/packages']);
                            }

                            if(this.storageService.Retrieve("Role") === "subscriber" && this.storageService.Retrieve("isQuick") === "false"){
                                console.log("quickstart");
                                this.router.navigate(['/quickstart']);
                            }
                            console.log("nothing");
                            this.router.navigate(['']);
                    }, (error) => {
                        console.log(error);
                    });

I log with the new user which should redirect to packages page since users role is "user. I get his in the console:

auth.service.ts:156 package
auth.service.ts:164 nothing

The meaning condition gets hit, but doesn't do anything aka router doesn't navigate to the requested page. I checked all conditions are met and all values are stored in localstorage. Why would router.navigate ignore redirection?

Routing module for packages:

@NgModule({
  imports: [
    RouterModule.forChild([
        { path: 'packages', component: PackageComponent, canActivate: [AuthGuardService] },
        { path: 'receipt', component: PackageReceiptComponent, canActivate: [AuthGuardService, RoleSubscriberGuardService] },
        { path: 'receipt/:token', component: PackageReceiptComponent, canActivate: [AuthGuardService, RoleSubscriberGuardService]}
    ])
  ],
  exports: [RouterModule]
})
export class PackageRoutingModule { }

Problem isn't located here: this.router.navigate(['']); ? This line is executed every time. Try to commend it or wrap with else — Maciej Treder
– Maciej Treder, Commented Mar 20, 2018 at 9:05
@MaciejTreder can you please put is an answer, that was it, not sure how it worked in first try probably because of cache. clean your cache people. — sensei
– sensei, Commented Mar 20, 2018 at 10:06

Maciej Treder · Accepted Answer · 2018-03-20 10:37:39Z

1

You have this.router.navigate(['']); at the end of your method - which is executed every time.

Try to wrap this line in the else block.

answered Mar 20, 2018 at 10:37

Maciej Treder

12.4k6 gold badges58 silver badges77 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Angular4 router.navigate doesn't do anything

1 Answer 1

Comments

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Related