Skip to content

fix(deltatech_website_vat_validation): pastreaza afisarea preturilor pe website la recalcul - #2841

Merged
VoicuStefan2001 merged 1 commit into
19.0from
19.0-fix-deltatech_website_vat_validation
Aug 21, 2026
Merged

fix(deltatech_website_vat_validation): pastreaza afisarea preturilor pe website la recalcul#2841
VoicuStefan2001 merged 1 commit into
19.0from
19.0-fix-deltatech_website_vat_validation

Conversation

@VoicuStefan2001

Copy link
Copy Markdown
Collaborator

Context / de ce

La un client cu localizare RO și sincronizare zilnică ANAF (l10n_ro_anaf_partner), setarea Website > Configurare > Setări > "Afișează prețurile produselor" (tax_included) revenea silențios la "fără taxe" (tax_excluded) în fiecare noapte.

Cauza (confirmată experimental, nu speculație): website.show_line_subtotals_tax_selection e un câmp calculat, nu simplu stocat (compute='_compute_show_line_subtotals_tax_selection', fără @api.depends real pe altceva decât company_id.account_fiscal_country_id). Odoo îl pune necondiționat pe tax_excluded la fiecare recalcul — există override pentru Argentina/Brazilia, dar nu pentru România.

Recalculul se declanșează la orice scriere pe company_id.account_fiscal_country_id, care depinde de adresa partenerului companiei (res.company._compute_address). Cron-ul l10n_ro_anaf_partner._cron_l10n_ro_anaf_sync_partners sincronizează toți partenerii RO cu CUI — inclusiv partenerul propriei companii — și rescrie strada/orașul/codul poștal cu datele de la ANAF, declanșând cascada în fiecare noapte.

Ce s-a schimbat

  • models/website.py (nou) — override pe _compute_show_line_subtotals_tax_selection: citește valoarea persistată din DB (cu env.flush_all() înainte, ca să vadă și scrierile ORM încă necomise), lasă website_sale să calculeze normal, apoi restaurează valoarea anterioară dacă exista una. Nu impune nicio valoare fixă — păstrează ce a ales administratorul, valabil și pentru clienți B2B care vor prețuri fără TVA afișat.
  • __init__.py — importă noul pachet models.
  • tests/test_website_tax_display.py (nou) — reproduce exact scenariul (scrie adresa companiei, verifică că prețul nu se resetează) + verifică că un website nou primește totuși comportamentul implicit normal.
  • readme/HISTORY.md, __manifest__.py — bump 19.0.0.0.2.

Testare

./run_module_tests.sh deltatech_website_vat_validation pe bază curată, demo-free: 6 teste, 0 eșecuri (4 preexistente + 2 noi). Cauza a fost întâi reprodusă manual, izolat, pe o copie locală neutralizată a bazei clientului (scriere directă pe adresa companiei → confirmat că prețul se resetează fără fix, rămâne stabil cu fix-ul instalat).

Note / riscuri

  • Nu instalat încă pe producție la clientul la care s-a găsit problema — momentan e activ doar un cron „watchdog" temporar care corectează valoarea la 15 minute, ca plasă de siguranță până se decide instalarea acestui modul.
  • Modulul depinde deja de website_sale; nu s-a adăugat nicio dependență nouă.

🤖 Generated with Claude Code

…pe website la recalcul

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@codecov

codecov Bot commented Aug 21, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 94.44444% with 1 line in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (19.0@6e00038). Learn more about missing BASE report.
⚠️ Report is 12 commits behind head on 19.0.

Files with missing lines Patch % Lines
deltatech_website_vat_validation/models/website.py 93.75% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             19.0    #2841   +/-   ##
=======================================
  Coverage        ?   73.48%           
=======================================
  Files           ?      852           
  Lines           ?    16113           
  Branches        ?     2828           
=======================================
  Hits            ?    11840           
  Misses          ?     3367           
  Partials        ?      906           
Flag Coverage Δ
account 75.84% <ø> (?)
misc-1 65.03% <ø> (?)
misc-2 77.11% <ø> (?)
misc-3 54.36% <ø> (?)
mrp 77.57% <ø> (?)
pos 87.09% <ø> (?)
purchase 73.88% <ø> (?)
sale 60.21% <ø> (?)
stock 68.21% <ø> (?)
website 72.17% <94.44%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@VoicuStefan2001
VoicuStefan2001 merged commit 55d1982 into 19.0 Aug 21, 2026
16 checks passed
@VoicuStefan2001
VoicuStefan2001 deleted the 19.0-fix-deltatech_website_vat_validation branch August 21, 2026 10:51
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