fix: fix login/register #172

Merged
jleibl merged 3 commits from bugfix/fix-login into main 2025-05-07 15:30:28 +00:00
2 changed files with 11 additions and 25 deletions

View file

@ -18,24 +18,21 @@ export class AuthService {
private authUrl = `${environment.apiUrl}/auth`; private authUrl = `${environment.apiUrl}/auth`;
private userUrl = `${environment.apiUrl}/users`; private userUrl = `${environment.apiUrl}/users`;
private currentUserSubject: BehaviorSubject<User | null>; userSubject: BehaviorSubject<User | null>;
public currentUser: Observable<User | null>;
constructor( constructor(
private http: HttpClient, private http: HttpClient,
private router: Router private router: Router
) { ) {
this.currentUserSubject = new BehaviorSubject<User | null>(this.getUserFromStorage()); this.userSubject = new BehaviorSubject<User | null>(this.getUserFromStorage());
this.currentUser = this.currentUserSubject.asObservable();
// Check if token exists and load user data
if (this.getToken()) { if (this.getToken()) {
this.loadCurrentUser(); this.loadCurrentUser();
} }
} }
public get currentUserValue(): User | null { public get currentUserValue(): User | null {
return this.currentUserSubject.value; return this.userSubject.value;
} }
login(loginRequest: LoginRequest): Observable<AuthResponse> { login(loginRequest: LoginRequest): Observable<AuthResponse> {
@ -54,7 +51,7 @@ export class AuthService {
logout(): void { logout(): void {
localStorage.removeItem(TOKEN_KEY); localStorage.removeItem(TOKEN_KEY);
localStorage.removeItem(USER_KEY); localStorage.removeItem(USER_KEY);
this.currentUserSubject.next(null); this.userSubject.next(null);
this.router.navigate(['/']); this.router.navigate(['/']);
} }
@ -83,7 +80,7 @@ export class AuthService {
private setUser(user: User): void { private setUser(user: User): void {
localStorage.setItem(USER_KEY, JSON.stringify(user)); localStorage.setItem(USER_KEY, JSON.stringify(user));
this.currentUserSubject.next(user); this.userSubject.next(user);
} }
private getUserFromStorage(): User | null { private getUserFromStorage(): User | null {

View file

@ -10,7 +10,6 @@ import { RouterModule } from '@angular/router';
import { AuthService } from '@service/auth.service'; import { AuthService } from '@service/auth.service';
import { Subscription } from 'rxjs'; import { Subscription } from 'rxjs';
import { AnimatedNumberComponent } from '@blackjack/components/animated-number/animated-number.component'; import { AnimatedNumberComponent } from '@blackjack/components/animated-number/animated-number.component';
import { UserService } from '@service/user.service';
@Component({ @Component({
selector: 'app-navbar', selector: 'app-navbar',
@ -24,29 +23,19 @@ export class NavbarComponent implements OnInit, OnDestroy {
private authService: AuthService = inject(AuthService); private authService: AuthService = inject(AuthService);
isLoggedIn = this.authService.isLoggedIn(); isLoggedIn = this.authService.isLoggedIn();
private userSubscription: Subscription | undefined; private authSubscription!: Subscription;
private authSubscription: Subscription | undefined;
public balance = signal(0); public balance = signal(0);
private userService = inject(UserService);
ngOnInit() { ngOnInit() {
// Subscribe to auth changes this.authSubscription = this.authService.userSubject.subscribe({
this.authSubscription = this.authService.currentUser.subscribe((user) => { next: (user) => {
this.isLoggedIn = !!user; this.balance.set(user?.balance ?? 0);
this.balance.set(user?.balance ?? 0); },
console.log('Updated navbar balance:', user?.balance);
}); });
this.userService.refreshCurrentUser();
} }
ngOnDestroy() { ngOnDestroy() {
if (this.userSubscription) { this.authSubscription.unsubscribe();
this.userSubscription.unsubscribe();
}
if (this.authSubscription) {
this.authSubscription.unsubscribe();
}
} }
logout() { logout() {