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

Invalid definition of the core/groupdata persistent cache

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.5
    • Fix Version/s: 2.5
    • Component/s: Caching
    • Labels:
      None
    • Testing Instructions:
      Hide

      Testing difficulty: trivial

      1. Make sure the site is upgraded after the patch is applied (you may need to bump the version manually)
      2. Configure the site so that it displays MUC usage stats in the footer ("Caches interaction by definition then store")
      3. TEST: Make sure you can see the line "** static persist **" below the "core/groupdata" title.

      Show
      Testing difficulty: trivial 1. Make sure the site is upgraded after the patch is applied (you may need to bump the version manually) 2. Configure the site so that it displays MUC usage stats in the footer ("Caches interaction by definition then store") 3. TEST: Make sure you can see the line "** static persist **" below the "core/groupdata" title.
    • Affected Branches:
      MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_25_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-37673-muc-groupdata-typo

      Description

      There is a trivial typo introduced by MDL-34398 in lib/db/caches.php that makes groupdata cache not being persistent.

      'persist' => true,
      

      should read

      'persistent' => true,
      

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            mudrd8mz David Mudrak added a comment -

            I'm on it.

            Show
            mudrd8mz David Mudrak added a comment - I'm on it.
            Hide
            mudrd8mz David Mudrak added a comment -

            Submitting for integration.

            Please note that the patch does not include the main version bump intentionally to make the integration easier.

            Show
            mudrd8mz David Mudrak added a comment - Submitting for integration. Please note that the patch does not include the main version bump intentionally to make the integration easier.
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Good catch thanks David!

            Show
            samhemelryk Sam Hemelryk added a comment - Good catch thanks David!
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

            TIA and ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Thanks David, has been integrated now

            Show
            samhemelryk Sam Hemelryk added a comment - Thanks David, has been integrated now
            Hide
            fred Frédéric Massart added a comment -

            Tough test! Passed on master, thanks!

            Show
            fred Frédéric Massart added a comment - Tough test! Passed on master, thanks!
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            A brilliant future is awaiting us out there, better with your code. Let's look towards the future together, this is now closed.

            (and won't be revisiting it unless some regression is found)

            Thanks and ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - A brilliant future is awaiting us out there, better with your code. Let's look towards the future together, this is now closed. (and won't be revisiting it unless some regression is found) Thanks and ciao

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  14/May/13