openerp-expert-framework team mailing list archive

[Olivier Dony <odo@xxxxxxxxxxx>]

On 09/13/2011 04:42 PM, Thibaut DIRLIK wrote:
> I'm using the latest rev, just updated, same problem. I saw the commit
> of Antonny Lesuisse about the fix, but it doesn't fix the part of the
> code I talked about

Indeed, that's the last place where _get_ids.clear_cache() was being
called - Antony must have missed it too during the merge of the new cache.

Bit of context: the new cache implementation is faster and simpler, and
it supports the "multi" use case (@tools.ormcache_multi).
With the multi-valued version, the cache knows that one of the method
parameters is a list of keys, in which case the result is cached for
each key individually, allowing a cache hit next time any one of these
keys is requested.
However, contrary to the old cache implementation, this implementation
does not (yet) support the same multi-value behavior for the clearing of
the cache with cache_clear(), so the multiple keys need to be cleared
separately.

It should be fixed after revision 3588. Checked the rest of the code,
this was the last call to ormcache_multi.cache_clear() with a list of keys.

Thanks for spotting it! And since you asked: yes, this would of course
have been appreciated as a LP bug report, or a comment/reopen on bug
838864 :-)