Re: [Openerp-community-reviewer] [Merge] lp:~luc-demeyer/account-financial-report/7.0-account_move_line_report_xls into lp:account-financial-report

2014-01-09 Thread Luc De Meyer (Noviat)
New defaults list available in https://code.launchpad.net/~luc-demeyer/account-financial-report/update-7.0-account_move_line_report_xls/+merge/200990 Luc -Original Message- From: boun...@canonical.com [mailto:boun...@canonical.com] On Behalf Of Frederic Clementi - Camptocamp Sent: woens

[Openerp-community-reviewer] [Merge] lp:~luc-demeyer/account-financial-report/update-7.0-account_move_line_report_xls into lp:account-financial-report

2014-01-09 Thread Luc De Meyer (Noviat)
Luc De Meyer (Noviat) has proposed merging lp:~luc-demeyer/account-financial-report/update-7.0-account_move_line_report_xls into lp:account-financial-report. Requested reviews: Account Report Core Editors (account-report-core-editor) For more details, see: https://code.launchpad.net/~luc-deme

[Openerp-community-reviewer] [Bug 1199858] Re: 7.0 : delivery - partial delivery will copy tracking number in the backorder picking

2014-01-09 Thread Alexandre Fayolle - camptocamp
** Branch linked: lp:~camptocamp/openobject-addons/7.0-595239-opw-skh- afe ** Also affects: ocb-addons Importance: Undecided Status: New ** Also affects: ocb-addons/7.0 Importance: Undecided Status: New -- You received this bug notification because you are a member of OpenER

[Openerp-community-reviewer] [Bug 796570] Re: Sale and purchase taxes are not get from account.accout default tax

2014-01-09 Thread Acsone
I linked this bug to ocb-addons as the fix has not been ported to 7.0. I see it is marked Opinion for official. I my opinion, the patch is a valuable addition to ocb 7.0. What do you think? ** Also affects: ocb-addons Importance: Undecided Status: New -- You received this bug notif

[Openerp-community-reviewer] [Merge] lp:~camptocamp/carriers-deliveries/7.0-delivery_carrier_label_dispatch-options-yvr into lp:carriers-deliveries

2014-01-09 Thread Yannick Vaucher @ Camptocamp
Yannick Vaucher @ Camptocamp has proposed merging lp:~camptocamp/carriers-deliveries/7.0-delivery_carrier_label_dispatch-options-yvr into lp:carriers-deliveries with lp:~camptocamp/carriers-deliveries/7.0-add-delivery_carrier_label_dispatch-yvr as a prerequisite. Requested reviews: Stock and

[Openerp-community-reviewer] [Merge] lp:~camptocamp/carriers-deliveries/7.0-add-base_delivery_carrier_label-yvr into lp:carriers-deliveries

2014-01-09 Thread Yannick Vaucher @ Camptocamp
The proposal to merge lp:~camptocamp/carriers-deliveries/7.0-add-base_delivery_carrier_label-yvr into lp:carriers-deliveries has been updated. Status: Needs review => Work in progress For more details, see: https://code.launchpad.net/~camptocamp/carriers-deliveries/7.0-add-base_delivery_car

[Openerp-community-reviewer] [Merge] lp:~camptocamp/carriers-deliveries/7.0-add-base_delivery_carrier_label-yvr into lp:carriers-deliveries

2014-01-09 Thread Yannick Vaucher @ Camptocamp
Yannick Vaucher @ Camptocamp has proposed merging lp:~camptocamp/carriers-deliveries/7.0-add-base_delivery_carrier_label-yvr into lp:carriers-deliveries. Commit message: Port of delivery_base and deliver_shipping_label removing direct print logic Requested reviews: Guewen Baconnier @ Camptoca

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/carriers-deliveries/7.0-add-base_delivery_carrier_label-yvr into lp:carriers-deliveries

2014-01-09 Thread Yannick Vaucher @ Camptocamp
Thanks for the review I fixed that issue -- https://code.launchpad.net/~camptocamp/carriers-deliveries/7.0-add-base_delivery_carrier_label-yvr/+merge/196753 Your team Stock and Logistic Core Editors is subscribed to branch lp:carriers-deliveries. -- Mailing list: https://launchpad.net/~openerp-

[Openerp-community-reviewer] [Merge] lp:~akretion-team/account-financial-tools/70-reversal-fix-improve into lp:account-financial-tools

2014-01-09 Thread Yannick Vaucher @ Camptocamp
The proposal to merge lp:~akretion-team/account-financial-tools/70-reversal-fix-improve into lp:account-financial-tools has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~akretion-team/account-financial-tools/70-reversal-fix-improve/+merge/

Re: [Openerp-community-reviewer] [Merge] lp:~akretion-team/account-financial-tools/70-reversal-fix-improve into lp:account-financial-tools

2014-01-09 Thread Yannick Vaucher @ Camptocamp
Review: Approve code review, no tests LGTM -- https://code.launchpad.net/~akretion-team/account-financial-tools/70-reversal-fix-improve/+merge/200378 Your team OpenERP Community Reviewer/Maintainer is subscribed to branch lp:account-financial-tools. -- Mailing list: https://launchpad.net/~open

Re: [Openerp-community-reviewer] [Merge] lp:~akretion-team/sale-wkfl/70-add-sale_fiscal_position_update into lp:sale-wkfl

2014-01-09 Thread Raphaël Valyi - http : //www . akretion . com
Review: Approve LGTM, no tests -- https://code.launchpad.net/~akretion-team/sale-wkfl/70-add-sale_fiscal_position_update/+merge/200611 Your team Sale Core Editors is subscribed to branch lp:sale-wkfl. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to : openerp-commu

[Openerp-community-reviewer] [Merge] lp:~camptocamp/ocb-addons/ocb-7.0-595239-opw-skh+afe into lp:ocb-addons

2014-01-09 Thread Alexandre Fayolle - camptocamp
Alexandre Fayolle - camptocamp has proposed merging lp:~camptocamp/ocb-addons/ocb-7.0-595239-opw-skh+afe into lp:ocb-addons. Requested reviews: OpenERP Community Backports Team (ocb) Related bugs: Bug #1199858 in OpenERP Community Backports (Addons): "7.0 : delivery - partial delivery will c

[Openerp-community-reviewer] [Merge] lp:~akretion-team/sale-reports/7.0-add-sale_group into lp:sale-reports

2014-01-09 Thread Raphaël Valyi - http : //www . akretion . com
The proposal to merge lp:~akretion-team/sale-reports/7.0-add-sale_group into lp:sale-reports has been updated. Status: Needs review => Rejected For more details, see: https://code.launchpad.net/~akretion-team/sale-reports/7.0-add-sale_group/+merge/200881 -- https://code.launchpad.net/~akret

Re: [Openerp-community-reviewer] [Merge] lp:~akretion-team/sale-reports/7.0-add-sale_group into lp:sale-reports

2014-01-09 Thread Raphaël Valyi - http : //www . akretion . com
as a second though, I think this module and coming sale_import_lines modules are better suited for the sale-wkf project, so the merge moved to there https://code.launchpad.net/~akretion-team/sale-wkfl/7.0-add-sale_group/+merge/201017 -- https://code.launchpad.net/~akretion-team/sale-reports/7.0-

Re: [Openerp-community-reviewer] [Merge] lp:~openerp-community/server-env-tools/7.0-modules-from-openobject-extension into lp:server-env-tools

2014-01-09 Thread Yannick Vaucher @ Camptocamp
Review: Approve code review, no tests Thanks for the changes LGTM -- https://code.launchpad.net/~openerp-community/server-env-tools/7.0-modules-from-openobject-extension/+merge/200744 Your team Server Environment And Tools Core Editors is requested to review the proposed merge of lp:~openerp-c

[Openerp-community-reviewer] [Merge] lp:~akretion-team/sale-wkfl/7.0-add-sale_group into lp:sale-wkfl

2014-01-09 Thread Raphaël Valyi - http : //www . akretion . com
Raphaël Valyi - http://www.akretion.com has proposed merging lp:~akretion-team/sale-wkfl/7.0-add-sale_group into lp:sale-wkfl. Requested reviews: Sale Core Editors (sale-core-editors) For more details, see: https://code.launchpad.net/~akretion-team/sale-wkfl/7.0-add-sale_group/+merge/201017 A

[Openerp-community-reviewer] [Merge] lp:~michelemilidoni/server-env-tools/it_tr into lp:server-env-tools

2014-01-09 Thread Yannick Vaucher @ Camptocamp
The proposal to merge lp:~michelemilidoni/server-env-tools/it_tr into lp:server-env-tools has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~michelemilidoni/server-env-tools/it_tr/+merge/199557 -- https://code.launchpad.net/~michelemilidoni

[Openerp-community-reviewer] [Bug 1267495] [NEW] [7.0] stock: call to stock.picking cancel_assign causes many unnecessary calls to workflow trg_write

2014-01-09 Thread Alexandre Fayolle - camptocamp
Public bug reported: When calling stock_picking.cancel_assign, a first call to trg_write for the picking is done, and then the stock_move.cancel_assign is called, which will cause a call to trg_write of the picking for each move. This is not efficient at all. The correct way of doing things is to

[Openerp-community-reviewer] [Bug 1267495] Re: [7.0] stock: call to stock.picking cancel_assign causes many unnecessary calls to workflow trg_write

2014-01-09 Thread Alexandre Fayolle - camptocamp
** Also affects: openobject-addons Importance: Undecided Status: New -- You received this bug notification because you are a member of OpenERP Community Backports Team, which is subscribed to OpenERP Community Backports (Addons). https://bugs.launchpad.net/bugs/1267495 Title: [7.0] s

[Openerp-community-reviewer] [Merge] lp:~akretion-team/sale-wkfl/7.0-add-sale_import_lines into lp:sale-wkfl

2014-01-09 Thread Raphaël Valyi - http : //www . akretion . com
Raphaël Valyi - http://www.akretion.com has proposed merging lp:~akretion-team/sale-wkfl/7.0-add-sale_import_lines into lp:sale-wkfl with lp:~akretion-team/sale-wkfl/7.0-add-sale_group as a prerequisite. Requested reviews: Sale Core Editors (sale-core-editors) For more details, see: https://c

[Openerp-community-reviewer] [Bug 1267495] Re: [7.0] stock: call to stock.picking cancel_assign causes many unnecessary calls to workflow trg_write

2014-01-09 Thread Alexandre Fayolle - camptocamp
** No longer affects: ocb-addons ** Also affects: ocb-addons Importance: Undecided Status: New ** Also affects: ocb-addons/7.0 Importance: Undecided Status: New -- You received this bug notification because you are a member of OpenERP Community Backports Team, which is subsc

[Openerp-community-reviewer] [Merge] lp:~camptocamp/ocb-addons/ocb-7.0-fix_1267495-afe into lp:ocb-addons

2014-01-09 Thread Alexandre Fayolle - camptocamp
Alexandre Fayolle - camptocamp has proposed merging lp:~camptocamp/ocb-addons/ocb-7.0-fix_1267495-afe into lp:ocb-addons. Requested reviews: OpenERP Community Backports Team (ocb) Related bugs: Bug #1267495 in OpenERP Community Backports (Addons): "[7.0] stock: call to stock.picking cancel_a

[Openerp-community-reviewer] [Merge] lp:~camptocamp/ocb-addons/ocb-7.0-fix_1264950-afe into lp:ocb-addons

2014-01-09 Thread Alexandre Fayolle - camptocamp
Alexandre Fayolle - camptocamp has proposed merging lp:~camptocamp/ocb-addons/ocb-7.0-fix_1264950-afe into lp:ocb-addons. Requested reviews: OpenERP Community Backports Team (ocb) Related bugs: Bug #1264950 in OpenERP Community Backports (Addons): "7.0: stock_picking.action_cancel can cause

[Openerp-community-reviewer] [Bug 1264950] Re: 7.0: stock_picking.action_cancel can cause RuntimeError: maximum recursion depth exceeded

2014-01-09 Thread Alexandre Fayolle - camptocamp
** Also affects: ocb-addons Importance: Undecided Status: New ** Also affects: ocb-addons/7.0 Importance: Undecided Status: New -- You received this bug notification because you are a member of OpenERP Community Backports Team, which is subscribed to OpenERP Community Backpor

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/ocb-addons/ocb-7.0-fix_1264950-afe into lp:ocb-addons

2014-01-09 Thread Raphaël Valyi - http : //www . akretion . com
Review: Approve LGTM no tests -- https://code.launchpad.net/~camptocamp/ocb-addons/ocb-7.0-fix_1264950-afe/+merge/201028 Your team OpenERP Community Backports Team is subscribed to branch lp:ocb-addons. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to : openerp-co

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/ocb-addons/ocb-7.0-fix_1264950-afe into lp:ocb-addons

2014-01-09 Thread Alexandre Fayolle - camptocamp
this is running in production for my customer -- https://code.launchpad.net/~camptocamp/ocb-addons/ocb-7.0-fix_1264950-afe/+merge/201028 Your team OpenERP Community Backports Team is requested to review the proposed merge of lp:~camptocamp/ocb-addons/ocb-7.0-fix_1264950-afe into lp:ocb-addons. -

[Openerp-community-reviewer] [Merge] lp:~camptocamp/account-consolidation/consolidation_vre_test_improvement_resubmit into lp:account-consolidation/7.0

2014-01-09 Thread noreply
The proposal to merge lp:~camptocamp/account-consolidation/consolidation_vre_test_improvement_resubmit into lp:account-consolidation/7.0 has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~camptocamp/account-consolidation/consolidation_vre_test_im

[Openerp-community-reviewer] [Bug 1247158] Re: [6.1, 7.0, trunk] Context is not propagated when writing translation on 'source' fields

2014-01-09 Thread Martin Trigaux (OpenERP)
Thanks for the updated patch, merged into 7 revno: 5194 [merge] revision-id: m...@openerp.com-20140109153601-13wo9gj5cz6ho3nm ** Changed in: openobject-server Status: Fix Committed => Fix Released -- You received this bug notification because you are a member of OpenERP Community Backpo

Re: [Openerp-community-reviewer] [Merge] lp:~mikel-martin/purchase-wkfl/6.1-purchase_landed_costs into lp:purchase-wkfl/6.1

2014-01-09 Thread Sandy Carter (http://www.savoirfairelinux.com)
Review: Approve code review, no test Your code passes my PEP8 check. One more thing purchase_landed_costs/stock.py:3:1: F401 'itemgetter' imported but unused -- https://code.launchpad.net/~mikel-martin/purchase-wkfl/6.1-purchase_landed_costs/+merge/199587 Your team Purchase Core Editors is subsc

[Openerp-community-reviewer] [Merge] lp:~camptocamp/account-financial-tools/draft_move_mdh_resubmit into lp:account-financial-tools

2014-01-09 Thread noreply
The proposal to merge lp:~camptocamp/account-financial-tools/draft_move_mdh_resubmit into lp:account-financial-tools has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~camptocamp/account-financial-tools/draft_move_mdh_resubmit/+merge/199150 -- h

[Openerp-community-reviewer] [Bug 1267523] Re: fetchmail attachment name invalid character

2014-01-09 Thread Ferdinand
** Also affects: ocb-addons Importance: Undecided Status: New ** Changed in: ocb-addons Importance: Undecided => High -- You received this bug notification because you are a member of OpenERP Community Backports Team, which is subscribed to OpenERP Community Backports (Addons). http

Re: [Openerp-community-reviewer] [Merge] lp:~dreis-pt/project-service/7.0-issue_task-dr into lp:project-service

2014-01-09 Thread invitu
Dear Daniel We think that task priority should be synchronized with issue priority In the other way, changes in task stage should synchronize with issue stage Please check out our MP -- https://code.launchpad.net/~dreis-pt/project-service/7.0-issue_task-dr/+merge/195985 Your team Project Core E

[Openerp-community-reviewer] [Merge] lp:~dr.dran/server-env-tools/remove_announcement_bar_popup into lp:server-env-tools

2014-01-09 Thread Franco Tampieri
Franco Tampieri has proposed merging lp:~dr.dran/server-env-tools/remove_announcement_bar_popup into lp:server-env-tools. Requested reviews: Server Environment And Tools Core Editors (server-env-tools-core-editors) For more details, see: https://code.launchpad.net/~dr.dran/server-env-tools/re

Re: [Openerp-community-reviewer] [Merge] lp:~dr.dran/server-env-tools/remove_announcement_bar_popup into lp:server-env-tools

2014-01-09 Thread Pedro Manuel Baeza
Review: Needs Information Hi, Franco, it isn't that handle already in the module? Regards. -- https://code.launchpad.net/~dr.dran/server-env-tools/remove_announcement_bar_popup/+merge/201053 Your team Server Environment And Tools Core Editors is subscribed to branch lp:server-env-tools. -- Ma

Re: [Openerp-community-reviewer] [Merge] lp:~dr.dran/server-env-tools/remove_announcement_bar_popup into lp:server-env-tools

2014-01-09 Thread Stefan Rijnhart (Therp)
Review: Needs Information Thanks Franco. Indeed, not the bar but the method behind it has been disarmed here: http://bazaar.launchpad.net/~server-env-tools-core-editors/server-env-tools/7.0/revision/53. Does this not work for you, or is there another reason for your proposal? -- https://code.l

Re: [Openerp-community-reviewer] [Merge] lp:~luc-demeyer/account-financial-report/update-7.0-account_move_line_report_xls into lp:account-financial-report

2014-01-09 Thread Alexis de Lattre
Review: Approve code review LGTM. Thanks Luc for this new MP. -- https://code.launchpad.net/~luc-demeyer/account-financial-report/update-7.0-account_move_line_report_xls/+merge/200990 Your team Account Report Core Editors is requested to review the proposed merge of lp:~luc-demeyer/account-fina

Re: [Openerp-community-reviewer] [Merge] lp:~dr.dran/server-env-tools/remove_announcement_bar_popup into lp:server-env-tools

2014-01-09 Thread Franco Tampieri
Hi Pedro and Stefan, my buildout installation have: - server revno 5152 - addons revno 9656 - web revno 4080 and I have modified the module with that patch because the toolbar appears every time I logon on a private session with firefox :( But let me try the revision 53 with the last revno of t

[Openerp-community-reviewer] [Merge] lp:~akretion-team/purchase-wkfl/70-add-purchase_fiscal_position_update into lp:purchase-wkfl

2014-01-09 Thread Alexis de Lattre
Alexis de Lattre has proposed merging lp:~akretion-team/purchase-wkfl/70-add-purchase_fiscal_position_update into lp:purchase-wkfl. Requested reviews: Purchase Core Editors (purchase-core-editors) For more details, see: https://code.launchpad.net/~akretion-team/purchase-wkfl/70-add-purchase_f

Re: [Openerp-community-reviewer] [Merge] lp:~akretion-team/account-invoicing/70-add-invoice_fiscal_position_update into lp:account-invoicing

2014-01-09 Thread Alexis de Lattre
By the way, here is the module purchase_fiscal_position_update : https://code.launchpad.net/~akretion-team/purchase-wkfl/70-add-purchase_fiscal_position_update/+merge/201115 We now have the full set for OpenERP 7 : sale + purchase + invoice -- https://code.launchpad.net/~akretion-team/account-in

Re: [Openerp-community-reviewer] [Merge] lp:~akretion-team/purchase-wkfl/70-add-purchase_fiscal_position_update into lp:purchase-wkfl

2014-01-09 Thread Pedro Manuel Baeza
Review: Approve code review Very good! Thanks for your work! Regards. -- https://code.launchpad.net/~akretion-team/purchase-wkfl/70-add-purchase_fiscal_position_update/+merge/201115 Your team Purchase Core Editors is subscribed to branch lp:purchase-wkfl. -- Mailing list: https://launchpad.net