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
[11.0] mig l10n_it_fatturapa_in #640
Conversation
@francesca-bianchini da revisionare ed eventualmente includere: #688 |
importando un file xml in fattura fornitore, aggancia un fornitore non inerente modificandogli la email e gli altri dati anagrafici. |
Relativamente ai commenti che avevo riportato in trello https://trello.com/c/ET8WFor5/27-porting-to-11-fatturazione-elettronica-b2b-elaborazione-xml per semplicità riporto qui i commit specifici da aggiungere a questa PR per iniziare l'allineamento alle versioni 10.0 e 12.0. |
@andreampiovesana @labaggio non sono riuscito a riprodurre il produrre il problema. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With this change we're able to import invoice with attachments
content = attach.Attachment | ||
_attach_dict = { | ||
'name': name, | ||
'datas': base64.b64encode(str(content)), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
with Python3 this should be:
'datas': base64.b64encode(content)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indeed see v12:
'datas': base64.b64encode(content), |
This part is covered by automated tests, if I remember well.
Making them green would also fix this.
@dcorio puoi controllare e se va bene togliere la change request? |
Ready to merge? |
Purtroppo questa PR è rimasta un po' indietro rispetto alle modifiche fatte al modulo sulle altre versioni Se volete fare comunque merge, poi magari qualcuno lo correggerà successivamente. In ogni caso @labaggio ci sarebbe da raggruppare i commit per insiemi logici (anche se mi sembrava l'avessimo già fatto) |
Preferisco integrare quanto manca, tra oggi e domani lo faccio.
Si hai ragione, ma una settimana fa l'ho ripresa in mano e probabilmente faccio qualcosa di sbagliato. |
Il problema segnalato #640 (review) mi pare risolto. I test sono verdi. |
porting l10n_it_fatturapa_in to 10 (REF withholding tax integration, invoice interface, fiscal document type ADD some DatiGeneraliDocumento fields) ADD _setTerzoIntermediarioOSoggettoEmittente to l10n_it_fatturapa_out
supplier fields only visible for suppliers Check mandatory customer fields before exporting XML Prevent to use sequence already used by journals
exceptions sending email Avoid to delete sent files Check duplicated files Send several files Reset to ready button Sending and delivery date Handling ACCETTAZIONE and CONSEGNA IMP 'link to existing invoice' wizard FIX invoice views within attachment view IMP attachments views IMP l10n_it_fatturapa_out export: allow multiple export , group by partner ALLOW to use newline char in invoice line description check sequence without date ranges Extract correcttle messages from NS
[IMP] revamp README and manifest [FIX] wrong tabs indentation
) Steps: 1 - Accounting → Purchases → Incoming E-Bill Files 2 - Click on Create 3 - Fill in Attachment Name 4 - Click Save File "/home/odoo/build/OCA/l10n-italy/l10n_it_fatturapa/models/ir_attachment.py", line 117, in get_xml_string if fatturapa_attachment.datas_fname.lower().endswith('.p7m'): AttributeError: 'bool' object has no attribute 'lower'
…fiscal code (=TIN) (#843) First search by VAT number, otherwise by fiscal code. Correctly check commercial partner
ADD PECDestinatario, CodiceDestinatario , FormatoTrasmissione handling ADD Lotto di fatture verso soggetto privato ADD gestione prodotti nelle fatture passive ADD l10n_it_fatturapa_in_purchase ADD einvoice.line model IMP form fatture ADD Livello di dettaglio Fatture elettroniche ADD PDF preview dell'XML ADD Link to existing supplier invoice and handle "registered" XML ADD rappresentante fiscale e stabile organizzazione in emissione gestione prodotti DatiBollo , DatiCassaPrevidenziale e ScontoMaggiorazione CodiceArticolo, ftpa_line_number IMP "show preview" as link FIX invoice duplication ADD constraints and Nome Cognome Avoid to cancel invoice with XML Mostra i dati fattura elettronica solo se il cliente è soggetto IMP READMEs Gestione IVA inclusa nel prezzo ADD l10n_it_fatturapa_out_ddt IMP PEP8 IMP decimal precision ADD Export E-invoice button IMP README
…voice line (#703) * FIX l10n_it_fatturapa_in: do not automatically add Bollo as invoice line; if charged to customer will be added as invoice line * do not automatically add DatiCassaPrevidenziale as invoice line; if charged to customer will be added as invoice line
@labaggio Travis ha dato errore su flake8, nome della classe va in CamelCase ..marunn preciso eh 😬😬 |
self.assertTrue(len(invoice.invoice_line_ids) == 1) | ||
|
||
def test_17_xml_import(self): | ||
res = self.run_wizard('test17', 'IT05979361218_010.xml') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@labaggio questo test mi pare che manchi sia nella 10 che nella 12. Di cosa si tratta?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah ok, grazie
Mi diceva @andreampiovesana che dopo aver aggiornato usando questa PR sono comparsi questi 2 errori qui sotto (il secondo andando in settings) Per risolvere ha dovuto aggiungere quanto presente in Inoltre c'è anche una PR successiva #757 che va a modificare la #664.
|
Non erano inclusi perché successivamente sono stati tolti (anche nelle altre versioni), non credo sia quella la strada corretta..
Ho fatto un confronto tra le versioni attuali della 11 e della 12 dei moduli e le uniche differenze presenti sono nel file /l10n_it_fatturapa/models/account.py ( nella 11.0 non sono stati tolti i campi Il problema, che magari riesci a spiegarmi, è un altro. |
Quindi, se non ho capito male, stai dicendo che per allineare la 11.0 alla 12.0 del modulo fatturapa restano solo quelle due modifiche da includere?
Se riesci ad aprire una PR con quanto resta da fare per allineare le versioni sarebbe ottimo!
Può essere che inizialmente stesse utilizzando direttamente la PR quando era ancora WIP, @andreampiovesana corretto?
Purtroppo non saprei risponderti. Vediamo cosa dicono @OCA/local-italy-maintainers |
Infatti se esegui la seguente query DELETE FROM ir_ui_view WHERE arch_fs
ILIKE '%fatturapa_in%'; e rifai l'aggiornamento del modulo, la cosa si
risolve.
Questa situazione capita solo durante un aggiornamento, installato da zero
non crea problemi.
confermo che questo sql funziona
grazie
Andrea Piovesana linux counter #601470 Linux Mint 19.1www.pnlug.it
Il giorno mer 13 feb 2019 alle ore 14:46 Sergio Zanchetta <
notifications@github.com> ha scritto:
… Ho fatto un confronto tra le versioni attuali della 11 e della 12 dei
moduli e le uniche differenze presenti sono nel file
/l10n_it_fatturapa/models/account.py ( nella 11.0 non sono stati tolti i
campi
virtual_stamp, e stamp_amount) e nella l10n_it_fatturapa/models/company.py
la modifica relativa ai campi fatturapa_rea_office, fatturapa_rea_number,
fatturapa_rea_capital, fatturapa_rea_partner, fatturapa_rea_liquidation)
Quindi, se non ho capito male, stai dicendo che per allineare la 11.0 alla
12.0 del modulo fatturapa restano solo quelle due modifiche da includere?
Queste modifiche posso portarle,
Se riesci ad aprire una PR con quanto resta da fare per allineare le
versioni sarebbe ottimo!
Più che altro è da capire se le modifiche coinvolgono altri moduli
collegati.
ma nessuna di queste modifiche ha creato l'errore indicato da
@andreampiovesana <https://github.com/andreampiovesana> .
Può essere che inizialmente stesse utilizzando direttamente la PR quando
era ancora WIP, @andreampiovesana <https://github.com/andreampiovesana>
corretto?
Il problema, che magari riesci a spiegarmi, è un altro.
In una delle varie modifiche del modulo che era in progress, sono stati
eliminati i campi dal model, anche dagli xml però.
Non so perché durante l'aggiornamento il database non si è aggiornato
correttamente non eliminando i riferimenti.
Infatti se esegui la seguente query DELETE FROM ir_ui_view WHERE arch_fs
ILIKE '%fatturapa_in%'; e rifai l'aggiornamento del modulo, la cosa si
risolve.
Questa situazione capita solo durante un aggiornamento, installato da zero
non crea problemi.
Se riuscite ad illuminarmi faccio subito una modifica.
Purtroppo non saprei risponderti.
Vediamo cosa dicono @OCA/local-italy-maintainers
<https://github.com/orgs/OCA/teams/local-italy-maintainers>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#640 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ANTkg5BMVLT5_WZhv6GFgVt87F6Rl8yeks5vNBc0gaJpZM4YQCwK>
.
|
In progress.
It depends on: