Re: [Openerp-connector-community] Jobs are not assign to worker

2014-04-23 Thread Adam Langowski / IDEA07 Studio Kreatywne
Hello, Finally I have found solution. The problem was that in my addons-path, root addons folder wasn't first. When I have changed the order everything is fine. That's my init.d line: start-stop-daemon --start --quiet --pidfile /var/run/${NAME}.pid --chuid ${USER} --background --make-pidfile

Re: [Openerp-community] Rationale for proposing the project openerp-nomenclature to join the OCA projects

2014-04-23 Thread Franco Tampieri
Hi Erdem! Nice to hear you, feel free to put Information on the page. Next level is to prepare a blueprint to prepare a Contact profile guidelines Regards Il mercoledì 23 aprile 2014, Info SHS-AV i...@shs-av.com ha scritto: Hi Erdem, I prepared a web page to insert and gather informations

Re: [Openerp-community] Rationale for proposing the project openerp-nomenclature to join the OCA projects

2014-04-23 Thread Markus Schneider
just update german information on the wiki we have two data we want to integrate to OpenERP: 1) city - zip code + geoinformation of the city is there a module for other country how to provide this data? 2) holidays is someone working on something simular? for hr module Greetings Markus On

Re: [Openerp-community] Duty roster module

2014-04-23 Thread Pedro Manuel Baeza Romero
Hi, Phui, It looks very good, and indeed it's a great addition to OCA repositories. I think best place for it is *lp:openerp-hr*. Doest itinclude automatic planning of the duty roster? It would be fantastic. Regards. 2014-04-23 13:56 GMT+02:00 Phui Hock phuih...@codekaki.com: Hi all, I have

[Openerp-community] Duty roster module

2014-04-23 Thread Phui Hock
Hi all, I have developed a duty roster module to allow shifts/schedule management. This is useful when used in conjunction with hr_holidays to calculate more accurately the number of leaves a staff can take, for example. A few screenshots are uploaded here: http://imgur.com/a/mo8jR#0 Code is

Re: [Openerp-community] Duty roster module

2014-04-23 Thread Phui Hock
Hi Pedro, It doesn't do automatic planning. I am not sure that should be done. This module is actually part of a larger project that I am working for my client in manufacturing industry, and thought it would be useful for someone else. It hasn't been thoroughly tested as a module, but should work

[Openerp-community] mail reception: catchall address required?

2014-04-23 Thread Alexandre Fayolle
Hello, Quick question : is it mandatory to have a mail server allowing for the configuration of a catch all email address in order to get correct mail exchange and record creation on some mail receptions ? I'm asking because all instances I've seen so far use that setting to get mails centralized

Re: [Openerp-community] mail reception: catchall address required?

2014-04-23 Thread Houssine BAKKALI
Hi Alexandre, I've been struggling with that too. Maybe the plus addressing could be a good work around. Have a look at the answer of Olivier for more details : http://help.openerp.com/question/35894/how-to-trigger-an-email-on-project-issue-changes/ Houssine 2014-04-23 15:11 GMT+02:00

Re: [Openerp-community] mail reception: catchall address required?

2014-04-23 Thread Stefan Rijnhart
On 04/23/2014 03:11 PM, Alexandre Fayolle wrote: Quick question : is it mandatory to have a mail server allowing for the configuration of a catch all email address in order to get correct mail exchange and record creation on some mail receptions ? Hi Alexandre, no, I don't use catchall

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

2014-04-23 Thread Alexandre Fayolle - camptocamp
Actually I had completely goofed the branches (uploaded my ocb branch to openobject-addons and proposed to merge there, hence the changes related to ocb present in the old MP). Should be fixed now. Thanks for the heads up, Yannick. --

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/openerp-rma/7.0-fix-1282584 into lp:openerp-rma

2014-04-23 Thread Alexandre Fayolle - camptocamp
Review: Approve code review, no test -- https://code.launchpad.net/~camptocamp/openerp-rma/7.0-fix-1282584/+merge/207450 Your team OpenERP RMA is subscribed to branch lp:openerp-rma. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to :

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

2014-04-23 Thread Alexandre Fayolle - camptocamp
Alexandre Fayolle - camptocamp has proposed merging lp:~camptocamp/ocb-addons/6.1-fix_1311004_account_move_line_index-afe into lp:ocb-addons/6.1. Requested reviews: OpenERP Core Team (openerp) Guewen Baconnier @ Camptocamp (gbaconnier-c2c): code review Yannick Vaucher @ Camptocamp

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

2014-04-23 Thread Pedro Manuel Baeza
Review: Approve code review LGTM -- https://code.launchpad.net/~camptocamp/ocb-addons/6.1-fix_1311004_account_move_line_index-afe/+merge/216820 Your team OpenERP Community Backports Team is subscribed to branch lp:ocb-addons/6.1. -- Mailing list:

[Openerp-community-reviewer] [Merge] lp:~camptocamp/openerp-rma/7.0-fix-1282584 into lp:openerp-rma

2014-04-23 Thread noreply
The proposal to merge lp:~camptocamp/openerp-rma/7.0-fix-1282584 into lp:openerp-rma has been updated. Status: Needs review = Merged For more details, see: https://code.launchpad.net/~camptocamp/openerp-rma/7.0-fix-1282584/+merge/207450 --

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/openerp-rma/7.0-crm_claim_rma-add-picking-buttons into lp:openerp-rma

2014-04-23 Thread Joël Grand-Guillaume
Review: Approve code review, no tests LGTM. Thanks ! -- https://code.launchpad.net/~camptocamp/openerp-rma/7.0-crm_claim_rma-add-picking-buttons/+merge/208334 Your team OpenERP RMA is subscribed to branch lp:openerp-rma. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post

[Openerp-community-reviewer] [Merge] lp:~camptocamp/openerp-rma/7.0-crm_claim_rma-add-picking-buttons into lp:openerp-rma

2014-04-23 Thread noreply
The proposal to merge lp:~camptocamp/openerp-rma/7.0-crm_claim_rma-add-picking-buttons into lp:openerp-rma has been updated. Status: Needs review = Merged For more details, see: https://code.launchpad.net/~camptocamp/openerp-rma/7.0-crm_claim_rma-add-picking-buttons/+merge/208334 --

Re: [Openerp-community-reviewer] [Merge] lp:~akretion-team/openerp-rma/rma-cleanup-cleanup-cleanup into lp:openerp-rma

2014-04-23 Thread Alexis de Lattre
Just adding a small comment to say that Raphaël Valyi discovered a bug when doing a -u crm_claim_rma with that branch (it breaks on the duplicate constraint). We started digging to find a solution, but we didn't had time to implement it yet. --

Re: [Openerp-community-reviewer] [Merge] lp:~akretion-team/openerp-rma/rma-cleanup-cleanup-cleanup into lp:openerp-rma

2014-04-23 Thread Joël Grand-Guillaume
Review: Needs Fixing code review, no tests Hi, Thanks for improving the module. I may not have the time to read all carefully, just what I saw in a few minutes here: * L269 : For the method _get_default_warehouse, I prefer to get the WH through the user because take one from

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/openerp-rma/rma-fix-lp1311509_rde into lp:openerp-rma

2014-04-23 Thread Pedro Manuel Baeza
Review: Approve code review LGTM Regards. -- https://code.launchpad.net/~camptocamp/openerp-rma/rma-fix-lp1311509_rde/+merge/216824 Your team OpenERP RMA is subscribed to branch lp:openerp-rma. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to :

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/purchase-wkfl/7.0-merge-po-hooks-lep into lp:purchase-wkfl

2014-04-23 Thread Leonardo Pistone - camptocamp
Pedro, I proposed this to core trunk: https://code.launchpad.net/~camptocamp/openobject-addons/trunk-refactor-po-merge-lep/+merge/216841 -- https://code.launchpad.net/~camptocamp/purchase-wkfl/7.0-merge-po-hooks-lep/+merge/216745 Your team Purchase Core Editors is subscribed to branch

[Openerp-community-reviewer] [Merge] lp:~camptocamp/openerp-rma/rma-fix-lp1311520_rde into lp:openerp-rma

2014-04-23 Thread Romain Deheele - Camptocamp
The proposal to merge lp:~camptocamp/openerp-rma/rma-fix-lp1311520_rde into lp:openerp-rma has been updated. Status: Needs review = Work in progress For more details, see: https://code.launchpad.net/~camptocamp/openerp-rma/rma-fix-lp1311520_rde/+merge/216839 --

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/sale-wkfl/7.0-dropshipping-merge-po-lep into lp:sale-wkfl

2014-04-23 Thread Leonardo Pistone - camptocamp
this depends on https://code.launchpad.net/~camptocamp/purchase-wkfl/7.0-merge-po-hooks-lep/+merge/216745 -- https://code.launchpad.net/~camptocamp/sale-wkfl/7.0-dropshipping-merge-po-lep/+merge/216756 Your team Sale Core Editors is requested to review the proposed merge of

[Openerp-community-reviewer] [Merge] lp:~camptocamp/openerp-rma/rma-fix-lp1311520_rde into lp:openerp-rma

2014-04-23 Thread Romain Deheele - Camptocamp
Romain Deheele - Camptocamp has proposed merging lp:~camptocamp/openerp-rma/rma-fix-lp1311520_rde into lp:openerp-rma. Requested reviews: OpenERP RMA (openerprma) Related bugs: Bug #1311520 in OpenERP RMA: In Product Return, link Invoice with a line without product_id raises an error

[Openerp-community-reviewer] [Merge] lp:~camptocamp/account-budgeting/7.0-fix-store-fields-mdh into lp:account-budgeting

2014-04-23 Thread Matthieu Dietrich @ camptocamp
The proposal to merge lp:~camptocamp/account-budgeting/7.0-fix-store-fields-mdh into lp:account-budgeting has been updated. Description changed to: The object in the store condition was the wrong one. I fixed it for the correct one + fixed the now-called function. For more details, see:

[Openerp-community-reviewer] [Merge] lp:~camptocamp/account-budgeting/7.0-fix-store-fields-mdh into lp:account-budgeting

2014-04-23 Thread Matthieu Dietrich @ camptocamp
Matthieu Dietrich @ camptocamp has proposed merging lp:~camptocamp/account-budgeting/7.0-fix-store-fields-mdh into lp:account-budgeting. Commit message: [FIX] fixed store condition on fields 'analytic_amount', 'analytic_real_amount' and 'analytic_diff_amount' + fixed function called by the

[Openerp-community-reviewer] [Bug 1210195] Re: [trunk/7.0][addons] access denied when a subsidiary company's user tries to create a reordering rule

2014-04-23 Thread Martin Trigaux (OpenERP)
Hello, Ravi's fix was integrated into 7.0. revno: 10007 [merge] revision-id: m...@openerp.com-20140423100324-6stypg5uapnl1s5f ** Changed in: openobject-addons Status: Fix Committed = Fix Released ** Changed in: openobject-addons Importance: Undecided = Low -- You received this bug

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/server-env-tools/7.0-monitoring into lp:server-env-tools

2014-04-23 Thread Daniel Reis
I also noticed that only 20-25% of process log lines have the model field filled. Why is this? -- https://code.launchpad.net/~camptocamp/server-env-tools/7.0-monitoring/+merge/215138 Your team Server Environment And Tools Core Editors is subscribed to branch lp:server-env-tools. -- Mailing

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/account-budgeting/7.0-fix-store-fields-mdh into lp:account-budgeting

2014-04-23 Thread Matthieu Dietrich @ camptocamp
Leonardo, thanks for the review. For your questions: 1. yes, that field needs to be stored. The initial commit mentions that a sum of the lines is used in a groupby view, which is the default view for budget lines, coming from a project. 2. I think the optimal solution would be to retrieve

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/ocb-addons/7.0-fix-lp-1210195-RDE into lp:ocb-addons

2014-04-23 Thread Stefan Rijnhart (Therp)
Competing fix was merged in upstream 7.0: https://code.launchpad.net/~openerp-dev/openobject-addons/7.0-opw-596679-rgo/+merge/186294. Setting to rejected. -- https://code.launchpad.net/~camptocamp/ocb-addons/7.0-fix-lp-1210195-RDE/+merge/181227 Your team OpenERP Community Backports Team is

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/account-budgeting/7.0-fix-store-fields-mdh into lp:account-budgeting

2014-04-23 Thread Matthieu Dietrich @ camptocamp
@Leonardo: here is a SQL version of the request. Tell me if that's okay for you. Regards, Matthieu Dietrich -- https://code.launchpad.net/~camptocamp/account-budgeting/7.0-fix-store-fields-mdh/+merge/216849 Your team Account Core Editors is requested to review the proposed merge of

[Openerp-community-reviewer] [Bug 1210195] Re: [trunk/7.0][addons] access denied when a subsidiary company's user tries to create a reordering rule

2014-04-23 Thread Stefan Rijnhart (Therp)
** No longer affects: ocb-addons/7.0 ** No longer affects: ocb-addons -- 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/1210195 Title:

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/account-budgeting/7.0-fix-store-fields-mdh into lp:account-budgeting

2014-04-23 Thread Alexandre Fayolle - camptocamp
Review: Needs Fixing code review, no test Your SQL request maybe works but it is not written in an efficient way (2 DISTINCT, 1 subquery). This should be written with EXISTS. And, each time you bypass the ORM to write raw SQL, $DEITY kills a kitten. $DEITY also maims a puppy if you don't

[Openerp-community-reviewer] [Merge] lp:~stefan-therp/ocb-addons/7.0-lp1311628-dont_overwrite_empty_mail_catchall_domain into lp:ocb-addons

2014-04-23 Thread Stefan Rijnhart (Therp)
Stefan Rijnhart (Therp) has proposed merging lp:~stefan-therp/ocb-addons/7.0-lp1311628-dont_overwrite_empty_mail_catchall_domain into lp:ocb-addons. Requested reviews: OpenERP Community Backports Team (ocb) Related bugs: Bug #1311628 in OpenERP Addons: Mail catchall domain parameter being

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/account-budgeting/7.0-fix-store-fields-mdh into lp:account-budgeting

2014-04-23 Thread Matthieu Dietrich @ camptocamp
@Alexandre, Leonardo: I went back for a classical ORM approach. I don't think a store condition's function is called by another user than the SUPERUSER, but it's better to be safe. I'm still refining a YAML test as well to add to this MP. Regards, Matthieu --

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/account-budgeting/7.0-fix-store-fields-mdh into lp:account-budgeting

2014-04-23 Thread Leonardo Pistone - camptocamp
thanks Matthieu. I'm also not a fan of the SQL if I don't have a gun pointed at me. Hopefully with the test it will be easier to improve that. -- https://code.launchpad.net/~camptocamp/account-budgeting/7.0-fix-store-fields-mdh/+merge/216849 Your team Account Core Editors is subscribed to

[Openerp-community-reviewer] [Bug 1309105] Re: stock.picking.out group by in multi companies

2014-04-23 Thread Martin Trigaux (OpenERP)
Hello, Thanks for the report and the patch, it was merged in 7.0 revno: 10010 [merge] revision-id: m...@openerp.com-20140423132552-hpnr2skutqranvsj ** Changed in: openobject-addons Status: New = Fix Released ** Changed in: openobject-addons Importance: Undecided = Low -- You

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

2014-04-23 Thread Stefan Rijnhart (Therp)
The proposal to merge lp:~camptocamp/ocb-addons/7.0-fix-1309105 into lp:ocb-addons has been updated. Status: Needs review = Rejected For more details, see: https://code.launchpad.net/~camptocamp/ocb-addons/7.0-fix-1309105/+merge/216681 --

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

2014-04-23 Thread Stefan Rijnhart (Therp)
Rejecting, as merged upstream http://bazaar.launchpad.net/~openerp/openobject-addons/7.0/revision/10010 -- https://code.launchpad.net/~camptocamp/ocb-addons/7.0-fix-1309105/+merge/216681 Your team OpenERP Community Backports Team is requested to review the proposed merge of

[Openerp-community-reviewer] [Bug 1311076] Re: multiple popup weird behavior

2014-04-23 Thread Ruchir Shukla(BizzAppDev)
** Also affects: ocb-web 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 (Web). https://bugs.launchpad.net/bugs/1311076 Title: multiple popup

[Openerp-community-reviewer] [Bug 1309105] Re: stock.picking.out group by in multi companies

2014-04-23 Thread Stefan Rijnhart (Therp)
** No longer affects: ocb-addons/7.0 ** No longer affects: ocb-addons -- 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/1309105 Title:

Re: [Openerp-community-reviewer] [Merge] lp:~stefan-therp/ocb-addons/7.0-lp1311628-dont_overwrite_empty_mail_catchall_domain into lp:ocb-addons

2014-04-23 Thread Lara (Therp)
Review: Approve test -- https://code.launchpad.net/~stefan-therp/ocb-addons/7.0-lp1311628-dont_overwrite_empty_mail_catchall_domain/+merge/216877 Your team OpenERP Community Backports Team is requested to review the proposed merge of

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/account-budgeting/7.0-fix-store-fields-mdh into lp:account-budgeting

2014-04-23 Thread Matthieu Dietrich @ camptocamp
Hello, I added the YAML test, in order to verify that the amounts are updated after a new analytic line on the same analytic account was added. Regards, Matthieu -- https://code.launchpad.net/~camptocamp/account-budgeting/7.0-fix-store-fields-mdh/+merge/216849 Your team Account Core Editors is

[Openerp-community-reviewer] [Bug 1311628] Re: Mail catchall domain parameter being reset by changing any general setting

2014-04-23 Thread Stefan Rijnhart (Therp)
** Branch linked: lp:~stefan-therp/ocb- addons/7.0-lp1311628-dont_overwrite_empty_mail_catchall_domain ** Also affects: ocb-addons Importance: Undecided Status: New ** Also affects: ocb-addons/7.0 Importance: Undecided Status: New ** Changed in: ocb-addons/7.0 Status:

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

2014-04-23 Thread Yannick Vaucher @ Camptocamp
That was fast :) -- https://code.launchpad.net/~camptocamp/ocb-addons/7.0-fix-1309105/+merge/216681 Your team OpenERP Community Backports Team is requested to review the proposed merge of lp:~camptocamp/ocb-addons/7.0-fix-1309105 into lp:ocb-addons. -- Mailing list:

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/ocb-addons/fix-1308364-nbi into lp:ocb-addons

2014-04-23 Thread Yannick Vaucher @ Camptocamp
Review: Approve -- https://code.launchpad.net/~camptocamp/ocb-addons/fix-1308364-nbi/+merge/216023 Your team OpenERP Community Backports Team is subscribed to branch lp:ocb-addons. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to :

[Openerp-community-reviewer] [Merge] lp:~camptocamp/ocb-addons/fix-1308364-nbi into lp:ocb-addons

2014-04-23 Thread noreply
The proposal to merge lp:~camptocamp/ocb-addons/fix-1308364-nbi into lp:ocb-addons has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~camptocamp/ocb-addons/fix-1308364-nbi/+merge/216023 --

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

2014-04-23 Thread noreply
The proposal to merge lp:~camptocamp/ocb-addons/ocb-7.0-fix_1311004_account_move_line_index-afe into lp:ocb-addons has been updated. Status: Approved = Merged For more details, see:

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

2014-04-23 Thread Yannick Vaucher @ Camptocamp
Review: Approve code review, no test Thanks Alexandre, looks good now. -- https://code.launchpad.net/~camptocamp/ocb-addons/6.1-fix_1311004_account_move_line_index-afe/+merge/216820 Your team OpenERP Community Backports Team is subscribed to branch lp:ocb-addons/6.1. -- Mailing list:

[Openerp-community-reviewer] [Bug 1308364] Re: [7.0] [trunk] report_webkit precise mode does not take in account set_context

2014-04-23 Thread Yannick Vaucher @ Camptocamp
** Changed in: ocb-addons/7.0 Status: Fix Committed = Fix Released -- 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/1308364 Title: [7.0]

[Openerp-community-reviewer] [Merge] lp:~stefan-therp/ocb-addons/7.0-lp1311628-dont_overwrite_empty_mail_catchall_domain into lp:ocb-addons

2014-04-23 Thread Stefan Rijnhart (Therp)
Stefan Rijnhart (Therp) has proposed merging lp:~stefan-therp/ocb-addons/7.0-lp1311628-dont_overwrite_empty_mail_catchall_domain into lp:ocb-addons. Requested reviews: Lara (Therp) (lfreeke): test OpenERP Community Backports Team (ocb) Related bugs: Bug #1311628 in OpenERP Community

[Openerp-community-reviewer] [Bug 1311004] Re: missing database index on account_move_line(date, id)

2014-04-23 Thread Yannick Vaucher @ Camptocamp
** Changed in: ocb-addons/7.0 Status: Fix Committed = Fix Released -- 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/1311004 Title: missing

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/account-budgeting/7.0-fix-store-fields-mdh into lp:account-budgeting

2014-04-23 Thread Yannick Vaucher @ Camptocamp
Review: Approve code review, no test Thanks, you saved some kittens -- https://code.launchpad.net/~camptocamp/account-budgeting/7.0-fix-store-fields-mdh/+merge/216849 Your team Account Core Editors is subscribed to branch lp:account-budgeting. -- Mailing list:

[Openerp-community-reviewer] [Merge] lp:~therp-nl/ocb-web/7.0-form_destroy_buttons into lp:ocb-web

2014-04-23 Thread Ruchir Shukla(BizzAppDev)
Ruchir Shukla(BizzAppDev) has proposed merging lp:~therp-nl/ocb-web/7.0-form_destroy_buttons into lp:ocb-web. Requested reviews: OpenERP Community Backports Team (ocb) Related bugs: Bug #1311076 in OpenERP Community Backports (Web): multiple popup weird behavior

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/openerp-rma/rma-fix-lp1311509_rde into lp:openerp-rma

2014-04-23 Thread Romain Deheele - Camptocamp
Just add a test to check invoice refund action -- https://code.launchpad.net/~camptocamp/openerp-rma/rma-fix-lp1311509_rde/+merge/216824 Your team OpenERP RMA is subscribed to branch lp:openerp-rma. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to :

[Openerp-community-reviewer] [Merge] lp:~agilebg/openerp-product-attributes/adding_product_supplier_info_7 into lp:openerp-product-attributes

2014-04-23 Thread Alex Comba - Agile BG
Alex Comba - Agile BG has proposed merging lp:~agilebg/openerp-product-attributes/adding_product_supplier_info_7 into lp:openerp-product-attributes. Commit message: [ADD] module product_supplier_info Requested reviews: Product Core Editors (product-core-editors) For more details, see:

Re: [Openerp-community-reviewer] [Merge] lp:~therp-nl/ocb-web/7.0-form_destroy_buttons into lp:ocb-web

2014-04-23 Thread Stefan Rijnhart (Therp)
Review: Needs Fixing Ruchir, thanks for taking this dangling branch of Holger and propose it. I assume you have tested it thoroughly. To qualify for a merge in OCB, you need to prepare a branch that proposes the same change into upstream openerp-web. Setting to needs-fixing to indicate that

Re: [Openerp-community-reviewer] [Merge] lp:~camptocamp/openerp-rma/rma-fix-lp1311509_rde into lp:openerp-rma

2014-04-23 Thread Yannick Vaucher @ Camptocamp
Review: Approve code review, no test Ok LGTM In view all commands are array of 3 elements: http://bazaar.launchpad.net/~openerp/openerp-web/7.0/view/head:/addons/web/static/src/js/view_form.js#L3446 -- https://code.launchpad.net/~camptocamp/openerp-rma/rma-fix-lp1311509_rde/+merge/216824 Your

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

2014-04-23 Thread Stefan Rijnhart (Therp)
Review: Approve -- https://code.launchpad.net/~camptocamp/ocb-server/ocb-7.0-fix_1204920-afe/+merge/208390 Your team OpenERP Community Backports Team is subscribed to branch lp:ocb-server. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to :

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

2014-04-23 Thread Stefan Rijnhart (Therp)
I see the upstream branch is green on runbot, and Holger's old comment has been honoured. Thanks! -- https://code.launchpad.net/~camptocamp/ocb-server/ocb-7.0-fix_1204920-afe/+merge/208390 Your team OpenERP Community Backports Team is subscribed to branch lp:ocb-server. -- Mailing list:

[Openerp-community-reviewer] [Bug 1180773] Re: [7.0] hr_timesheet_sheet - attendances constraint can be messed up by removing an attendance between 2 others

