Moodle

enrol_localcoursefield in enrol/ldap breaks enrol/database

Details

  • Type: Bug Bug
  • Status: Open Open
  • Priority: Minor Minor
  • Resolution: Unresolved
  • Affects Version/s: 1.8.1
  • Fix Version/s: None
  • Component/s: Enrolments
  • Labels:
    None
  • Affected Branches:
    MOODLE_18_STABLE

Description

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);

Activity

Hide
Michael Woods added a comment -

Just confirming that this is still an issue in 1.9.2+

Show
Michael Woods added a comment - Just confirming that this is still an issue in 1.9.2+

People

Vote (0)
Watch (3)

Dates

  • Created:
    Updated: