Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.4
    • Fix Version/s: 2.4
    • Component/s: Libraries
    • Labels:
    • Testing Instructions:
      Hide

      Test 1

      1. Browse through the enrolment options
      2. Create a mess, do whatever you want
      • Make sure no exceptions are raised on any page

      (If an exception is raised, make sure it is related to get_context_instance() before failing the test)

      Easter bunny Test (optional)

      1. Find the White-striped easter egg
      2. Make sure Moodle works
      Show
      Test 1 Browse through the enrolment options Create a mess, do whatever you want Make sure no exceptions are raised on any page (If an exception is raised, make sure it is related to get_context_instance() before failing the test) Easter bunny Test (optional) Find the White-striped easter egg Make sure Moodle works
    • Affected Branches:
      MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-34461-master-integration
    • Rank:
      42876

      Description

      Replace get_context_instance with context_XXXX::instance() in set location (group 5)
      Location

      • enrol/

        Activity

        Hide
        Dan Poltawski added a comment -

        Thanks Fred, this has been integrated now.

        Show
        Dan Poltawski added a comment - Thanks Fred, this has been integrated now.
        Hide
        David Monllaó added a comment -

        Tested playing with the enrolment plugins settings pages and enrolling users using the methods guest access, self enrolment, cohort IMS enterprise file and course meta link. No errors nor problems, it passes.

        Show
        David Monllaó added a comment - Tested playing with the enrolment plugins settings pages and enrolling users using the methods guest access, self enrolment, cohort IMS enterprise file and course meta link. No errors nor problems, it passes.
        Hide
        Aparup Banerjee added a comment -

        yay, it works!

        This issue has been put through rigorous processes and finally swam upstream along with some 65 others this week.

        Thank you all for taking the time to get us here.

        cheers!

        Show
        Aparup Banerjee added a comment - yay, it works! This issue has been put through rigorous processes and finally swam upstream along with some 65 others this week. Thank you all for taking the time to get us here. cheers!

          People

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

            Dates

            • Created:
              Updated:
              Resolved: