Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-9411

MDL-9012 has partially wrong fix

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 1.7.2
    • 1.7.3
    • Enrolments
    • None
    • Any
    • MOODLE_17_STABLE
    • MOODLE_17_STABLE

    Description

      The patch provided is partially wrong. I copy & pasted the code from sync_enrolments and forgot to adjust a few variable names.

      As a result, there are several places where we use $extcourse which is not defined in the contextd it's used, so we use an empty value for several critical course fields ($CFG->enrol_localcoursefield, fullname and shortname.).

      So we need to rename all those $extcourse to $coursefield. The attached patch does it.

      Sorry for the (big) mistake. Btw, we need to apply this to HEAD too.

      Saludos. Iñaki.

      Attachments

        Issue Links

          Activity

            People

              skodak Petr Skoda
              iarenaza Iñaki Arenaza
              Nobody Nobody
              David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                11/Oct/07