Merge lp://qastaging/~openerp-community-testers/openobject-addons/trunk-sbi-bug1067375 into lp://qastaging/openobject-addons

Proposed by Stéphane Bidoul (Acsone)
Status: Needs review
Proposed branch: lp://qastaging/~openerp-community-testers/openobject-addons/trunk-sbi-bug1067375
Merge into: lp://qastaging/openobject-addons
Diff against target: 158 lines (+115/-22)
3 files modified
account/account_invoice.py (+28/-22)
account/tests/__init__.py (+27/-0)
account/tests/test_multi_company.py (+60/-0)
To merge this branch: bzr merge lp://qastaging/~openerp-community-testers/openobject-addons/trunk-sbi-bug1067375
Reviewer Review Type Date Requested Status
Olivier Dony (Odoo) Pending
Review via email: mp+132356@code.qastaging.launchpad.net
To post a comment you must log in.
7901. By Stéphane Bidoul (Acsone)

account tests: add checks in __init__.py

Revision history for this message
Stéphane Bidoul (Acsone) (sbi) wrote :

Hi,

Anything blocking this MP from the test sprint?

-sbi

Revision history for this message
Olivier Dony (Odoo) (odo-openerp) wrote :

Hi Stephane,

Nothing blocking but the surrounding (older) code looks suspiciously complicated/redundant so I wanted to have a closer look at it before merging, if you don't mind too much (if I merge directly it will be forgotten)

Revision history for this message
Stéphane Bidoul (Acsone) (sbi) wrote :

Hi Olivier,

Yep indeed I also noticed the whole area looks quite convoluted.

Nothing urgent on my side.

Unmerged revisions

7901. By Stéphane Bidoul (Acsone)

account tests: add checks in __init__.py

7900. By Stéphane Bidoul (Acsone)

[FIX] account_invoice: system error creating invoice in multi-company context when customer has no payable/receivable account for current user's company

7899. By Stéphane Bidoul (Acsone)

account_invoice: added test for 1067375

7898. By Launchpad Translations on behalf of openerp

Launchpad automatic translations update.

7897. By Olivier Dony (Odoo)

[FIX] account: currency_id must be passed to the chart installation wizard

Otherwise the installation of a chart of account
via the settings page fails miserably.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: