Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

MDL-31466 Tags Adding a capability check to make sure hidden courses …

…are not displayed in tag search results
  • Loading branch information...
commit ba184c9f7601cb5484549f3f0d1e5e7e7c33112a 1 parent c4a12af
@ankitagarwal ankitagarwal authored
Showing with 4 additions and 8 deletions.
  1. +4 −8 tag/coursetagslib.php
View
12 tag/coursetagslib.php
@@ -444,15 +444,11 @@ function coursetag_get_tagged_courses($tagid) {
$courses = array();
if ($crs = $DB->get_records_select('tag_instance', "tagid=:tagid AND itemtype='course'", array('tagid'=>$tagid))) {
foreach ($crs as $c) {
- //this capability check was introduced to stop display of courses that a student could not
- //view, but arguably it is best that when clicking on a tag, the tagged course summary should
- //be seen and then if the student clicks on that they will be given the opportunity to join
- //note courses not visible should not have their tagid sent to this function
- // $context = get_context_instance(CONTEXT_COURSE, $c->itemid);
- //if (is_enrolled($context) oe is_viewing($context)) {
- $course = $DB->get_record('course', array('id'=>$c->itemid));
+ $course = $DB->get_record('course', array('id'=>$c->itemid));
+ // check if the course is hidden
+ if ($course->visible == 1 || has_capability('moodle/course:viewhiddencourses', context_course::instance($course->id))) {
$courses[$c->itemid] = $course;
- //}
+ }
}
}
return $courses;
Please sign in to comment.
Something went wrong with that request. Please try again.