openerp-community-reviewer team mailing list archive

Thread
Date

Re: [Merge] lp:~camptocamp/ocb-addons/7.0-fix-1189480 into lp:ocb-addons

To: mp+168463@xxxxxxxxxxxxxxxxxx
From: "Matthieu Dietrich @ camptocamp" <matthieu.dietrich@xxxxxxxxxxxxxx>
Date: Wed, 27 Nov 2013 13:08:00 -0000
In-reply-to: <20130610143037.5086.17449.launchpad@ackee.canonical.com>
Reply-to: mp+168463@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing

Actually, I investigated this branc, since it caused performance issues in our instance using it. The multiple read() are a major impact, given it will be done for each child project.
It seems that the computation is off a bit as well; projects with children will not add their own values for the fields, but those values will be added to their parents.

I'll rework this and do a YAML test to validate it.
-- 
https://code.launchpad.net/~camptocamp/ocb-addons/7.0-fix-1189480/+merge/168463
Your team OpenERP Community Backports Team is subscribed to branch lp:ocb-addons.