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

EQUELLA repository passing through database ID for a course rather than the Course ID Number

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 2.3
    • Fix Version/s: 2.3
    • Component/s: Repositories
    • Labels:
      None

      Description

      The EQUELLA module and original repository implementation would pass through the Course ID Number as part of the integration for copyright activations in EQUELLA. During the rewrite to the new repository implementation, this had been changed from $COURSE->idnumber to $COURSE->id which is incorrect. Not only will the database IDs change, but they will not match existing course ID numbers in EQUELLA which will cause errors.

        Gliffy Diagrams

          Activity

          Hide
          nickread Nick Read added a comment -

          Added pull request

          Show
          nickread Nick Read added a comment - Added pull request
          Hide
          dougiamas Martin Dougiamas added a comment -

          I was wondering about that. OK.

          Show
          dougiamas Martin Dougiamas added a comment - I was wondering about that. OK.
          Hide
          dougiamas Martin Dougiamas added a comment -

          Dan, this is uncontroversial. Please integrate to 2.3

          Show
          dougiamas Martin Dougiamas added a comment - Dan, this is uncontroversial. Please integrate to 2.3
          Hide
          poltawski Dan Poltawski added a comment -

          Integrated, thanks

          Show
          poltawski Dan Poltawski added a comment - Integrated, thanks
          Hide
          poltawski Dan Poltawski added a comment -

          Going to check for regressions.

          Show
          poltawski Dan Poltawski added a comment - Going to check for regressions.
          Hide
          poltawski Dan Poltawski added a comment -

          Still works, hurrah.

          Show
          poltawski Dan Poltawski added a comment - Still works, hurrah.
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          Yay. just in time for Moodle 2.3 release! Many thanks!

          Closing as fixed, ciao

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - Yay. just in time for Moodle 2.3 release! Many thanks! Closing as fixed, ciao

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                25/Jun/12