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:

      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.

        Gliffy Diagrams

          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: