Re: [Openerp-community-reviewer] [Merge] lp:~hirt/ocb-addons/6.1_stock-move-tree into lp:ocb-addons/6.1

2014-06-10 Thread Stefan Rijnhart (Therp)
Review: Needs Fixing Thanks! Agree with Pedro. Apart from that, the diff looks odd with its executable permission changes. Could you revert those? -- https://code.launchpad.net/~hirt/ocb-addons/6.1_stock-move-tree/+merge/222463 Your team OpenERP Community Backports is subscribed to branch

Re: [Openerp-community-reviewer] [Merge] lp:~numerigraphe-team/ocb-addons/7.0-fill-inventory-OOM into lp:ocb-addons

2014-06-10 Thread Guewen Baconnier @ Camptocamp
Review: Approve code review, test LGTM. Thanks! -- https://code.launchpad.net/~numerigraphe-team/ocb-addons/7.0-fill-inventory-OOM/+merge/217049 Your team OpenERP Community Backports is subscribed to branch lp:ocb-addons. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post

[Openerp-community-reviewer] [Merge] lp:~numerigraphe-team/ocb-addons/7.0-fill-inventory-OOM into lp:ocb-addons

2014-06-10 Thread noreply
The proposal to merge lp:~numerigraphe-team/ocb-addons/7.0-fill-inventory-OOM into lp:ocb-addons has been updated. Status: Needs review = Merged For more details, see: https://code.launchpad.net/~numerigraphe-team/ocb-addons/7.0-fill-inventory-OOM/+merge/217049 --

[Openerp-community-reviewer] [Bug 1312045] Re: Importing inventory pushes the OS to out-of-memory condition when many Stock Moves exist

2014-06-10 Thread Guewen Baconnier @ Camptocamp
** Changed in: ocb-addons Status: Fix Committed = In Progress ** Changed in: ocb-addons Status: In Progress = Fix Released -- You received this bug notification because you are a member of OpenERP Community Backports, which is subscribed to OpenERP Community Backports (Addons).

[Openerp-community-reviewer] [Merge] lp:~numerigraphe-team/ocb-addons/7.0-fill-inventory-OOM into lp:ocb-addons

2014-06-10 Thread Lionel Sausin - Numérigraphe
The proposal to merge lp:~numerigraphe-team/ocb-addons/7.0-fill-inventory-OOM into lp:ocb-addons has been updated. Description changed to: We're having problems importing an inventory for a location that contains 63 Stock Moves. The problem seems to lie in the python loop based on the

Re: [Openerp-community-reviewer] [Merge] lp:~numerigraphe-team/ocb-addons/7.0-fill-inventory-OOM into lp:ocb-addons

2014-06-10 Thread Lionel Sausin - Numérigraphe
Thank you. For the archives, the MP on official addons was turned into a pull request on github here: https://github.com/odoo/odoo/pull/341 -- https://code.launchpad.net/~numerigraphe-team/ocb-addons/7.0-fill-inventory-OOM/+merge/217049 Your team OpenERP Community Backports is subscribed to

Re: [Openerp-community-reviewer] [Merge] lp:~numerigraphe-team/ocb-addons/7.0-fill-inventory-OOM into lp:ocb-addons

