diff options
| author | Jan-Hendrik Willms <tleilax+studip@gmail.com> | 2023-01-26 13:09:09 +0000 |
|---|---|---|
| committer | Ron Lucke <lucke@elan-ev.de> | 2023-01-26 13:09:09 +0000 |
| commit | 7b872489628f0157fb940c1ab0c01f26994455d1 (patch) | |
| tree | ae82bbb95a070af64c2483da191438ad212529c9 /lib/models/Courseware/StructuralElement.php | |
| parent | eb09fdeba1dd6259fe8a6beab1fe0413b74d8317 (diff) | |
prevent warning in CourseWare\StructuralElement::hasEditingPermission(), fixes #2070
Closes #2070
Merge request studip/studip!1346
Diffstat (limited to 'lib/models/Courseware/StructuralElement.php')
| -rw-r--r-- | lib/models/Courseware/StructuralElement.php | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/lib/models/Courseware/StructuralElement.php b/lib/models/Courseware/StructuralElement.php index dbe2e62..ba51238 100644 --- a/lib/models/Courseware/StructuralElement.php +++ b/lib/models/Courseware/StructuralElement.php @@ -373,9 +373,9 @@ class StructuralElement extends \SimpleORMap */ public function hasEditingPermission($user): bool { - return $GLOBALS['perm']->have_perm('root', $user->id) || - $GLOBALS['perm']->have_studip_perm( - \CourseConfig::get($this->range_id)->COURSEWARE_EDITING_PERMISSION[$this->getCoursewareCourse($this->range_id)->id], + return $GLOBALS['perm']->have_perm('root', $user->id) + || $GLOBALS['perm']->have_studip_perm( + \CourseConfig::get($this->range_id)->COURSEWARE_EDITING_PERMISSION[$this->getCoursewareCourse($this->range_id)->id] ?? 'tutor', $this->range_id, $user->id ); @@ -661,7 +661,7 @@ class StructuralElement extends \SimpleORMap $root = $this->findAncestors()[0]; } - return Unit::findOneBySQL('structural_element_id = ?', [$root->id]); + return Unit::findOneBySQL('structural_element_id = ?', [$root->id]); } /** |