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

Use observers instead of handlers in enrol_cohort

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. Create a new cohort and add 2 users to it
      2. Create a new course
      3. Enable the enrolment method Cohort sync and select the cohort previously created
      4. Make sure both users are enrolled in the course
      5. Remove one of the users from the cohort
      6. Make sure it is NOT enrolled in the course
      7. Re-add the users to the cohort
      8. Make sure it IS enrolled in the course
      9. Delete the cohort
      10. Make sure none of the users are enrolled in the course
      Show
      Create a new cohort and add 2 users to it Create a new course Enable the enrolment method Cohort sync and select the cohort previously created Make sure both users are enrolled in the course Remove one of the users from the cohort Make sure it is NOT enrolled in the course Re-add the users to the cohort Make sure it IS enrolled in the course Delete the cohort Make sure none of the users are enrolled in the course
    • Affected Branches:
      MOODLE_26_STABLE
    • Fixed Branches:
      MOODLE_26_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-42004-master
    • Story Points:
      5
    • Sprint:
      BACKEND Sprint 5

      Description

      They should have been converted along with the cohort_* events.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              fred Frédéric Massart
              Reporter:
              fred Frédéric Massart
              Peer reviewer:
              Adrian Greeve
              Integrator:
              Eloy Lafuente (stronk7)
              Tester:
              Jason Fowler
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona)
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                18/Nov/13