2014-06-10 Thread Laurent Mignon (Acsone)
Review: Approve code review Thanks for the MP. LGTM for V7 Note that for V8, with the new API (https://github.com/odoo-dev/odoo/blob/master-apiculture/openerp/osv/orm.py), the cache is managed in an other way and is no more reset on each call to the browse method. lmi --

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/sale-wkfl/product_procurement_type into lp:sale-wkfl

2014-06-10 Thread Romain Deheele - Camptocamp
After several various changes, I think it's time to review it again. Thanks, Romain -- https://code.launchpad.net/~camptocamp/sale-wkfl/product_procurement_type/+merge/222111 Your team Sale Core Editors is subscribed to branch lp:sale-wkfl. -- Mailing list:

[Openerp-community-reviewer] [Merge] lp:~hirt/ocb-addons/6.1_stock-move-tree into lp:ocb-addons/6.1

2014-06-10 Thread noreply
The proposal to merge lp:~hirt/ocb-addons/6.1_stock-move-tree into lp:ocb-addons/6.1 has been updated. Status: Needs review = Merged For more details, see: https://code.launchpad.net/~hirt/ocb-addons/6.1_stock-move-tree/+merge/222463 --

[Openerp-community-reviewer] [Merge] lp:~akretion-team/partner-contact-management/partner-firstname-inheritable into lp:partner-contact-management

2014-06-10 Thread Alexis de Lattre
Alexis de Lattre has proposed merging lp:~akretion-team/partner-contact-management/partner-firstname-inheritable into lp:partner-contact-management. Requested reviews: Partner and Contact Core Editors (partner-contact-core-editors) For more details, see:

[Openerp-community-reviewer] [Merge] lp:~hirt/ocb-addons/6.1_stock-move-tree into lp:ocb-addons/6.1

2014-06-10 Thread Etienne Hirt
Etienne Hirt has proposed merging lp:~hirt/ocb-addons/6.1_stock-move-tree into lp:ocb-addons/6.1. Requested reviews: OpenERP Community Backports (ocb) Related bugs: Bug #1327887 in OpenERP Community Backports (Addons): [6.1] stock.move.tree2 wrong priority, stock.move.tree3 leftover

Re: [Openerp-community-reviewer] [Merge] lp:~akretion-team/partner-contact-management/partner-firstname-inheritable into lp:partner-contact-management

2014-06-10 Thread Pedro Manuel Baeza
Review: Approve code review LGTM -- https://code.launchpad.net/~akretion-team/partner-contact-management/partner-firstname-inheritable/+merge/222730 Your team Partner and Contact Core Editors is subscribed to branch lp:partner-contact-management. -- Mailing list:

[Openerp-community-reviewer] [Merge] lp:~hirt/ocb-addons/6.1_stock-move-tree into lp:ocb-addons/6.1

2014-06-10 Thread Pedro Manuel Baeza
The proposal to merge lp:~hirt/ocb-addons/6.1_stock-move-tree into lp:ocb-addons/6.1 has been updated. Status: Merged = Rejected For more details, see: https://code.launchpad.net/~hirt/ocb-addons/6.1_stock-move-tree/+merge/222463 --

Re: [Openerp-community-reviewer] [Merge] lp:~hirt/ocb-addons/6.1_stock-move-tree into lp:ocb-addons/6.1

2014-06-10 Thread Pedro Manuel Baeza
See new MP: https://code.launchpad.net/~hirt/ocb-addons/6.1_stock-move-tree/+merge/222726 -- https://code.launchpad.net/~hirt/ocb-addons/6.1_stock-move-tree/+merge/222463 Your team OpenERP Community Backports is subscribed to branch lp:ocb-addons/6.1. -- Mailing list:

Re: [Openerp-community-reviewer] [Merge] lp:~akretion-team/partner-contact-management/partner-firstname-inheritable into lp:partner-contact-management

2014-06-10 Thread Acsone
Review: Approve code review LGTM -- https://code.launchpad.net/~akretion-team/partner-contact-management/partner-firstname-inheritable/+merge/222730 Your team Partner and Contact Core Editors is subscribed to branch lp:partner-contact-management. -- Mailing list:

Re: [Openerp-community-reviewer] [Merge] lp:~akretion-team/partner-contact-management/partner-firstname-inheritable into lp:partner-contact-management

2014-06-10 Thread David BEAL
Review: Approve LGTM -- https://code.launchpad.net/~akretion-team/partner-contact-management/partner-firstname-inheritable/+merge/222730 Your team Partner and Contact Core Editors is subscribed to branch lp:partner-contact-management. -- Mailing list: