openerp-community-reviewer team mailing list archive

[Lorenzo Battistini - Agile BG <lorenzo.battistini@xxxxxxxxxxx>]

On 12/05/2013 11:49 PM, Alexis de Lattre wrote:
> @Lorenzo :
>
> I see that we agree on points 1, 3 and 4.
>
> My answer on point 2 : after reflexion, I eventually think that a fields.function is fine. It will be OK for me to use another many2many field for the link between maintenance invoice lines and prodlots, or inherit the fields.function. And if you have a perf problem, we can always add a store={} to the current fields.function.

Sorry, I thought you also wanted to increase performances.
For me, for now, it can stay a normal function.


> By the way, do you want me to sumbit a patch for points 1, 3 and 4, or do you plan to do it yourself ?

Unfortunately, we can't work on that in the next days.
So, if you want to submit the patches, it's perfect.


-- 
https://code.launchpad.net/~agilebg/account-invoice-report/adding_account_invoice_production_lot_7/+merge/191230
Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:account-invoice-report.