2014-04-23 Thread Yannick Vaucher @ Camptocamp
** Changed in: openobject-addons Status: Fix Committed = Fix Released -- 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/1180773 Title: [7.0]

Re: [Openerp-community-reviewer] [Merge] lp:~stefan-therp/ocb-addons/7.0-lp1311628-dont_overwrite_empty_mail_catchall_domain into lp:ocb-addons

2014-04-23 Thread Pedro Manuel Baeza
Review: Approve code review LGTM. Regards. -- https://code.launchpad.net/~stefan-therp/ocb-addons/7.0-lp1311628-dont_overwrite_empty_mail_catchall_domain/+merge/216877 Your team OpenERP Community Backports Team is subscribed to branch lp:ocb-addons. -- Mailing list:

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

2014-04-23 Thread noreply
The proposal to merge lp:~camptocamp/ocb-server/ocb-7.0-fix_1204920-afe into lp:ocb-server has been updated. Status: Needs review = Merged For more details, see: https://code.launchpad.net/~camptocamp/ocb-server/ocb-7.0-fix_1204920-afe/+merge/208390 --

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

2014-04-23 Thread Stefan Rijnhart (Therp)
Review: Approve Approve as per http://en.wikipedia.org/wiki/Polish_z%C5%82oty. -- https://code.launchpad.net/~camptocamp/ocb-server/7.0-fix_1281290-afe/+merge/215612 Your team OpenERP Community Backports Team is subscribed to branch lp:ocb-server. -- Mailing list:

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

2014-04-23 Thread Stefan Rijnhart (Therp)
Review: Approve -- https://code.launchpad.net/~camptocamp/ocb-addons/7.0-fix_1281290-afe/+merge/215614 Your team OpenERP Community Backports Team is subscribed to branch lp:ocb-addons. -- Mailing list: https://launchpad.net/~openerp-community-reviewer Post to :

[Openerp-community-reviewer] [Bug 1204920] Re: [trunk] UnboundLocalError: local variable 'partner' referenced before assignment

2014-04-23 Thread Stefan Rijnhart (Therp)
** Changed in: ocb-server/7.0 Status: Fix Committed = Fix Released -- You received this bug notification because you are a member of OpenERP Community Backports Team, which is subscribed to OpenERP Community Backports (Server). https://bugs.launchpad.net/bugs/1204920 Title: [trunk]

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

2014-04-23 Thread noreply
The proposal to merge lp:~camptocamp/ocb-addons/7.0-fix_1281290-afe into lp:ocb-addons has been updated. Status: Needs review = Merged For more details, see: https://code.launchpad.net/~camptocamp/ocb-addons/7.0-fix_1281290-afe/+merge/215614 --

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

2014-04-23 Thread noreply
The proposal to merge lp:~camptocamp/ocb-server/7.0-fix_1281290-afe into lp:ocb-server has been updated. Status: Needs review = Merged For more details, see: https://code.launchpad.net/~camptocamp/ocb-server/7.0-fix_1281290-afe/+merge/215612 --

[Openerp-community-reviewer] [Bug 1281290] Re: Wrong currency code - Poland

2014-04-23 Thread Stefan Rijnhart (Therp)
** Changed in: ocb-server/7.0 Status: Fix Committed = Fix Released -- 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/1281290 Title: Wrong

[Openerp-community-reviewer] [Bug 1281290] Re: Wrong currency code - Poland

2014-04-23 Thread Stefan Rijnhart (Therp)
** Changed in: ocb-addons/7.0 Status: Fix Committed = Fix Released -- 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/1281290 Title: Wrong

Re: [Openerp-community-reviewer] [Merge] lp:~stefan-therp/ocb-addons/7.0-lp1311628-dont_overwrite_empty_mail_catchall_domain into lp:ocb-addons

2014-04-23 Thread Sylvain LE GAL (GRAP)
Review: Approve code review, no test Thanks a lot ! -- https://code.launchpad.net/~stefan-therp/ocb-addons/7.0-lp1311628-dont_overwrite_empty_mail_catchall_domain/+merge/216877 Your team OpenERP Community Backports Team is subscribed to branch lp:ocb-addons. -- Mailing list: