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

Type in enrollib.php => enrol_sharing_course

          Details

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

            Developer required

            1. Log in as an admin
            2. Create two courses
            3. Create two users
            4. Enrol user 1 into course 1
            5. Enrol user 2 into course 2
            6. Write a small script to call enrol_sharing_course with the two user ids
            7. Ensure that you get an empty array.
            Show
            Developer required Log in as an admin Create two courses Create two users Enrol user 1 into course 1 Enrol user 2 into course 2 Write a small script to call enrol_sharing_course with the two user ids Ensure that you get an empty array.
          • Affected Branches:
            MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE
          • Fixed Branches:
            MOODLE_20_STABLE, MOODLE_21_STABLE
          • Pull from Repository:
          • Pull Master Branch:
            wip-MDL-29487-master
          • Pull Master Diff URL:

            Description

            Found a typo in enrol_sharing_course

              Gliffy Diagrams

                Attachments

                  Activity

                  Ascending order - Click to sort in descending order
                  Hide
                  Permalink
                  samhemelryk Sam Hemelryk added a comment -

                  Up for integration now

                  Show
                  samhemelryk Sam Hemelryk added a comment - Up for integration now
                  Hide
                  Permalink
                  samhemelryk Sam Hemelryk added a comment -

                  Attached a file that can be used to test this, simply put it in your root directory and call /enroltest.php?user1=x&user2=y where x is the first id and y is the second.

                  Show
                  samhemelryk Sam Hemelryk added a comment - Attached a file that can be used to test this, simply put it in your root directory and call /enroltest.php?user1=x&user2=y where x is the first id and y is the second.
                  Hide
                  Permalink
                  nebgor Aparup Banerjee added a comment -

                  Sam, the master branch seems to be broken on your repository.

                  Show
                  nebgor Aparup Banerjee added a comment - Sam, the master branch seems to be broken on your repository.
                  Hide
                  Permalink
                  nebgor Aparup Banerjee added a comment -

                  this has been integrated now, its a go for test.

                  Show
                  nebgor Aparup Banerjee added a comment - this has been integrated now, its a go for test.
                  Hide
                  Permalink
                  rajeshtaneja Rajesh Taneja added a comment -

                  Works Great
                  Thanks for fixing this Sam.

                  Show
                  rajeshtaneja Rajesh Taneja added a comment - Works Great Thanks for fixing this Sam.
                  Hide
                  Permalink
                  nebgor Aparup Banerjee added a comment -

                  fixes have been rolled merrily up the stream! Thanks everybody!

                  Show
                  nebgor Aparup Banerjee added a comment - fixes have been rolled merrily up the stream! Thanks everybody!

                    People

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

                      Dates

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