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

MDL-9012 has partially wrong fix

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.7.2
    • Fix Version/s: 1.7.3
    • Component/s: Enrolments
    • Labels:
      None
    • Database:
      Any
    • Affected Branches:
      MOODLE_17_STABLE
    • Fixed Branches:
      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

              • Assignee:
                skodak Petr Skoda
                Reporter:
                iarenaza Iñaki Arenaza
                Tester:
                Nobody
                Participants:
                Component watchers:
                Amaia Anabitarte, Carlos Escobedo, Sara Arjona (@sarjona), Víctor Déniz Falcón
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  11/Oct/07