Skip to content
Snippets Groups Projects
Commit 8830544b authored by David Siegfried's avatar David Siegfried
Browse files

prevent php-warnings in Institute-Class, fixes #4534

Closes #4534

Merge request studip/studip!3329
parent 4d5a9a7c
No related branches found
No related tags found
No related merge requests found
......@@ -311,12 +311,18 @@ class Institute extends SimpleORMap implements Range
public function isEditableByUser($user_id = null): bool
{
if ($user_id === null) {
$user_id = User::findCurrent()->id;
$user = User::findCurrent();
} else {
$user = User::find($user_id);
}
if (!$user) {
return false;
}
$member = $this->members->findOneBy('user_id', $user_id);
$member = $this->members->findOneBy('user_id', $user->id);
return ($member && $member->inst_perms === 'admin')
|| User::find($user_id)->perms === 'root';
|| $user->perms === 'root';
}
/**
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment