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

          Attachments

            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