[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