On 23/08/10 08:41 +0200, Cédric Krier wrote: > On 22/08/10 21:51 +0200, Mathias Behrle wrote: > > * Betr.: " [tryton-dev] Circular import" (Sat, 21 Aug 2010 18:24:34 +0200): > > > > > To fix [1] issue1653 [2], I have moved Cache decorator from trytond.tools > > > into > > > trytond.model.cache. > > > I think it is the right way, because Cache usage is on Model methods. > > > > > > Any objections? > > > > No, seems much better to me than under tools. I would even prefer to have > > trytond.cache.cache . > > Yes, it seems better because when putting it in trytond.model I got also a > circular dependencies. > I have updated the codereview.
I needed also to move trytond.tools.convert to trytond.convert I think it was also logical as it is a part of the kernel of Tryton. -- Cédric Krier B2CK SPRL Rue de Rotterdam, 4 4000 Liège Belgium Tel: +32 472 54 46 59 Email/Jabber: [email protected] Website: http://www.b2ck.com/
pgpRj4HWyWnYV.pgp
Description: PGP signature
