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

[16.0][FIX] account_financial_report: take multiple account tags per tax into account #1267

Open
wants to merge 1 commit into
base: 16.0
Choose a base branch
from

Conversation

dhx
Copy link

@dhx dhx commented Jan 5, 2025

  • When there are multiple repartition lines having different tax tags the VAT report will take all the tax lines for all tags into consideration leading to wrong results (e.g.: l10n_at.account_tax_template_sales_rev_charge_0_code021_1e)
  • This patch will fix this by only using the balance of account move lines for the linked tax tags.
  • As this change leads to the account move line balance being only taken into account for the tags being explicitly set in the repartition line the net sum will not be affected when the tag is not associated to the base repartition line (leading to changes in the expected behaviour in test_01_compute).

…to account

- When there are multiple repartition lines having different tax tags the VAT report
  will take all the tax lines for all tags into consideration leading to wrong
  results (e.g.: l10n_at.account_tax_template_sales_rev_charge_0_code021_1e)
- This patch will fix this by only using the balance of account move lines for
  the linked tax tags.
- As this change leads to the account move line balance being only taken into
  account for the tags being explicitly set in the repartition line the net
  sum will not be affected when the tag is not associated to the base repartition
  line (leading to changes in the expected behaviour in test_01_compute).
@dhx dhx changed the title [FIX] account_financial_report: take multiple account tags per tax into account [16.0][FIX] account_financial_report: take multiple account tags per tax into account Jan 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant