[Tickets #12735] Re: Xcache not expiring on get() if prefix is set

[noreply at bugs.horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/12735
------------------------------------------------------------------------------
  Ticket             | 12735
  Updated By         | wbreyha at gmx.net
  Summary            | Xcache not expiring on get() if prefix is set
  Queue              | Horde Framework Packages
  Version            | Git master
  Type               | Bug
  State              | Unconfirmed
  Priority           | 3. High
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


wbreyha at gmx.net (2013-10-04 23:59) wrote:

I reconsidered my patch...

_setExpire is protected and used only in the Xcache Module for get()  
and exists(). Both add the prefix before calling _setExpire.

Removing the second addition of the prefix in _setExpire() seems more  
reasonable.

--------
--- Xcache.php.orig	2013-10-05 01:44:50.000000000 +0200
+++ Xcache.php	2013-10-05 01:55:58.000000000 +0200
@@ -97,7 +97,6 @@
              // don't expire
              return;
          }
-        $key = $this->_params['prefix'] . $key;
          $expire = xcache_get($key . '_expire');

          // set prune period
--------