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

MUC: can get a value from static cache after having purged it

    Details

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

      Description

      The following code generates the output "hello world". The expected output is "hello".

      <?php
      define('CLI_SCRIPT', true);
      require_once('config.php');
       
      $cache = cache::make_from_params(cache_store::MODE_REQUEST, 'foo', 'bar');
      $cache->set('hello', 'world');
      $cache->purge();
      echo 'hello ' . $cache->get('hello') . "\n";

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              brianking Brian King added a comment -

              Added master pull URL; this seems to fix it.

              Show
              brianking Brian King added a comment - Added master pull URL; this seems to fix it.
              Hide
              samhemelryk Sam Hemelryk added a comment -

              Hi Brian,

              Thanks for reporting, diagnosing, and providing a fix for this issue.
              Its of immense help when people are so proactive.

              I've verified the issue and have checked the fix which looks spot on so I'm putting this up for integration review now.

              Many thanks
              Sam

              Show
              samhemelryk Sam Hemelryk added a comment - Hi Brian, Thanks for reporting, diagnosing, and providing a fix for this issue. Its of immense help when people are so proactive. I've verified the issue and have checked the fix which looks spot on so I'm putting this up for integration review now. Many thanks Sam
              Hide
              samhemelryk Sam Hemelryk added a comment -

              I'm choosing to move this through integration myself as I'm very confident about this fix.

              Show
              samhemelryk Sam Hemelryk added a comment - I'm choosing to move this through integration myself as I'm very confident about this fix.
              Hide
              samhemelryk Sam Hemelryk added a comment -

              This has been integrated now thanks Brian.

              Show
              samhemelryk Sam Hemelryk added a comment - This has been integrated now thanks Brian.
              Hide
              samhemelryk Sam Hemelryk added a comment -

              Tested and passed during integration review.

              Show
              samhemelryk Sam Hemelryk added a comment - Tested and passed during integration review.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Did you know that your awesome changes are the last ones landing to CVS?

              After 11 years, 2 months and 8 days... it's ended, died, closed, kaputt. Big thanks to it, has served us really well.

              Closing this as fixed, many thanks!

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Did you know that your awesome changes are the last ones landing to CVS? After 11 years, 2 months and 8 days... it's ended, died, closed, kaputt. Big thanks to it, has served us really well. Closing this as fixed, many thanks!

                People

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

                  Dates

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