javascript - NGXS State not changing - Stack Overflow

admin管理员组

文章数量:1026925

There might be someone who knows why NGXS state cannot change inside of HttpClient POST request.

ctx.patchState()

Only works outside HTTP POST request.

    @Action(SignIn)
    signin(ctx: StateContext<AppStateModel>, { payload }: SignIn) {

        // ctx.patchState({isLoggedIn:true}) this works!

        return this.api$.signin(payload)
            .pipe(
                tap((user: User) => {
                    console.log(user);
                    ctx.patchState({ isLoggedIn: true })

                })
            )
            .subscribe(
                (user: User) => {
                    ctx.patchState({ isLoggedIn: true })

                }
            )
    }

There might be someone who knows why NGXS state cannot change inside of HttpClient POST request.

ctx.patchState()

Only works outside HTTP POST request.

    @Action(SignIn)
    signin(ctx: StateContext<AppStateModel>, { payload }: SignIn) {

        // ctx.patchState({isLoggedIn:true}) this works!

        return this.api$.signin(payload)
            .pipe(
                tap((user: User) => {
                    console.log(user);
                    ctx.patchState({ isLoggedIn: true })

                })
            )
            .subscribe(
                (user: User) => {
                    ctx.patchState({ isLoggedIn: true })

                }
            )
    }

• javascript
• angular
• ngxs

Share Improve this question Follow edited Aug 26, 2018 at 19:28 R. Richards 25.2k1010 gold badges6666 silver badges6565 bronze badges asked Aug 26, 2018 at 19:20 Nacho CastilloNacho Castillo 5322 silver badges66 bronze badges 4

• Is your console.log statement from inside the tap being printed? – user184994 Commented Aug 26, 2018 at 19:32
• Yes, the data is printed, but state not change, only change when I call ctx.patchState() outside of subscribe. – Nacho Castillo Commented Aug 26, 2018 at 19:36
• You should really emit an action with payload that data were loaded. Also don't subscribe inside of the action. Ngxs will subscribe for you. – Martin Nuc Commented Aug 26, 2018 at 19:40
• Thank you!! that's work better, I do not know that ngxs subscribe the observables , thank you. – Nacho Castillo Commented Aug 26, 2018 at 19:46

Add a ment  | 

1 Answer 1

Sorted by: Reset to default 6

Actually, the state is changing, but you don't see it because you return subscription that hasn't been pleted. In other words - You'll see the action being dispatched once the subscription of the returned observable pletes.

As mentioned in the ments, the returned observable of the actions are being subscribed behind the scene, so there's no need to subscribe to it again.

After that being said, you can pass take(1) in the pipe.

What it does, it pletes the subscription of the observable after it triggered once.

    @Action(SignIn)
    signin(ctx: StateContext<AppStateModel>, { payload }: SignIn) {
        return this.api$.signin(payload)
            .pipe(
                take(1), // <-- Add that
                tap((user: User) => ctx.patchState({ isLoggedIn: true }))
            );
    }

There might be someone who knows why NGXS state cannot change inside of HttpClient POST request.

ctx.patchState()

Only works outside HTTP POST request.

    @Action(SignIn)
    signin(ctx: StateContext<AppStateModel>, { payload }: SignIn) {

        // ctx.patchState({isLoggedIn:true}) this works!

        return this.api$.signin(payload)
            .pipe(
                tap((user: User) => {
                    console.log(user);
                    ctx.patchState({ isLoggedIn: true })

                })
            )
            .subscribe(
                (user: User) => {
                    ctx.patchState({ isLoggedIn: true })

                }
            )
    }

There might be someone who knows why NGXS state cannot change inside of HttpClient POST request.

ctx.patchState()

Only works outside HTTP POST request.

    @Action(SignIn)
    signin(ctx: StateContext<AppStateModel>, { payload }: SignIn) {

        // ctx.patchState({isLoggedIn:true}) this works!

        return this.api$.signin(payload)
            .pipe(
                tap((user: User) => {
                    console.log(user);
                    ctx.patchState({ isLoggedIn: true })

                })
            )
            .subscribe(
                (user: User) => {
                    ctx.patchState({ isLoggedIn: true })

                }
            )
    }

• javascript
• angular
• ngxs

Share Improve this question Follow edited Aug 26, 2018 at 19:28 R. Richards 25.2k1010 gold badges6666 silver badges6565 bronze badges asked Aug 26, 2018 at 19:20 Nacho CastilloNacho Castillo 5322 silver badges66 bronze badges 4

• Is your console.log statement from inside the tap being printed? – user184994 Commented Aug 26, 2018 at 19:32
• Yes, the data is printed, but state not change, only change when I call ctx.patchState() outside of subscribe. – Nacho Castillo Commented Aug 26, 2018 at 19:36
• You should really emit an action with payload that data were loaded. Also don't subscribe inside of the action. Ngxs will subscribe for you. – Martin Nuc Commented Aug 26, 2018 at 19:40
• Thank you!! that's work better, I do not know that ngxs subscribe the observables , thank you. – Nacho Castillo Commented Aug 26, 2018 at 19:46

Add a ment  | 

1 Answer 1

Sorted by: Reset to default 6

Actually, the state is changing, but you don't see it because you return subscription that hasn't been pleted. In other words - You'll see the action being dispatched once the subscription of the returned observable pletes.

As mentioned in the ments, the returned observable of the actions are being subscribed behind the scene, so there's no need to subscribe to it again.

After that being said, you can pass take(1) in the pipe.

What it does, it pletes the subscription of the observable after it triggered once.

    @Action(SignIn)
    signin(ctx: StateContext<AppStateModel>, { payload }: SignIn) {
        return this.api$.signin(payload)
            .pipe(
                take(1), // <-- Add that
                tap((user: User) => ctx.patchState({ isLoggedIn: true }))
            );
    }

本文标签： javascriptNGXS State not changingStack Overflow