Odoo Addons (MOVED TO GITHUB)

Merge lp://qastaging/~therp-nl/openobject-addons/7.0-lp827649-voucher_tax_inclusive into lp://qastaging/openobject-addons

  1. 7.0-lp827649-voucher_tax_inclusive
  2. Merge into trunk
Proposed by Stefan Rijnhart (Opener)
Status: Needs review
Proposed branch: lp://qastaging/~therp-nl/openobject-addons/7.0-lp827649-voucher_tax_inclusive
Merge into: lp://qastaging/openobject-addons
Diff against target: 186 lines (+55/-58)
1 file modified
account_voucher/account_voucher.py (+55/-58)
To merge this branch: bzr merge lp://qastaging/~therp-nl/openobject-addons/7.0-lp827649-voucher_tax_inclusive
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email: mp+136534@code.qastaging.launchpad.net

Description of the change

This branch adds support for using inclusive taxes on sales and purchase vouchers.

Tax is only computed using the account.tax model. As the workflow did not force this, the call to compute_tax() was added to action_move_line_create(). Compute_tax() was then modified to do nothing in the case of payment or receipt vouchers.

Using an inclusive tax can lead to small rounding differences that were not caught by the manual summing of voucher line amounts, and thus to unbalanced moves. This summing was therefore replaced with a call to account_move._compute_balance() on the voucher's move so that the writeoff amount is always correct (by definition).

To post a comment you must log in.

Unmerged revisions

8143. By Stefan Rijnhart (Opener)

[FIX] Indentation
[FIX] Remove domain on inclusive taxes from the voucher's tax

8142. By Stefan Rijnhart (Opener)

[FIX] lp:827649, account_voucher: generate correct move lines using inclusive tax

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: