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

Allow non-default cache stores to be uninstalled

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide

      On a fresh Moodle install:

      1. Go to plugins overview page
      2. Confirm that file, session and static plugins can't be deleted, but that others can
      3. Set up a memcache(d) or mongodb store
      4. Go to plugins overview page and confirm that the plugin you used in the previous step can no longer be uninstalled
      Show
      On a fresh Moodle install: Go to plugins overview page Confirm that file, session and static plugins can't be deleted, but that others can Set up a memcache(d) or mongodb store Go to plugins overview page and confirm that the plugin you used in the previous step can no longer be uninstalled
    • Affected Branches:
      MOODLE_30_STABLE
    • Fixed Branches:
      MOODLE_29_STABLE, MOODLE_30_STABLE
    • Pull Master Branch:

      Description

      The cache plugin type prevents any store plugin from being uninstalled, even though only 3 of them are required (file, session, static). This has some negative consequences for server administrators, not least that third party store plugins that are no longer used (or possibly even supported) can't be removed.

      It would be much better if only the required default stores couldn't be uninstalled.

      It seems particularly strange that this is prevented given that Moodle continues to work as normal even if a plugin is uninstalled while in use (give or take a debug message or two)!

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                maherne Michael Aherne
                Reporter:
                maherne Michael Aherne
                Peer reviewer:
                David Monllaó
                Integrator:
                Dan Poltawski
                Tester:
                Rajesh Taneja
                Participants:
                Component watchers:
                Andrew Nicols, Mathew May, Michael Hawkins, Shamim Rezaie, Simey Lameze, Matteo Scaramuccia, Amaia Anabitarte, Bas Brands, Carlos Escobedo, Sara Arjona (@sarjona), Víctor Déniz Falcón
              • Votes:
                2 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  14/Mar/16