Moodle

Type in enrollib.php => enrol_sharing_course

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor 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

Activity

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

Up for integration now

Show
Sam Hemelryk added a comment - Up for integration now
Hide
Permalink
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
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
Aparup Banerjee added a comment -

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

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

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

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

Works Great
Thanks for fixing this Sam.

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

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

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

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: