trytond-edocument-ubl 8.0.3__tar.gz → 8.0.5__tar.gz

{trytond_edocument_ubl-8.0.3 → trytond_edocument_ubl-8.0.5}/PKG-INFO

@@ -1,6 +1,6 @@
-Metadata-Version: 2.4
+Metadata-Version: 2.5
 Name: trytond_edocument_ubl
-Version: 8.0.3
+Version: 8.0.5
 Summary: UBL electronic document for Tryton
 Project-URL: homepage, https://www.tryton.org/
 Project-URL: documentation, https://docs.tryton.org/modules-edocument-ubl/

{trytond_edocument_ubl-8.0.3 → trytond_edocument_ubl-8.0.5}/edocument.py

@@ -321,11 +321,24 @@ class Invoice(Model):
                     type_code=type_code))
 
         invoice = Invoice(type='in')
+
+        if (customer_party := root.find('./{*}AccountingCustomerParty')
+                ) is not None:
+            invoice.company = cls._parse_2_company(customer_party)
+        else:
+            invoice.company = Invoice.default_company()
+        if not invoice.company:
+            raise InvoiceError(gettext(
+                    'edocument_ubl.msg_company_not_found',
+                    company=etree.tostring(
+                        customer_party, pretty_print=True).decode()
+                    if customer_party else ''))
+
         invoice.reference = root.findtext('./{*}ID')
         invoice.invoice_date = dt.date.fromisoformat(
             root.findtext('./{*}IssueDate'))
         invoice.party = cls._parse_2_supplier(
-            root.find('./{*}AccountingSupplierParty'), create=True)
+            root.find('./{*}AccountingSupplierParty'), create=True).id
         payees = [invoice.party]
         invoice.set_journal()
         invoice.on_change_party()
@@ -341,17 +354,6 @@ class Invoice(Model):
             supplier = cls._parse_2_supplier(seller)
         else:
             supplier = invoice.party
-        if (customer_party := root.find('./{*}AccountingCustomerParty')
-                ) is not None:
-            invoice.company = cls._parse_2_company(customer_party)
-        else:
-            invoice.company = Invoice.default_company()
-        if not invoice.company:
-            raise InvoiceError(gettext(
-                    'edocument_ubl.msg_company_not_found',
-                    company=etree.tostring(
-                        customer_party, pretty_print=True).decode()
-                    if customer_party else ''))
 
         if (payee_party := root.find('./{*}PayeeParty')) is not None:
             party = cls._parse_2_party(payee_party)
@@ -382,6 +384,9 @@ class Invoice(Model):
             root.findall('./{*}PaymentMeans'), payees=payees)
         invoice.payment_term_date = cls._parse_2_payment_term_date(
             root.findall('./{*}PaymentTerms'))
+        if not invoice.payment_term_date:
+            if due_date := root.findtext('./{*}DueDate'):
+                invoice.payment_term_date = dt.date.fromisoformat(due_date)
         lines = [
             cls._parse_invoice_2_line(
                 line,
@@ -460,7 +465,9 @@ class Invoice(Model):
         if (invoiced_quantity := invoice_line.find('./{*}InvoicedQuantity')
                 ) is not None:
             line.quantity = float(invoiced_quantity.text)
-            digits = -Decimal(invoiced_quantity.text).as_tuple().exponent
+            digits = (
+                -Decimal(invoiced_quantity.text)
+                .normalize().as_tuple().exponent)
             if (unit_code := invoiced_quantity.get('unitCode')) not in {
                     None, 'ZZ', 'XZZ'}:
                 try:
@@ -654,11 +661,24 @@ class Invoice(Model):
                     type_code=type_code))
 
         invoice = Invoice(type='in')
+
+        if (customer_party := root.find('./{*}AccountingCustomerParty')
+                ) is not None:
+            invoice.company = cls._parse_2_company(customer_party)
+        else:
+            invoice.company = Invoice.default_company()
+        if not invoice.company:
+            raise InvoiceError(gettext(
+                    'edocument_ubl.msg_company_not_found',
+                    company=etree.tostring(
+                        customer_party, pretty_print=True).decode()
+                    if customer_party else ''))
+
         invoice.reference = root.findtext('./{*}ID')
         invoice.invoice_date = dt.date.fromisoformat(
             root.findtext('./{*}IssueDate'))
         invoice.party = cls._parse_2_supplier(
-            root.find('./{*}AccountingSupplierParty'), create=True)
+            root.find('./{*}AccountingSupplierParty'), create=True).id
         payees = [invoice.party]
         invoice.set_journal()
         invoice.on_change_party()
@@ -666,11 +686,7 @@ class Invoice(Model):
             supplier = cls._parse_2_supplier(seller)
         else:
             supplier = invoice.party
-        if (customer_party := root.find('./{*}AccountingCustomerParty')
-                ) is not None:
-            invoice.company = cls._parse_2_company(customer_party)
-        else:
-            invoice.company = Invoice.default_company()
+
         if (payee_party := root.find('./{*}PayeeParty')) is not None:
             party = cls._parse_2_party(payee_party)
             if not party:
@@ -695,6 +711,9 @@ class Invoice(Model):
             root.findall('./{*}PaymentMeans'), payees=payees)
         invoice.payment_term_date = cls._parse_2_payment_term_date(
             root.findall('./{*}PaymentTerms'))
+        if not invoice.payment_term_date:
+            if due_date := root.findtext('./{*}DueDate'):
+                invoice.payment_term_date = dt.date.fromisoformat(due_date)
         lines = [
             cls._parse_credit_note_2_line(
                 line, company=invoice.company, currency=invoice.currency,
@@ -771,7 +790,9 @@ class Invoice(Model):
         if (credited_quantity := credit_note_line.find('./{*}CreditedQuantity')
                 ) is not None:
             line.quantity = -float(credited_quantity.text)
-            digits = -Decimal(credited_quantity.text).as_tuple().exponent
+            digits = (
+                -Decimal(credited_quantity.text)
+                .normalize().as_tuple().exponent)
             if (unit_code := credited_quantity.get('unitCode')) not in {
                     None, 'ZZ', 'XZZ'}:
                 try:
@@ -1070,9 +1091,8 @@ class Invoice(Model):
         address.building_number = address_el.findtext('./{*}BuildingNumber')
         address.city = address_el.findtext('./{*}CityName')
         address.postal_code = address_el.findtext('./{*}PostalZone')
-        if (country_code := address_el.findtext(
-                    './{*}Country/{*}IdentificationCode[@listId="ISO3166-1"]')
-                ) is not None:
+        if country_code := address_el.findtext(
+                './{*}Country/{*}IdentificationCode[@listId="ISO3166-1"]'):
             try:
                 country, = Country.search([
                         ('code', '=', country_code),
@@ -1188,8 +1208,7 @@ class Invoice(Model):
     def _parse_2_payment_term_date(cls, payment_terms):
         dates = []
         for payment_term in payment_terms:
-            if (date := payment_term.findtext('./{*}PaymentDueDate')
-                    ) is not None:
+            if date := payment_term.findtext('./{*}PaymentDueDate'):
                 dates.append(dt.date.fromisoformat(date))
         return min(dates, default=None)
 
@@ -1212,8 +1231,7 @@ class Invoice(Model):
                     pass
                 else:
                     return product
-        if (code := item.findtext('./{*}BuyersItemIdentification/{*}ID')
-                ) is not None:
+        if code := item.findtext('./{*}BuyersItemIdentification/{*}ID'):
             try:
                 product, = Product.search([
                         ('code', '=', code),
@@ -1314,13 +1332,16 @@ class Invoice(Model):
                         ) is not None:
                     mime_code = (
                         data.get('mimeCode') or 'application/octet-stream')
-                    name += mimetypes.guess_extension(mime_code) or ''
+                    if extension := mimetypes.guess_extension(mime_code):
+                        if not name.lower().endswith(extension):
+                            name += extension
                     attachment.type = 'data'
                     data = b64decode(data.text)
                     attachment.data = data
                 elif data := document.findtext(
                         './{*}Attachment/{*}EmbeddedDocument'):
-                    name += '.txt'
+                    if not name.lower().endswith('.txt'):
+                        name += '.txt'
                     attachment.type = 'data'
                     attachment.data = data
                 elif url := document.findtext(

{trytond_edocument_ubl-8.0.3 → trytond_edocument_ubl-8.0.5}/tests/scenario_ubl_2_credit_note_parse.rst

@@ -62,7 +62,7 @@ Parse the UBL invoice::
     >>> with file_open(
     ...         'edocument_ubl/tests/UBL-CreditNote-2.1-Example.xml',
     ...         mode='rb') as fp:
-    ...     invoice_id = EInvoice.parse(fp.read(), config.context)
+    ...     invoice_id = EInvoice.parse(fp.read(), {**config.context, 'company': None})
 
     >>> invoice = Invoice(invoice_id)
 

{trytond_edocument_ubl-8.0.3 → trytond_edocument_ubl-8.0.5}/tests/scenario_ubl_2_invoice_parse.rst

@@ -62,7 +62,7 @@ Parse the UBL invoice::
     >>> with file_open(
     ...         'edocument_ubl/tests/UBL-Invoice-2.1-Example.xml',
     ...         mode='rb') as fp:
-    ...     invoice_id = EInvoice.parse(fp.read(), config.context)
+    ...     invoice_id = EInvoice.parse(fp.read(), {**config.context, 'company': None})
 
     >>> invoice = Invoice(invoice_id)
 

{trytond_edocument_ubl-8.0.3 → trytond_edocument_ubl-8.0.5}/tryton.cfg

@@ -1,5 +1,5 @@
 [tryton]
-version=8.0.3
+version=8.0.5
 depends:
     edocument_unece
     ir