Issue Details (XML | Word | Printable)

Key: MDL-10329
Type: Bug Bug
Status: Open Open
Priority: Minor Minor
Assignee: Martin Dougiamas
Reporter: Clinton Graham
Votes: 0
Watchers: 3
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Moodle

enrol_localcoursefield in enrol/ldap breaks enrol/database

Created: 03/Jul/07 08:58 PM   Updated: 07/Aug/09 12:20 PM
Component/s: Enrolments
Affects Version/s: 1.8.1
Fix Version/s: None

Participants: Clinton Graham, Martin Dougiamas and Michael Woods
Security Level: None
Affected Branches: MOODLE_18_STABLE


 Description  « Hide
The config setting enrol_localcoursefield is not used by enrol/ldap except to be unset at enrol/ldap/enrol.php, lines 390-393, version 1.20.

Since enrol/database uses still uses this nomenclature, saving the enrol/ldap config blanks this required field for enrol/database and breaks database enrollment.

I think these lines should be removed:
390 if (!isset ($config->enrol_localcoursefield)) { 391 $config->enrol_localcoursefield = ''; 392 }
393 set_config('enrol_localcoursefield', $config->enrol_localcoursefield);



 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Michael Woods added a comment - 07/Aug/09 12:20 PM
Just confirming that this is still an issue in 1.9.2+