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

Memcached assumes getAllKeys will always return a valid array

    XMLWordPrintable

    Details

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

      Description

      getAllKeys can return false when the operation fails.
      The new Memcached purge has the result of the function in a foreach loop without checking the value is not false.

      In practise this means during upgrades or when you hit "purge all cache" you get spammed with:
      Warning: Invalid argument supplied for foreach() in <blah>/cache/stores/memcached/lib.php on line 513

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              skylarkelty Skylar Kelty
              Reporter:
              skylarkelty Skylar Kelty
              Peer reviewer:
              Mark Nelson
              Integrator:
              David Monllaó
              Tester:
              Jun Pataleta
              Participants:
              Component watchers:
              Matteo Scaramuccia, Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Víctor Déniz Falcón
              Votes:
              1 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                23/May/16