[Tickets #8626] lib/Horde/SessionHandler/memcache.php

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/8626
------------------------------------------------------------------------------
  Ticket             | 8626
  Created By         | whatdoyouwant at gmail.com
  Summary            | lib/Horde/SessionHandler/memcache.php
  Queue              | Horde Base
  Version            | 3.3.5
  Type               | Bug
  State              | Unconfirmed
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


whatdoyouwant at gmail.com (2009-10-06 14:40) wrote:

destroy($id) does not return true if successfully deleted the session  
data from memcache.  This can be seen when turning on horde logging to  
DEBUG.  This was also present in 3.3.4

it's checking against false
should check against true like this:
line 253-256 of memcache.php (only line 253 changed to true instead of false):
         if ($result !== true) {
             Horde::logMessage('Failed to delete session (id = ' . $id  
. ')', __FILE__, __LINE__, PEAR_LOG_DEBUG);
             return false;
         }