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

enrol_database requires idnumber in external courses

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0, 2.1
    • Fix Version/s: 2.0.5, 2.1.2
    • Component/s: Enrolments
    • Labels:
    • Testing Instructions:
      Hide

      1/ set up external enrol course table
      2/ add course with and without idnumbers
      3/ make sure empty idnumbers are accepted

      Show
      1/ set up external enrol course table 2/ add course with and without idnumbers 3/ make sure empty idnumbers are accepted
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE
    • Pull from Repository:

      Description

      I have discovered this while testing MDL-29020

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              skodak Petr Skoda added a comment -

              To integrators: patch for master is included in MDL-29020

              Show
              skodak Petr Skoda added a comment - To integrators: patch for master is included in MDL-29020
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              See silly question @ MDL-29020...

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - See silly question @ MDL-29020 ...
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Integrated, thanks!

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Integrated, thanks!
              Hide
              samhemelryk Sam Hemelryk added a comment -

              Presuming I tested this correctly it passes, perhaps in the future more detailed testing instructions would help

              Show
              samhemelryk Sam Hemelryk added a comment - Presuming I tested this correctly it passes, perhaps in the future more detailed testing instructions would help
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Many thanks for the hard work, this has been sent upstream and is available in all the git and cvs repositories.

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Many thanks for the hard work, this has been sent upstream and is available in all the git and cvs repositories.

                People

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

                  Dates

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