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

[14.0] [MIG] l10n_it_fatturapa_out #1985

Merged
merged 206 commits into from
May 28, 2021

Conversation

TheMule71
Copy link
Contributor

Porting del modulo l10n_it_fatturapa_out dalla versione 12.0 alla 14.0

Da verificare test.

--
Confermo di aver firmato il CLA https://odoo-community.org/page/cla e di aver letto le linee guida su https://odoo-community.org/page/contributing

@TheMule71 TheMule71 changed the title 14.0 mig l10n it fatturapa out [14.0] [MIG] l10n_it_fatturapa_out Dec 15, 2020
@TheMule71 TheMule71 mentioned this pull request Dec 15, 2020
74 tasks
@TheMule71 TheMule71 force-pushed the 14.0-mig-l10n_it_fatturapa_out branch 6 times, most recently from 43781be to cd73c63 Compare December 18, 2020 13:15
@TheMule71 TheMule71 force-pushed the 14.0-mig-l10n_it_fatturapa_out branch 2 times, most recently from 7ea0870 to c0c71ee Compare January 15, 2021 14:05
@victoralmau
Copy link
Member

Please, include these change: #2051

@TheMule71 TheMule71 force-pushed the 14.0-mig-l10n_it_fatturapa_out branch 2 times, most recently from fc68faf to abb9c88 Compare February 12, 2021 18:18
@TheMule71 TheMule71 force-pushed the 14.0-mig-l10n_it_fatturapa_out branch 2 times, most recently from a8062ee to 41df749 Compare February 25, 2021 16:53
@dcorio
Copy link
Contributor

dcorio commented Mar 1, 2021

Ciao @TheMule71,
con l'ultimo commit mi capita il seguente errore (con qualsiasi versione di xmlschema).
Il motivo è che la cartella bindings è stata rinominata in data:

File "/opt/odoo/oca/l10n-italy/l10n_it_fatturapa_out/init.py", line 4, in
from . import wizard
File "/opt/odoo/oca/l10n-italy/l10n_it_fatturapa_out/wizard/init.py", line 2, in
from . import efattura
File "/opt/odoo/oca/l10n-italy/l10n_it_fatturapa_out/wizard/efattura.py", line 54, in
class EFatturaOut:
File "/opt/odoo/oca/l10n-italy/l10n_it_fatturapa_out/wizard/efattura.py", line 56, in EFatturaOut
_validator = FPAValidator()
File "/opt/odoo/oca/l10n-italy/l10n_it_fatturapa_out/wizard/efattura.py", line 43, in init
loglevel=20,
File "/usr/local/lib/python3.7/dist-packages/xmlschema/validators/schema.py", line 319, in init
self.source = XMLResource(source, base_url, allow, defuse, timeout)
File "/usr/local/lib/python3.7/dist-packages/xmlschema/resources.py", line 385, in init
self.parse(source, lazy)
File "/usr/local/lib/python3.7/dist-packages/xmlschema/resources.py", line 657, in parse
"or a file-like object is required." % type(source)
xmlschema.exceptions.XMLSchemaTypeError: wrong type <class 'bool'> for 'source' attribute: an ElementTree object or an Element instance or a string containing XML data or an URL or a file-like object is required.

@dcorio
Copy link
Contributor

dcorio commented Mar 1, 2021

@TheMule71 TheMule71#4

@TheMule71
Copy link
Contributor Author

TheMule71 commented Mar 1, 2021

Ciao @TheMule71,
con l'ultimo commit mi capita il seguente errore (con qualsiasi versione di xmlschema).
Il motivo è che la cartella bindings è stata rinominata in data:

File "/opt/odoo/oca/l10n-italy/l10n_it_fatturapa_out/init.py", line 4, in
from . import wizard
File "/opt/odoo/oca/l10n-italy/l10n_it_fatturapa_out/wizard/init.py", line 2, in
from . import efattura
File "/opt/odoo/oca/l10n-italy/l10n_it_fatturapa_out/wizard/efattura.py", line 54, in
class EFatturaOut:
File "/opt/odoo/oca/l10n-italy/l10n_it_fatturapa_out/wizard/efattura.py", line 56, in EFatturaOut
_validator = FPAValidator()
File "/opt/odoo/oca/l10n-italy/l10n_it_fatturapa_out/wizard/efattura.py", line 43, in init
loglevel=20,
File "/usr/local/lib/python3.7/dist-packages/xmlschema/validators/schema.py", line 319, in init
self.source = XMLResource(source, base_url, allow, defuse, timeout)
File "/usr/local/lib/python3.7/dist-packages/xmlschema/resources.py", line 385, in init
self.parse(source, lazy)
File "/usr/local/lib/python3.7/dist-packages/xmlschema/resources.py", line 657, in parse
"or a file-like object is required." % type(source)
xmlschema.exceptions.XMLSchemaTypeError: wrong type <class 'bool'> for 'source' attribute: an ElementTree object or an Element instance or a string containing XML data or an URL or a file-like object is required.

Grazie,
errore mio.
Il commit c'era, non avevo pushato il mio branch.
Pushato e mergiato nel supermerge...
TheMule71@3000991

@TheMule71 TheMule71 force-pushed the 14.0-mig-l10n_it_fatturapa_out branch from 75c6fff to 4620db8 Compare March 5, 2021 15:02
@TheMule71 TheMule71 force-pushed the 14.0-mig-l10n_it_fatturapa_out branch 2 times, most recently from 73981bd to d65e183 Compare April 2, 2021 09:20
@TheMule71 TheMule71 force-pushed the 14.0-mig-l10n_it_fatturapa_out branch 2 times, most recently from 9ad713c to a411edb Compare April 9, 2021 16:34
],
}
)
self.assertTrue(invoice.fiscal_document_type_id)
Copy link
Contributor

Choose a reason for hiding this comment

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

ho provato ad eseguire i test in locale ma falliscono perchè non viene trovato il campo fiscal_document_type_id in account.move. Manca la dipendenza da l10n_it_fiscal_document_type nel manifest

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ho eliminato le assert. Era l'unica cosa che creava la dipendenza.

Comment on lines 97 to 114
if any([ia.is_pdf_invoice_print for ia in invoice_attachments]):
continue
else:
break
Copy link
Contributor

@fredzamoabg fredzamoabg Apr 15, 2021

Choose a reason for hiding this comment

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

questo si può ridurre a:
if not ... : break

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fatto.

Copy link
Contributor

@fredzamoabg fredzamoabg left a comment

Choose a reason for hiding this comment

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

vedi i 2 commenti che ti ho lasciato oltre alla pre-commit fallita. per il resto mi sembra ok!

@TheMule71 TheMule71 force-pushed the 14.0-mig-l10n_it_fatturapa_out branch from a411edb to 4e6ce10 Compare April 16, 2021 09:35
<DatiGenerali>
<DatiGeneraliDocumento>
<!--2.1.1-->
<TipoDocumento t-esc="record.fiscal_document_type_id.code" />
Copy link
Contributor

Choose a reason for hiding this comment

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

anche qui c'è una dipendenza da fiscal_document_type

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Rimessa dipendenza.

@TheMule71 TheMule71 force-pushed the 14.0-mig-l10n_it_fatturapa_out branch from 0324670 to 0bf7fa6 Compare April 30, 2021 16:25
@TheMule71 TheMule71 deleted the 14.0-mig-l10n_it_fatturapa_out branch May 28, 2021 15:26
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request May 28, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request May 28, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request May 28, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 6, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 18, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 18, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 2, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 2, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 9, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 17, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 20, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 30, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jul 30, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Aug 6, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Aug 20, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Sep 3, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Sep 17, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 1, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 8, 2021
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Oct 15, 2021
TheMule71 added a commit to TheMule71/l10n-italy that referenced this pull request Jan 21, 2022
TheMule71 added a commit to TheMule71/l10n-italy that referenced this pull request Jan 21, 2022
TheMule71 added a commit to odoo-italia/l10n-italy that referenced this pull request Jun 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.