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

Steffen Kamper info at sk-typo3.de
Thu Nov 6 20:10:39 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:
>  set()
>  get()
>  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.
> 
> Regards,
> Karsten

+1 for that, same as for 4.x. These names are simple and tell exactly 
what they do.

vg Steffen


More information about the TYPO3-team-core mailing list