launchpad-dev team mailing list archive

[Jonathan Lange <jml@xxxxxxxxxxxxx>]

On Mon, Aug 16, 2010 at 4:16 PM, Julian Edwards
<julian.edwards@xxxxxxxxxxxxx> wrote:
> On Monday 16 August 2010 12:11:39 Robert Collins wrote:
>> In SQLBase classes, do not write __storm_invalidation__ hooks anymore:
>> just use cachedproperty, and its all taken care of for you. If you
>> do write a hook, be sure to upcall, or else cachedproperty caches
>> won't be cleared.
>>
>> If you are writing pure storm without SQLBase, either don't use
>> cachedproperty at all, or call clear_cachedproperties(self) in a storm
>> invalidation hook.
>
> Do you think it's a good use of our time to finally convert all our old-style
> model classes into pure Storm so we don't have this (and probably other)
> inconsistencies to have to remember?

You didn't ask me and it was ten days ago, but yes, I think so.
Perhaps on a "when you touch it, change it" basis. Programming is hard
enough with having to remember stupid things.

jml