Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FIX] account, account_tax_cash_basis: unreconcile payment #16653

Merged
merged 1 commit into from
May 10, 2017

Conversation

nim-odoo
Copy link
Contributor

  • Activate multi-currency: EUR is main currency, USD is secondary
    currency
  • USD currency rate is:
    1.2 on 2017-01-01
    1.5 on 2017-02-01
  • Create a journal in USD (e.g. Cash)
  • Create a tax with Cash Basis activated
  • Create an invoice of 100 USD on 2017-01-15
  • Register a payment of 100 USD on this invoice on 2017-02-15
  • 4 journal entries are created:
    the payment itself
    the exchange rate difference
    the case basis linked to the payment
    the cash basis linked to the exchange rate difference
  • Unreconcile the payment (on the invoice, from the payment pop-up)

The following journal entries are reversed:
the exchange rate difference
the case basis linked to the payment

The cash basis entry linked to the exchange rate difference is not
reversed, sinced it is not directly linked to the payment.

opw-728767

Description of the issue/feature this PR addresses:

Current behavior before PR:

Desired behavior after PR is merged:

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@nim-odoo nim-odoo self-assigned this Apr 26, 2017
@nim-odoo nim-odoo added the OE the report is linked to a support ticket (opw-...) label Apr 26, 2017
@wtaferner
Copy link
Contributor

Yay, very much appreciated to take this responsibility :-)

@nim-odoo nim-odoo force-pushed the 9.0-opw-728767-cash_basis-nim branch from e8fb022 to 6f5aa64 Compare April 28, 2017 08:16
@nim-odoo nim-odoo requested a review from qdp-odoo May 2, 2017 14:23
@nim-odoo nim-odoo force-pushed the 9.0-opw-728767-cash_basis-nim branch 2 times, most recently from f04f7da to fb30bac Compare May 9, 2017 10:07
- Activate multi-currency: EUR is main currency, USD is secondary
  currency
- USD currency rate is:
  1.2 on 2017-01-01
  1.5 on 2017-02-01
- Create a journal in USD (e.g. Cash)
- Create a tax with Cash Basis activated
- Create an invoice of 100 USD on 2017-01-15
- Register a payment of 100 USD on this invoice on 2017-02-15
- 4 journal entries are created:
  the payment itself
  the exchange rate difference
  the cash basis linked to the payment
  the cash basis linked to the exchange rate difference

The cash basis linked to the exchange rate difference should not be
created, it is not necessary. The amounts are already included in the
cash basis linked to the payment.

opw-728767
@nim-odoo nim-odoo force-pushed the 9.0-opw-728767-cash_basis-nim branch from fb30bac to 3bdea9e Compare May 9, 2017 13:14
@nim-odoo nim-odoo merged commit 3bdea9e into odoo:9.0 May 10, 2017
@nim-odoo nim-odoo deleted the 9.0-opw-728767-cash_basis-nim branch May 10, 2017 07:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OE the report is linked to a support ticket (opw-...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants