diff options
| author | Jan-Hendrik Willms <tleilax+studip@gmail.com> | 2022-07-06 13:25:06 +0000 |
|---|---|---|
| committer | Jan-Hendrik Willms <tleilax+studip@gmail.com> | 2022-07-06 13:25:06 +0000 |
| commit | 89512696839235e4b3d2ca135917ae038ff5fc3c (patch) | |
| tree | e7870c1f8313dc39836745408dbb7a01f73a9b4e /lib/modules/LtiToolModule.class.php | |
| parent | a5813cb77af590aaf2c49aea66f9d33d47b066f6 (diff) | |
fixes 'Method/function should return XY but return statement is missing', fixes #1278
Closes #1278
Merge request studip/studip!783
Diffstat (limited to 'lib/modules/LtiToolModule.class.php')
| -rw-r--r-- | lib/modules/LtiToolModule.class.php | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/lib/modules/LtiToolModule.class.php b/lib/modules/LtiToolModule.class.php index 3093615..0383f11 100644 --- a/lib/modules/LtiToolModule.class.php +++ b/lib/modules/LtiToolModule.class.php @@ -55,6 +55,10 @@ class LtiToolModule extends CorePlugin implements StudipModule, SystemPlugin, Pr */ public function getTabNavigation($course_id) { + if ($GLOBALS['user']->id === 'nobody') { + return []; + } + $title = CourseConfig::get($course_id)->LTI_TOOL_TITLE; $grades = LtiData::countBySQL('course_id = ?', [$course_id]); @@ -67,9 +71,7 @@ class LtiToolModule extends CorePlugin implements StudipModule, SystemPlugin, Pr $navigation->addSubNavigation('grades', new Navigation(_('Ergebnisse'), 'dispatch.php/course/lti/grades')); } - if ($GLOBALS['user']->id !== 'nobody') { - return ['lti' => $navigation]; - } + return ['lti' => $navigation]; } /** |