Version 5.0.14

In the same way as for categories the unece tax codes must be searched
on parents.

README.rst

@@ -9,37 +9,38 @@ pip install mds-edocument-xrechnung
 
 Requires
 ========
-- Tryton 7.0
+- Tryton 5.0
 
 Changes
 =======
 
-*7.0.10 - 12.12.2024*
+- handle tax childs (Jan Grasnick <jan@mittelwind.de>)
+- fix: rounding of unit_price at invoice-line
+
+*5.0.14 - 29.01.2025*
+
+- updt:  optimize rounding of unit_price
+
+*5.0.11 - 19.12.2024*
+
+- Lookup parent taxes for unece tax codes.
+- Improve the help text of xrechnung_routeid. Correct a translation.
+  (Mathias Behrle <mathiasb@m9s.biz>)
+
+*5.0.10 - 17.12.2024*
 
 - fix missing views
 - Remove arguments in super() calls. (Mathias Behrle)
 
-*7.0.9 - 11.12.2024*
+*5.0.9 - 11.12.2024*
 
 - fix name of party in exceptions
 
-*7.0.7 - 10.12.2024*
+*5.0.8 - 11.12.2024*
 
-- add iban to xml-export
+- fix setup
 
-*7.0.6 - 09.12.2024*
+*5.0.7 - 10.12.2024*
 
-- add: check for valid data to generate xml
+- compatibility to Tryton 5.0
-
+- add: export of factur-x 1.07.2, XRechnung 2.3 + 3.0
-*7.0.5 - 05.12.2024*
-
-- add: export Factur-X 1.07.2
-
-*7.0.4 - 05.12.2024*
-
-- add: export XRechnung 2.3 + 3.0
-- updt: xrechnung-route-id optional
-
-*7.0.3 - 22.12.2023*
-
-- compatibility to Tryton 7.0

__init__.py

@@ -6,7 +6,7 @@
 from trytond.pool import Pool
 from .edocument import XRechnung, FacturX
 from .bank import AccountNumber
-from .party import PartyConfiguration, Party
+from .party import Party, PartyIdentifier
 from .configuration import Configuration, BankEdocumentRel
 
 
@@ -18,5 +18,5 @@ def register():
         BankEdocumentRel,
         FacturX,
         Party,
-        PartyConfiguration,
+        PartyIdentifier,
         module='edocument_xrechnung', type_='model')

locale/de.po

@@ -46,7 +46,7 @@ msgstr "Für die Adresse der Partei '%(party)s' ist  kein Land festgelegt."
 #######################
 # party.configuration #
 #######################
-msgctxt "selection:party.configuration,identifier_types:"
+msgctxt "selection:party.identifier,type:"
 msgid "X-Rechnung Route-ID"
 msgstr "X-Rechnung Leitweg-ID"
 

locale/en.po

@@ -38,7 +38,7 @@ msgctxt "model:ir.message,text:msg_no_address_country"
 msgid "No country is specified for the address of the party '%(party)s'."
 msgstr "No country is specified for the address of the party '%(party)s'."
 
-msgctxt "selection:party.configuration,identifier_types:"
+msgctxt "selection:party.identifier,type:"
 msgid "X-Rechnung Route-ID"
 msgstr "X-Rechnung Route-ID"
 
@@ -50,10 +50,6 @@ msgctxt "field:party.party,xrechnung_routeid:"
 msgid "X-Rechnung Route-ID"
 msgstr "X-Rechnung Route-ID"
 
-msgctxt "help:party.party,xrechnung_routeid:"
-msgid "Enables the need for an XRechnung route ID at the party for exporting the XRechnung."
-msgstr "Enables the need for an XRechnung route ID at the party for exporting the XRechnung."
-
 msgctxt "field:account.tax,xrtax_category:"
 msgid "Tax Category"
 msgstr "Tax Category"

mixin.py

@@ -7,10 +7,18 @@
 from decimal import Decimal
 import html
 from trytond.exceptions import UserError
-from trytond.i18n import gettext
+from trytond.modules.tryton6_backport.i18n import gettext
-from trytond.tools import cached_property
+from cached_property import cached_property
 from trytond.pool import Pool
-from trytond.modules.product import round_price
+from trytond.modules.product import price_digits
+
+
+def round_price(value, rounding=None):
+    "Round price using the price digits"
+    if isinstance(value, int):
+        return Decimal(value)
+    return value.quantize(
+        Decimal(1) / 10 ** price_digits[1], rounding=rounding)
 
 
 class EdocumentMixin(object):

party.py

@@ -5,10 +5,22 @@
 
 from trytond.pool import PoolMeta
 from trytond.exceptions import UserError
-from trytond.i18n import gettext
+from trytond.modules.tryton6_backport.i18n import gettext
 from trytond.model import fields
 
 
+class PartyIdentifier(metaclass=PoolMeta):
+    __name__ = 'party.identifier'
+
+    @classmethod
+    def __setup__(cls):
+        super(PartyIdentifier, cls).__setup__()
+        cls.type.selection.append(
+            ('edoc_route_id', 'X-Rechnung Route-ID'))
+
+# end PartyIdentifier
+
+
 class Party(metaclass=PoolMeta):
     __name__ = 'party.party'
 
@@ -52,15 +64,3 @@ class Party(metaclass=PoolMeta):
             record.get_xrechnung_route_id()
 
 # end Party
-
-
-class PartyConfiguration(metaclass=PoolMeta):
-    __name__ = 'party.configuration'
-
-    @classmethod
-    def __setup__(cls):
-        super().__setup__()
-        cls.identifier_types.selection.append(
-                ('edoc_route_id', 'X-Rechnung Route-ID'))
-
-# end Configuration

setup.py

@@ -36,7 +36,7 @@ with open(path.join(here, 'versiondep.txt'), encoding='utf-8') as f:
         modversion[l2[0]] = {'min': l2[1], 'max': l2[2], 'prefix': l2[3]}
 
 # tryton-version
-major_version = 7
+major_version = 5
 minor_version = 0
 
 requires = []
