Controller cache API is missing

[Elliot Sprehn]

Originally posted by Elliot Sprehn on 5/17/2008 at 1:42 AM: entered by Ezra Parker on port to new system

The cache API from the modelglue.unity.controller.Controller is missing.

These functions are:

AddToCache()
ExistsInCache()
GetFromCache()
RemoveFromCache()

I was thinking that if these are implemented as part of the content cache then it should probably be documented that a prefix of "modelglue." on the key is appended, plus the controller name since the cache was separate for each controller?

Although, I could see wanting objects (ex. queries) to be cached by default for X amount of time and content cached with a different timeout. Would it be possible to just leave the old modelglue.util.TimedCache and defaultCacheTimeout?

Thanks.

[boomfish]

Pushed ticket to next milestone. We may consider pushing this to the 3.2 release.

[boomfish]

I've implemented the missing methods in my working copy according to the documentation in the Wiki. However while testing the methods I discovered a number of problems with Gesture's cache adapter.

I should have a solution ready in the next 24 hours, but it is likely to involve significant changes (fixes) to the cache adapter. If you'd rather I not commit such changes directly to SVN let me know and I'll post a patch instead.

The new methods use a cache prefix of "value." without the controller name. I did not see anything in the documentation that suggests that each controller should have its own cache. If a controller wants to keep its cached values separate it can add its own prefix.

I see the point about allowing the default timeout for values and content to be different. We'll look at that as part of the plan for a broader approach to caching that we hope to have ready for the 3.2 release.

[boomfish]

Fixed in SVN revision 278.

I also fixed a few other caching bugs, the biggest one preventing the cache adapter from expiring elements properly. To help make sure that I got the cache adapter fixes right, I expanded the code in TestSimpleTimedCache.cfc almost sixfold. TDD FTW!