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] l10n_ch: tax report box 200 computation fix #158008

Closed

Conversation

clbr-odoo
Copy link
Contributor

@clbr-odoo clbr-odoo commented Mar 18, 2024

With this PR #129717 we fixed multiple problems of the Swiss tax report but the 200 box is still not correct.

Fix the computation of the box 200 of the Swiss tax report. We remove 382 and 383 boxes that are for purchases and shouldn't be included.
We add box 205 and 289 so that the box 200 somehow represent the "gross" taxed amount, and 299 is therefore the "net" amount.

200 = 302 to 343 + 205 + 289

opw-3766215

@clbr-odoo clbr-odoo requested a review from a team March 18, 2024 13:36
@clbr-odoo clbr-odoo self-assigned this Mar 18, 2024
@clbr-odoo clbr-odoo requested review from jco-odoo and removed request for a team March 18, 2024 13:37
@robodoo
Copy link
Contributor

robodoo commented Mar 18, 2024

@clbr-odoo clbr-odoo requested review from Levizar and removed request for jco-odoo March 18, 2024 13:44
@C3POdoo C3POdoo added the OE the report is linked to a support ticket (opw-...) label Mar 18, 2024
With this PR odoo#129717 we fixed
multiple problems of the Swiss tax report but the 200 box is still
not correct.

Fix the computation of the box 200 of the Swiss tax report.
We remove 382 and 383 boxes that are for purchases and shouldn't be
included.
We add box 205 and 289 so that the box 200 somehow represent the "gross"
taxed amount, and 299 is therefore the "net" amount.

200 = 302 to 343 + 205 + 289

opw-3766215
@clbr-odoo clbr-odoo force-pushed the 14.0-l10n_ch_tax_report_fix-clbr branch from 3a8806f to 52a587d Compare March 18, 2024 14:03
Copy link
Contributor

@Levizar Levizar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

200 is all the sales = all taxable sale + all non taxable sales = 302~343 + 289
289 is all the non taxed sales
299 is all the taxable sales = 200 - 289

Makes sense to me, good job!

@robodoo r+

robodoo pushed a commit that referenced this pull request Mar 18, 2024
With this PR #129717 we fixed
multiple problems of the Swiss tax report but the 200 box is still
not correct.

Fix the computation of the box 200 of the Swiss tax report.
We remove 382 and 383 boxes that are for purchases and shouldn't be
included.
We add box 205 and 289 so that the box 200 somehow represent the "gross"
taxed amount, and 299 is therefore the "net" amount.

200 = 302 to 343 + 205 + 289

opw-3766215

closes #158008

Signed-off-by: Brice Bartoletti (bib) <bib@odoo.com>
@Levizar
Copy link
Contributor

Levizar commented Mar 18, 2024

For the records:
image

@fw-bot
Copy link
Contributor

fw-bot commented Mar 22, 2024

@fw-bot fw-bot deleted the 14.0-l10n_ch_tax_report_fix-clbr branch April 1, 2024 15:46
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.

None yet

5 participants