@@ -84,6 +84,8 @@ setup(
         'Natural Language :: English',
         'Operating System :: OS Independent',
         'License :: OSI Approved :: GNU General Public License (GPL)',
+        'Programming Language :: Python :: 3.6',
+        'Programming Language :: Python :: 3.7',
         'Programming Language :: Python :: 3.8',
         'Programming Language :: Python :: 3.9',
         'Programming Language :: Python :: 3.10',

template/Factur-X-1.07.2-extended/invoice.xml

@@ -24,7 +24,7 @@ this repository contains the full copyright notices and license terms. -->
         </ram:SpecifiedTaxRegistration>
     </py:def>
     <py:def function="TradeAddress(address)">
-        <ram:PostcodeCode py:if="address.postal_code">${address.postal_code}</ram:PostcodeCode>
+        <ram:PostcodeCode py:if="address.zip">${address.zip}</ram:PostcodeCode>
         <py:with vars="lines = (address.street or '').splitlines()">
             <ram:LineOne py:if="len(lines) > 0">${this.quote_text(lines[0])}</ram:LineOne>
             <ram:LineTwo py:if="len(lines) > 1">${this.quote_text(lines[1])}</ram:LineTwo>

template/XRechnung-2.2/XRechnung_credit.xml

@@ -59,8 +59,8 @@
 
     <py:def function="TaxSubTotal(value)">
         <cac:TaxSubtotal>
-            <cbc:TaxableAmount py:attrs="{'currencyID': value.currency.code}">${this.negate_amount(value.base)}</cbc:TaxableAmount>
+            <cbc:TaxableAmount py:attrs="{'currencyID': value.invoice.currency.code}">${this.negate_amount(value.base)}</cbc:TaxableAmount>
-            <cbc:TaxAmount py:attrs="{'currencyID': value.currency.code}">${this.negate_amount(value.amount)}</cbc:TaxAmount>
+            <cbc:TaxAmount py:attrs="{'currencyID': value.invoice.currency.code}">${this.negate_amount(value.amount)}</cbc:TaxAmount>
             <cac:TaxCategory>
                 ${TaxCategory(value.tax, True)}
             </cac:TaxCategory>

template/XRechnung-2.2/XRechnung_invoice.xml

@@ -59,8 +59,8 @@
 
     <py:def function="TaxSubTotal(value)">
         <cac:TaxSubtotal>
-            <cbc:TaxableAmount py:attrs="{'currencyID': value.currency.code}">${value.base}</cbc:TaxableAmount>
+            <cbc:TaxableAmount py:attrs="{'currencyID': value.invoice.currency.code}">${value.base}</cbc:TaxableAmount>
-            <cbc:TaxAmount py:attrs="{'currencyID': value.currency.code}">${value.amount}</cbc:TaxAmount>
+            <cbc:TaxAmount py:attrs="{'currencyID': value.invoice.currency.code}">${value.amount}</cbc:TaxAmount>
             <cac:TaxCategory>
                 ${TaxCategory(value.tax, True)}
             </cac:TaxCategory>
@@ -90,7 +90,7 @@
   <cbc:CustomizationID>urn:cen.eu:en16931:2017#compliant#urn:xoev-de:kosit:standard:xrechnung_2.1</cbc:CustomizationID>
   <cbc:ID>${this.invoice.number}</cbc:ID>
   <cbc:IssueDate>${this.invoice.invoice_date.isoformat()}</cbc:IssueDate>
-  <cbc:DueDate>${(this.invoice.payment_term_date or this.invoice.invoice_date).isoformat()}</cbc:DueDate>
+  <cbc:DueDate>${this.invoice.invoice_date.isoformat()}</cbc:DueDate>
   <cbc:InvoiceTypeCode>${this.type_code}</cbc:InvoiceTypeCode>
   <cbc:Note py:if="this.invoice_note()">${this.invoice_note()}</cbc:Note>
   <cbc:DocumentCurrencyCode>${this.invoice.currency.code}</cbc:DocumentCurrencyCode>

template/XRechnung-2.3/XRechnung_credit.xml

@@ -59,8 +59,8 @@
 
     <py:def function="TaxSubTotal(value)">
         <cac:TaxSubtotal>
-            <cbc:TaxableAmount py:attrs="{'currencyID': value.currency.code}">${this.negate_amount(value.base)}</cbc:TaxableAmount>
+            <cbc:TaxableAmount py:attrs="{'currencyID': value.invoice.currency.code}">${this.negate_amount(value.base)}</cbc:TaxableAmount>
-            <cbc:TaxAmount py:attrs="{'currencyID': value.currency.code}">${this.negate_amount(value.amount)}</cbc:TaxAmount>
+            <cbc:TaxAmount py:attrs="{'currencyID': value.invoice.currency.code}">${this.negate_amount(value.amount)}</cbc:TaxAmount>
             <cac:TaxCategory>
                 ${TaxCategory(value.tax, True)}
             </cac:TaxCategory>

template/XRechnung-2.3/XRechnung_invoice.xml

@@ -59,8 +59,8 @@
 
     <py:def function="TaxSubTotal(value)">
         <cac:TaxSubtotal>
-            <cbc:TaxableAmount py:attrs="{'currencyID': value.currency.code}">${value.base}</cbc:TaxableAmount>
+            <cbc:TaxableAmount py:attrs="{'currencyID': value.invoice.currency.code}">${value.base}</cbc:TaxableAmount>
-            <cbc:TaxAmount py:attrs="{'currencyID': value.currency.code}">${value.amount}</cbc:TaxAmount>
+            <cbc:TaxAmount py:attrs="{'currencyID': value.invoice.currency.code}">${value.amount}</cbc:TaxAmount>
             <cac:TaxCategory>
                 ${TaxCategory(value.tax, True)}
             </cac:TaxCategory>
@@ -90,7 +90,7 @@
   <cbc:CustomizationID>urn:cen.eu:en16931:2017#compliant#urn:xoev-de:kosit:standard:xrechnung_2.3</cbc:CustomizationID>
   <cbc:ID>${this.invoice.number}</cbc:ID>
   <cbc:IssueDate>${this.invoice.invoice_date.isoformat()}</cbc:IssueDate>
-  <cbc:DueDate>${(this.invoice.payment_term_date or this.invoice.invoice_date).isoformat()}</cbc:DueDate>
+  <cbc:DueDate>${this.invoice.invoice_date.isoformat()}</cbc:DueDate>
   <cbc:InvoiceTypeCode>${this.type_code}</cbc:InvoiceTypeCode>
   <cbc:Note py:if="this.invoice_note()">${this.invoice_note()}</cbc:Note>
   <cbc:DocumentCurrencyCode>${this.invoice.currency.code}</cbc:DocumentCurrencyCode>

template/XRechnung-3.0/XRechnung_credit.xml

@@ -59,8 +59,8 @@
 
     <py:def function="TaxSubTotal(value)">
         <cac:TaxSubtotal>
-            <cbc:TaxableAmount py:attrs="{'currencyID': value.currency.code}">${this.negate_amount(value.base)}</cbc:TaxableAmount>
+            <cbc:TaxableAmount py:attrs="{'currencyID': value.invoice.currency.code}">${this.negate_amount(value.base)}</cbc:TaxableAmount>
-            <cbc:TaxAmount py:attrs="{'currencyID': value.currency.code}">${this.negate_amount(value.amount)}</cbc:TaxAmount>
+            <cbc:TaxAmount py:attrs="{'currencyID': value.invoice.currency.code}">${this.negate_amount(value.amount)}</cbc:TaxAmount>
             <cac:TaxCategory>
                 ${TaxCategory(value.tax, True)}
             </cac:TaxCategory>

template/XRechnung-3.0/XRechnung_invoice.xml

@@ -59,8 +59,8 @@
 
     <py:def function="TaxSubTotal(value)">
         <cac:TaxSubtotal>
-            <cbc:TaxableAmount py:attrs="{'currencyID': value.currency.code}">${value.base}</cbc:TaxableAmount>
+            <cbc:TaxableAmount py:attrs="{'currencyID': value.invoice.currency.code}">${value.base}</cbc:TaxableAmount>
-            <cbc:TaxAmount py:attrs="{'currencyID': value.currency.code}">${value.amount}</cbc:TaxAmount>
+            <cbc:TaxAmount py:attrs="{'currencyID': value.invoice.currency.code}">${value.amount}</cbc:TaxAmount>
             <cac:TaxCategory>
                 ${TaxCategory(value.tax, True)}
             </cac:TaxCategory>
@@ -90,7 +90,7 @@
   <cbc:CustomizationID>urn:cen.eu:en16931:2017#compliant#urn:xoev-de:kosit:standard:xrechnung_3.0</cbc:CustomizationID>
   <cbc:ID>${this.invoice.number}</cbc:ID>
   <cbc:IssueDate>${this.invoice.invoice_date.isoformat()}</cbc:IssueDate>
-  <cbc:DueDate>${(this.invoice.payment_term_date or this.invoice.invoice_date).isoformat()}</cbc:DueDate>
+  <cbc:DueDate>${this.invoice.invoice_date.isoformat()}</cbc:DueDate>
   <cbc:InvoiceTypeCode>${this.type_code}</cbc:InvoiceTypeCode>
   <cbc:Note py:if="this.invoice_note()">${this.invoice_note()}</cbc:Note>
   <cbc:DocumentCurrencyCode>${this.invoice.currency.code}</cbc:DocumentCurrencyCode>

tests/__init__.py

@@ -2,3 +2,17 @@
 # This file is part of the edocument-module for Tryton from m-ds.de.
 # The COPYRIGHT file at the top level of this repository contains the
 # full copyright notices and license terms.
+
+import trytond.tests.test_tryton
+import unittest
+
+from .test_edocument import EdocTestCase
+
+__all__ = ['suite']
+
+
+def suite():
+    suite = trytond.tests.test_tryton.suite()
+    suite.addTests(unittest.TestLoader().loadTestsFromTestCase(
+        EdocTestCase))
+    return suite

tests/test_edocument.py

@@ -17,15 +17,10 @@ from trytond.exceptions import UserError
 def set_invoice_sequences(fiscalyear):
     pool = Pool()
     Sequence = pool.get('ir.sequence.strict')
-    SequenceType = pool.get('ir.sequence.type')
     InvoiceSequence = pool.get('account.fiscalyear.invoice_sequence')
-    ModelData = pool.get('ir.model.data')
 
-    sequence = Sequence(
+    sequence = Sequence(name=fiscalyear.name, code='account.invoice')
-        name=fiscalyear.name,
+    sequence.company = fiscalyear.company
-        sequence_type=SequenceType(ModelData.get_id(
-            'account_invoice', 'sequence_type_account_invoice')),
-        company=fiscalyear.company)
     sequence.save()
     fiscalyear.invoice_sequences = []
     invoice_sequence = InvoiceSequence()
@@ -105,7 +100,7 @@ class EdocTestCase(ModuleTestCase):
             'addresses': [('create', [{
                 'invoice': True,
                 'street': 'Customer Street 1',
-                'postal_code': '12345',
+                'zip': '12345',
                 'city': 'Usertown',
                 'country': country_de.id,
                 }])],

tryton.cfg

@@ -1,10 +1,11 @@
 [tryton]
-version=7.0.10
+version=5.0.14
 depends:
     edocument_uncefact
     party
     bank
     account_invoice
+    tryton6_backport
 xml:
     message.xml
     configuration.xml

versiondep.txt

@@ -1 +1,2 @@
-
+tryton6_backport;5.0.3;5.0.999;mds
+edocument_uncefact;5.0.6;5.0.999;mds

view/configuration_form.xml

@@ -4,7 +4,7 @@
  full copyright notices and license terms. -->
 <data>
 
-    <xpath expr="/form/separator[@id='currency_exchange']" position="before">
+    <xpath expr="/form/separator[@id='invoice']" position="before">
 
         <separator id="edocument" colspan="4" string="eDocument"/>
         <field name="edocument_bank" colspan="2" height="200"/>