Moodle
  1. Moodle
  2. MDL-29995

Mark all the legacy filters support as deprecated for 2.2

    Details

    • Type: Task Task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2
    • Fix Version/s: 2.2
    • Component/s: Filters
    • Labels:
    • Testing Instructions:
      Hide

      No testing needed, just documentation and some comments and phpdocs in code, the only post-integration task is to link from the release notes to the new filter/upgrade.txt file.

      Show
      No testing needed, just documentation and some comments and phpdocs in code, the only post-integration task is to link from the release notes to the new filter/upgrade.txt file.
    • Affected Branches:
      MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE
    • Pull from Repository:
    • Pull Master Branch:
    • Rank:
      19539

      Description

      Once we have moved all core filters to the new proper ones, it's time to mark all places in code relating to legacy filters and legacy locations as deprecated. The (long) plan is to:

      • In 2.2 they are in deprecated status so will continue working (this issue).
      • In 2.3 they won't work anymore, all deprecated filters and locations will be ignored.
      • In 2.4 (or perhaps 2.3 too) some filters stuff will be moved to use proper component names instead of current, imperfect, 'filter/xxxx'.

      Back to this issue, we'll be adding comments along codebase about the deprecation and commenting in the release notes.

        Issue Links

          Activity

          Hide
          Eloy Lafuente (stronk7) added a comment -

          This requires changes in code and commenting in release notes.

          Show
          Eloy Lafuente (stronk7) added a comment - This requires changes in code and commenting in release notes.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Sending to integration, please, if approved, link to the new upgrade.txt file from the 2.2 release notes @ http://docs.moodle.org/dev/Moodle_2.2_release_notes#Plugin_API_changes

          Ciao

          Show
          Eloy Lafuente (stronk7) added a comment - Sending to integration, please, if approved, link to the new upgrade.txt file from the 2.2 release notes @ http://docs.moodle.org/dev/Moodle_2.2_release_notes#Plugin_API_changes Ciao
          Hide
          Aparup Banerjee added a comment -

          This has been integrated. (Added MDL-29998 to a Todo missing MDL number)

          I've appended to http://docs.moodle.org/dev/Moodle_2.2_release_notes#Plugin_API_changes

          (which is broken until rolling upstream)

          Show
          Aparup Banerjee added a comment - This has been integrated. (Added MDL-29998 to a Todo missing MDL number) I've appended to http://docs.moodle.org/dev/Moodle_2.2_release_notes#Plugin_API_changes https://github.com/moodle/moodle/blob/master/filter/upgrade.txt (which is broken until rolling upstream)
          Hide
          Rajesh Taneja added a comment -

          Thanks Eloy, for updating docs.
          No testing done (as mentioned)

          Show
          Rajesh Taneja added a comment - Thanks Eloy, for updating docs. No testing done (as mentioned)
          Hide
          Eloy Lafuente (stronk7) added a comment -

          And this has landed upstream, just on time for the upcoming new releases week. Thanks for it!

          Ciao

          Show
          Eloy Lafuente (stronk7) added a comment - And this has landed upstream, just on time for the upcoming new releases week. Thanks for it! Ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved: