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

filter_activitynames can be improved after dropping of global text cache

    XMLWordPrintable

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.6
    • 2.7
    • Caching, Filters
    • MOODLE_26_STABLE
    • MOODLE_27_STABLE
    • w13_MDL-44368_m27_actfilter
    • Hide

      1/ enable activity filtering
      2/ create some activities in course
      3/ use the names of activities in description
      4/ verify that hidden activities are not linked
      5/ verify the groupmembersonly activities are linked only if you can access them

      Show
      1/ enable activity filtering 2/ create some activities in course 3/ use the names of activities in description 4/ verify that hidden activities are not linked 5/ verify the groupmembersonly activities are linked only if you can access them
    • 20
    • BACKEND Sprint 12

    Description

      As a follow-on to MDL-43524 which removed global text caching, we need to examine the activity names filter and probably add MUC caching there to improve performance.

      The activitynames filter is what puts links in texts to other activities in the same course.

      Petr has raised that the fully correct way would be to use new events or the as-yet-unwritten hooks to invalidate the caches, but I think that might be too much to do before 2.7, so I want to see a quick solution here even if it may mean relying on less-perfect cache invalidation.

      Tests showing performance before/after are required.

      Attachments

        Issue Links

          Activity

            People

              skodak Petr Skoda
              dougiamas Martin Dougiamas
              Marina Glancy Marina Glancy
              Sam Hemelryk Sam Hemelryk
              Jetha Chan Jetha Chan
              Matteo Scaramuccia, David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo, David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                12/May/14