Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-34434 META: Collect together deprecated code changes for Moodle 2.6
  3. MDL-35009

Deprecate get_context_instance_by_id()

    XMLWordPrintable

    Details

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

      No real testing required. Just grep for get_context_instance_by_id and make sure it's only defined in lib/deprecationlib.php (and 1 occurrence in lib/tests/accesslib_test.php where the debugging behavior is covered).

      Show
      No real testing required. Just grep for get_context_instance_by_id and make sure it's only defined in lib/deprecationlib.php (and 1 occurrence in lib/tests/accesslib_test.php where the debugging behavior is covered).
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_26_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      wip-mdl-35009
    • Pull Master Diff URL:
    • Story Points:
      1
    • Sprint:
      BACKEND Sprint 2

      Description

      Deprecate get_context_instance_by_id()
      As discussed in MDL-34472 this function's usage was removed from core in 2.4 . But since this function was used everywhere, the deprecation warning was not added in 2.4 . The warning should be added to the function before 2.6, in this issue. Later the function definition should be removed from core in 2.8

      Thanks

        Attachments

          Issue Links

          blocks

          Task - A task that needs to be completed. Tasks usually refer to work that must be done outside the product. MDL-34550 Remove the method get_context_instance_by_id() from core

          • Minor - Minor loss of function where workaround is possible
          • Closed
          is blocked by

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-34549 Remove all calls to get_context_instance_by_id() from core

          • Minor - Minor loss of function where workaround is possible
          • Closed

            Activity

              People

              Assignee:
              rajeshtaneja Rajesh Taneja
              Reporter:
              ankit_frenz Ankit Agarwal
              Peer reviewer:
              Ankit Agarwal
              Integrator:
              Eloy Lafuente (stronk7)
              Tester:
              Mark Nelson
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Sara Arjona (@sarjona)
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

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