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

View file

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