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, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze, Matteo Scaramuccia, Amaia Anabitarte, Carlos Escobedo, Ferran Recio, 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