handle tax childs #6

Merged

mdsfred merged 2 commits from jangras/edocument_xrechnung:wip_jan into main

2025-01-09 08:33:34 +00:00

jangras commented

2025-01-03 21:59:21 +00:00

Member

Sehr geehrte Kollegen,
ich bitte um Review und Anmerkungen wie man das anders lösen könnte. Ich habe bei mir "Umsatzsteuer - Normalsatz" als parent und darunter coronabedingt 3 Childs mit Anfangs- und Enddatum. Illustration des Problems im Code.

Sehr geehrte Kollegen, ich bitte um Review und Anmerkungen wie man das anders lösen könnte. Ich habe bei mir "Umsatzsteuer - Normalsatz" als parent und darunter coronabedingt 3 Childs mit Anfangs- und Enddatum. Illustration des Problems im Code.

jangras added 1 commit 2025-01-03 21:59:22 +00:00

handle tax childs f3b4849e0c

jangras changed title from ~~handle tax childs~~ to WIP: handle tax childs

2025-01-03 21:59:28 +00:00

jangras changed title from ~~WIP: handle tax childs~~ to handle tax childs

2025-01-03 21:59:55 +00:00

jangras commented

2025-01-04 07:32:17 +00:00

Author

Member

Muss man eigentlich konsequenterweise rekursiv machen

jangras added 1 commit 2025-01-04 20:09:13 +00:00

fix typo dba1965894

mdsfred commented

2025-01-06 11:45:16 +00:00

Member

Erstmal gute Idee. Anstelle die Auswahl der Steuer selbst zu machen würde ich hier folgendes vorschlagen:

taxlines = Tax.compute(
            line.taxes, Decimal('1'), 1.0, 
            line.invoice.accounting_date or line.invoice.invoice_date)
assert(len(taxlines), 1)
tax = taxlines[0]['tax']

Die Funktion berechnet eigentlich den Steuerbetrag, wir nutzen nur die gewählte Steuer.

Erstmal gute Idee. Anstelle die Auswahl der Steuer selbst zu machen würde ich hier folgendes vorschlagen: ``` taxlines = Tax.compute( line.taxes, Decimal('1'), 1.0, line.invoice.accounting_date or line.invoice.invoice_date) assert(len(taxlines), 1) tax = taxlines[0]['tax'] ``` Die Funktion berechnet eigentlich den Steuerbetrag, wir nutzen nur die gewählte Steuer.