From 2eb27c9cb7d316e4665ac9fda78fa79836544427 Mon Sep 17 00:00:00 2001 From: Michael M Slusarz Date: Tue, 6 Oct 2009 16:13:10 -0600 Subject: [PATCH] Bug #8626: Fix updating memcache tracking info on session destroy --- .../SessionHandler/lib/Horde/SessionHandler/Memcache.php | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/framework/SessionHandler/lib/Horde/SessionHandler/Memcache.php b/framework/SessionHandler/lib/Horde/SessionHandler/Memcache.php index f7209ff58..a82aecdbe 100644 --- a/framework/SessionHandler/lib/Horde/SessionHandler/Memcache.php +++ b/framework/SessionHandler/lib/Horde/SessionHandler/Memcache.php @@ -234,16 +234,15 @@ class Horde_SessionHandler_Memcache extends Horde_SessionHandler $result = $this->_memcache->delete($id); $this->_memcache->unlock($id); - if ($result !== false && - isset($this->_persistent)) { - $result = $this->_persistent->destroy($id); - } - - if ($result !== false) { + if ($result === false) { Horde::logMessage('Failed to delete session (id = ' . $id . ')', __FILE__, __LINE__, PEAR_LOG_DEBUG); return false; } + if (isset($this->_persistent)) { + $result = $this->_persistent->destroy($id); + } + if (!empty($this->_params['track'])) { $this->_memcache->lock($this->_trackID); $ids = $this->_memcache->get($this->_trackID); -- 2.11.0