2014-07-09 12:49 GMT+02:00 Cédric Krier <cedric.kr...@b2ck.com>: > On 09 Jul 12:31, Jesús Martín Jiménez wrote: > > 2014-07-09 12:22 GMT+02:00 Cédric Krier <cedric.kr...@b2ck.com>: > > > > > 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. > > > > > > > But all those depends are already in the core module [1]. Do you mean is > > wrong repeat them? > > It is in other module not your. > > Ok. Understood.
-- Jesús Martín Zikzakmedia SL Dr. Fleming, 28, baixos 08720 Vilafranca del Penedès ☏ 93 890 21 08