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, Ferran Recio, 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