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

Refactor filters a bit once support for legacy filters and locations have been performed

    Details

    • Type: Task
    • Status: Closed
    • Priority: Blocker
    • Resolution: Duplicate
    • Affects Version/s: 2.3
    • Fix Version/s: None
    • Component/s: Filters
    • Labels:
    • Affected Branches:
      MOODLE_23_STABLE

      Description

      In 2.2 we marked all legacy filters and locations as deprecated (see MDL-29995 and release notes). And in 2.3 support for them has been dropped (see MDL-29996).

      So now it's safe to improve the filters plugin type to start using proper component names here and there. This issue is about to collect and perform the needed changes.

      Note that while it was originally planned for 2.4 (see comment @ MDL-27961) it's highly possible to do it for 2.3 (once MDL-29996 has been done), so I'm pointing to it as target.

      Ciao

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment - - edited

              Note for me, unrelated but here to forget:

              Also, look for get_config() uses looking for wrong component names (like auth/xxx and so on). Migrate all the uses and all the information in DB. Look if there is already one issue for that, creating it if necessary. Discovered while checking some get_config() uses @ MDL-31006

              This regexp will show a bunch of them:

              grep -r '[^>:]get_config(.*\/.*)' *

              Ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - - edited Note for me, unrelated but here to forget: Also, look for get_config() uses looking for wrong component names (like auth/xxx and so on). Migrate all the uses and all the information in DB. Look if there is already one issue for that, creating it if necessary. Discovered while checking some get_config() uses @ MDL-31006 This regexp will show a bunch of them: grep -r '[^>:]get_config(.*\/.*)' * Ciao
              Hide
              skodak Petr Skoda added a comment -

              The linked issue already contains the refactoring, thanks for the report.

              Show
              skodak Petr Skoda added a comment - The linked issue already contains the refactoring, thanks for the report.

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: