AZEN-SGG/Kavita
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Ensure that if a user doesn't have access to a person, person detail page redirects and informs them.

Browse source
This commit is contained in:
Joseph Milazzo 2024-12-08 10:32:18 -06:00
parent d522bccf86
commit 634b165318
2 changed files with 11 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

2
UI/Web/src/app/_services/person.service.ts
View file

@ -26,7 +26,7 @@ export class PersonService {
}
get(name: string) {
return this.httpClient.get<Person>(this.baseUrl + `person?name=${name}`);
return this.httpClient.get<Person | null>(this.baseUrl + `person?name=${name}`);
}
getRolesForPerson(personId: number) {

11
UI/Web/src/app/person-detail/person-detail.component.ts
View file

@ -39,6 +39,7 @@ import {translate, TranslocoDirective} from "@jsverse/transloco";
import {ChapterCardComponent} from "../cards/chapter-card/chapter-card.component";
import {ThemeService} from "../_services/theme.service";
import {DefaultModalOptions} from "../_models/default-modal-options";
import {ToastrService} from "ngx-toastr";
@Component({
selector: 'app-person-detail',
@ -74,6 +75,7 @@ export class PersonDetailComponent {
protected readonly imageService = inject(ImageService);
protected readonly accountService = inject(AccountService);
private readonly themeService = inject(ThemeService);
private readonly toastr = inject(ToastrService);
protected readonly TagBadgeCursor = TagBadgeCursor;
@ -104,7 +106,14 @@ export class PersonDetailComponent {
this.personName = personName;
return this.personService.get(personName);
}),
tap(person => {
tap((person) => {
if (person == null) {
this.toastr.error(translate('toasts.unauthorized-1'));
this.router.navigateByUrl('/home');
return;
}
this.person = person;
this.personSubject.next(person); // emit the person data for subscribers
this.themeService.setColorScape(person.primaryColor || '', person.secondaryColor);