On 09 Jul 12:06, Jesús Martín Jiménez wrote: > 2014-07-09 12:04 GMT+02:00 Cédric Krier <cedric.kr...@b2ck.com>: > > > On 09 Jul 11:41, Jesús Martín Jiménez wrote: > > https://bitbucket.org/zikzakmedia/trytond-sale_margin/commits/a85edbe77cde8e99d2e60f28fd5337d80c5ff699#Lsale.pyT69 > > > > > > > > > > It was a refactored of the code, adding all existing dependences on an > > > inherited method. > > > > That's wrong. > > > > > Ok. But, what's wrong?
I don't know how to say it differentrly: You must not add depends on fields that the method doesn't depend on. What you do is just breaking the modularity. -- Cédric Krier - B2CK SPRL Email/Jabber: cedric.kr...@b2ck.com Tel: +32 472 54 46 59 Website: http://www.b2ck.com/
pgpZphfFPCGY0.pgp
Description: PGP signature