mirrors/hedgedoc
1
0
Fork 0
mirror of https://github.com/hedgedoc/hedgedoc.git synced 2025-05-25 04:24:43 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge pull request #1851 from hedgedoc/fix-permissions-types

Browse source
This commit is contained in:
David Mehren 2021-11-29 22:54:26 +01:00 • committed by GitHub
commit b4ce0dc889
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

12
src/permissions/permissions.service.ts
View file

@ -21,7 +21,7 @@ export enum GuestPermission {
@Injectable() @Injectable()
export class PermissionsService { export class PermissionsService {
public guestPermission: GuestPermission; // TODO change to configOption public guestPermission: GuestPermission; // TODO change to configOption
mayRead(user: User, note: Note): boolean { mayRead(user: User | null, note: Note): boolean {
if (this.isOwner(user, note)) return true; if (this.isOwner(user, note)) return true;
if (this.hasPermissionUser(user, note, false)) return true; if (this.hasPermissionUser(user, note, false)) return true;
@ -32,7 +32,7 @@ export class PermissionsService {
return false; return false;
} }
mayWrite(user: User, note: Note): boolean { mayWrite(user: User | null, note: Note): boolean {
if (this.isOwner(user, note)) return true; if (this.isOwner(user, note)) return true;
if (this.hasPermissionUser(user, note, true)) return true; if (this.hasPermissionUser(user, note, true)) return true;
@ -43,7 +43,7 @@ export class PermissionsService {
return false; return false;
} }
mayCreate(user: User): boolean { mayCreate(user: User | null): boolean {
if (user) { if (user) {
return true; return true;
} else { } else {
@ -58,14 +58,14 @@ export class PermissionsService {
return false; return false;
} }
isOwner(user: User, note: Note): boolean { isOwner(user: User | null, note: Note): boolean {
if (!user) return false; if (!user) return false;
if (!note.owner) return false; if (!note.owner) return false;
return note.owner.id === user.id; return note.owner.id === user.id;
} }
private hasPermissionUser( private hasPermissionUser(
user: User, user: User | null,
note: Note, note: Note,
wantEdit: boolean, wantEdit: boolean,
): boolean { ): boolean {
@ -84,7 +84,7 @@ export class PermissionsService {
} }
private hasPermissionGroup( private hasPermissionGroup(
user: User, user: User | null,
note: Note, note: Note,
wantEdit: boolean, wantEdit: boolean,
): boolean { ): boolean {