[TYPO3-core] FYI: fixed a cache framework issue

Oliver Hader oliver at typo3.org
Thu Nov 6 13:55:03 CET 2008


Hi Karsten,

Karsten Dambekalns schrieb:
> Hi Ingo, everyone.
> 
> Ingo Renner wrote:
>> Fixed an inconsitency with between the name, the documentation and the
>> functionality of method loadByTag() in the new caching framework
> 
> Reasonable change. It made me think, and here is the new problem :)
> 
> First I wanted to rename the "findByTag" I added to FLOW3's cache to
> make this consistent. But then Robert said he disliked "load" itself, as
> that doesn't sounds as if it returns things.
> 
> I remember discussing this with Robert a while ago, not really falling
> in love with either get/set nor load/save.
> 
> So, to make a long story short (and as long as it s still possible to
> change this in 4.x code without too much work) - what about a unified
> refactoring of the cache frontend API to have this:

Only for the frontend API (AbstractCache)? Why not for the backend
(AbstractBackend)?

>  set()
>  get()
>  getByTag()

This means:
save() -> set()
load() -> get()
loadByTag() -> getByTag()

> I think it is important to have good names as well as consistency
> between 4.x and FLOW3/5.x in the "shared" parts.

Yes, of course!

olly
-- 
Oliver Hader
TYPO3 4.3 Release Manager


More information about the TYPO3-team-core mailing list