[dev] [cvs] commit: imp/lib/IMAP MailboxCache.php

Chuck Hagenbuch chuck at horde.org
Fri Mar 9 03:27:17 UTC 2007


Quoting Michael M Slusarz <slusarz at horde.org>:

>> Is there any advantage to this? The old code would work regardless,
>> and introducing if checks based on Horde versions is fragile and just
>> hard to maintain. I'd _strongly_ prefer to avoid it.
>
> Of course.  Horde_Cache_file:: is completely unusable otherwise for
> development purposes.  i.e. you can't switch backends transparently
> (for example, zps4 driver accepts an array but file driver doesn't).

Well, the API may have been unclear, but in this case the file backend  
is right - the cache should only accept a string. And I'm pretty sure  
we don't pass non-string data to the cache at the moment.

-chuck

-- 
"we are plastered to the windshield of the bus that is time." - Chris


More information about the dev mailing list