Skip to content
This repository has been archived by the owner on Dec 2, 2022. It is now read-only.

Commit

Permalink
Fix: 'courselistteachericon' not showing.
Browse files Browse the repository at this point in the history
  • Loading branch information
gjb2048 committed Feb 1, 2019
1 parent 627cb61 commit b47ce10
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGES.txt
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ New in 3.4.1.5
- FIX: Issue #888: Margins on block icons.
- FIX: Participants menu only on course and module pages.
- FIX: Administration / Naviagtion menu still appearing when flat navigation is on because they are not in the side-pre region.
- FIX: 'courselistteachericon' not showing.
- NEW: Show / hide the participants menu with the 'participantsmenu' setting in the 'Header' settings.
- NEW: Separate 'People' from 'This course' menu and create 'Participants' menu.

Expand Down
2 changes: 1 addition & 1 deletion classes/core_course_renderer.php
Original file line number Diff line number Diff line change
Expand Up @@ -245,7 +245,7 @@ protected function coursecat_coursebox_content(coursecat_helper $chelper, $cours
}
foreach ($course->get_course_contacts() as $userid => $coursecontact) {
$faiconsetting = \theme_essential\toolbox::get_setting('courselistteachericon');
$faiconsettinghtml = (empty($faiconsetting)) ? '' : '<span aria-hidden="true" class="'.
$faiconsettinghtml = (empty($faiconsetting)) ? '' : '<span aria-hidden="true" class="fa fa-'.
$faiconsetting.'"></span> ';
$name = $faiconsettinghtml.$coursecontact['rolename'].': '.
html_writer::link(new moodle_url('/user/view.php',
Expand Down

0 comments on commit b47ce10

Please sign in to comment.