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

Looks like purge_current_user is a little aggressive

    XMLWordPrintable

    Details

      Description

      Inside cache_session class, there is this:

          /**
           * Purges the session cache of all data belonging to the current user.
           */
          public function purge_current_user() {
              $keys = $this->get_store()->find_all($this->get_key_prefix());
              $this->get_store()->delete_many($keys);
          }
      

      The find_all method doesn't take any arguments. I believe this should be find_by_prefix instead.

        Attachments

          Activity

            People

            Assignee:
            bushido Mark Nielsen
            Reporter:
            bushido Mark Nielsen
            Peer reviewer:
            Sam Hemelryk
            Integrator:
            Eloy Lafuente (stronk7)
            Tester:
            Frédéric Massart
            Participants:
            Component watchers:
            Matteo Scaramuccia, Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona)
            Votes:
            1 Vote for this issue
            Watchers:
            6 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              10/Mar/14