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

Remove 'not implemented' cache definitions before release

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.4
    • Fix Version/s: 2.4
    • Component/s: Caching
    • Labels:

      Description

      Hi Sam,

      We don't have 'config' cache in use, and its shown in the cache UI, so I was wondering if it'd be less 'misleading' if we remove this from lib/db/caches.php before release so it doesn't look like there is a cache on the config table and people start configuring their sites wondering why its not working.

        Gliffy Diagrams

          Activity

          Hide
          samhemelryk Sam Hemelryk added a comment -

          Yes, good thinking thanks Dan.

          Those definitions were left in there originally in order to avoid conflicts when we integrated the conversions themselves.
          Now that config has been delayed best we remove it and revert this when it is integrated.

          Many thanks
          Sam

          Show
          samhemelryk Sam Hemelryk added a comment - Yes, good thinking thanks Dan. Those definitions were left in there originally in order to avoid conflicts when we integrated the conversions themselves. Now that config has been delayed best we remove it and revert this when it is integrated. Many thanks Sam
          Hide
          samhemelryk Sam Hemelryk added a comment -

          Forgot to mention this is up for integration review now

          Show
          samhemelryk Sam Hemelryk added a comment - Forgot to mention this is up for integration review now
          Hide
          poltawski Dan Poltawski added a comment -

          Thanks Sam, i've integrated this now

          Show
          poltawski Dan Poltawski added a comment - Thanks Sam, i've integrated this now
          Hide
          poltawski Dan Poltawski added a comment -

          Looks good here, thanks Sam

          Show
          poltawski Dan Poltawski added a comment - Looks good here, thanks Sam
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          Y E S !

          Closing as fixed, many thanks!

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - Y E S ! Closing as fixed, many thanks!

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                3/Dec/12