Moodle
  1. Moodle
  2. MDL-37225

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

    Details

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

      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";
      

        Issue Links

          Activity

          Hide
          Brian King added a comment -

          Added master pull URL; this seems to fix it.

          Show
          Brian King added a comment - Added master pull URL; this seems to fix it.
          Hide
          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
          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
          Sam Hemelryk added a comment -

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

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

          This has been integrated now thanks Brian.

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

          Tested and passed during integration review.

          Show
          Sam Hemelryk added a comment - Tested and passed during integration review.
          Hide
          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
          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: