Skip to content
Browse files

Remove additional check when dealing with hidden courses in LDAP enro…

…lment.

The check is superfluous and uses an inexistant variable ($type) that
was removed when we switched to role based enrolment.

Fixes MDL-11098 MDL-10282 and MDL-8346. (backported from HEAD)
  • Loading branch information...
1 parent 062c59e commit e2a7235da83fb311a762b4edee30e758f582f70f iarenaza committed Sep 15, 2007
Showing with 1 addition and 1 deletion.
  1. +1 −1 enrol/ldap/enrol.php
View
2 enrol/ldap/enrol.php
@@ -76,7 +76,7 @@ function setup_enrolments(&$user) {
error_log("[ENROL_LDAP] User $user->username enrolled to a nonexistant course $course_ext_id \n");
}
} else { // the course object exists before we call...
- if ($course_obj->visible==0 && $user->{$type}[$course_obj->id] == 'ldap') {
+ if ($course_obj->visible==0) {
// non-visible courses don't show up in the enrolled
// array, so we should skip them --
continue;

0 comments on commit e2a7235

Please sign in to comment.
Something went wrong with that request. Please try again.