search

OCA / l10n-italy

Odoo Italian localization
https://www.odoo-italia.org
GNU Affero General Public License v3.0
154 stars 306 forks source link

Migration to version 14.0 #1905

Open OCA-git-bot opened 4 years ago

OCA-git-bot commented 4 years ago

Todo

https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-14.0

Modules to migrate

Missing module? Check https://github.com/OCA/maintainer-tools/wiki/%5BFAQ%5D-Missing-modules-in-migration-issue-list

Per la categoria dei moduli: Localization/Italy

corrispondenza account.invoice con account.move v14

Se un modulo viene rinominato da ITA a ENG, per poter testare lo script di migrazione si possono seguire i seguenti step:

dcorio commented 4 years ago

Anche il modulo l10n_it_reverse_charge richiede migrazione. E' probabile necessiti di un parziale refactoring per far si che "integrazione iva" funzioni correttamente (se ricordo bene non vengono valorizzati correttamente i registri iva). Si crea prima una issue e poi aggiungiamo il modulo qui linkando la issue?

tafaRU commented 4 years ago

@dcorio suggerisco di utilizzare la procedura standard: link alla PR della migrazione a fianco del modulo presente nell'elenco di https://github.com/OCA/l10n-italy/issues/1905#issue-714284052 Altrimenti, se ritieni necessario, puoi farla precedere da una issue [RFC] (rif. https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/oca_module_lifecycle_development_status.rst#2alpha-and-beta-modules) .

Borruso commented 4 years ago

sto portando aventi l10n_it_ipa

SimoRubi commented 4 years ago

Inizio a migrare l10n_it_fiscalcode

Borruso commented 4 years ago

Inizio a migrare l10n_it_fiscalcode

Avevo iniziato a vedere il modulo ma falliscono i test, ti passo i file?

SimoRubi commented 4 years ago

No continua pure tu, l'avevo preso solo perché era il primo libero. Allora inizio l10n_it_rea

jado95 commented 4 years ago

@SimoRubi stavo vedendo io il modulo l10n_it_rea non l'avevo scritto qui perchè l'aveva detto @As400it in call

MarcoCalcagni commented 4 years ago

parte del lavoro di oggi [14.0] Migration: l10n_it_fiscalcode #1931 [14.0] mig l10n_it_rea #1930 [14.0] Migration: l10n_it_ipa #1929

eLBati commented 4 years ago

@dcorio @tafaRU https://github.com/OCA/l10n-italy/issues/1937

Borruso commented 4 years ago

Inizio a migrare l10n_it_esigibilita_iva

Borruso commented 4 years ago

Inizio a migrare l10n_it_esigibilita_iva

[14.0] Migration: l10n_it_esigibilita_iva #1938

Borruso commented 4 years ago

Inizio a migrare l10n_it_fiscal_payment_term

Borruso commented 4 years ago

Inizio a migrare l10n_it_fiscal_payment_term

[14.0] Migration: l10n_it_fiscal_payment_term #1939

Borruso commented 4 years ago

Inizio a migrare l10n_it_split_payment

fredzamoabg commented 4 years ago

inizio a migrare l10n_it_fiscal_document_type

fredzamoabg commented 4 years ago

[14.0] Migration: l10n_it_fiscal_document_type #1942

SimoRubi commented 4 years ago

account_fiscal_year: https://github.com/OCA/account-financial-tools/pull/1081

jado95 commented 4 years ago

Se nessuno sta migrando l10n_it_account_tax_kind, posso iniziare a fare io la migrazione

jado95 commented 4 years ago

Visto che l10n_it_account_tax_kind dipende da l10n_it_account inizio da quest'ultimo

Borruso commented 4 years ago

Inizio a migrare l10n_it_split_payment

[14.0] Migration: l10n_it_split_payment #1959 Bisogna riscrivere i test. C'è qualcuno che mi può aiutare ??

fredzamoabg commented 4 years ago

inizio l10n_it_delivery_note_base

fredzamoabg commented 4 years ago

14.0 mig l10n it delivery note base #1962

fredzamoabg commented 4 years ago

inizio l10n_it_delivery_note

primes2h commented 4 years ago

Riporto anche qui https://github.com/OCA/l10n-italy/pull/1938#discussion_r531601847

fredzamoabg commented 3 years ago

inizio l10n_it_delivery_note_batch

Borruso commented 3 years ago

inizio l10n_it_delivery_note_order_link

jado95 commented 3 years ago

@eLBati PR per l10n_it_account_tax_kind: #1973

jado95 commented 3 years ago

inizio l10n_it_account_stamp

Borruso commented 3 years ago

inizio l10n_it_delivery_note_order_link

[14.0] Migration: l10n_it_delivery_note_order_link #1974

fredzamoabg commented 3 years ago

14.0 mig l10n it delivery note batch #1975

TheMule71 commented 3 years ago

Per la cronaca, con l'aggiunta di https://github.com/SimoRubi/account-financial-tools/tree/14.0-mig-account_fiscal_year, questo branch passa tutti i test (dei nostri moduli): https://github.com/TheMule71/l10n-italy/tree/14.0-supermerge Merge branches '14.0-mig-l10n_it_account', '14.0-mig-l10n_it_account_tax_kind', '14.0-mig-l10n_it_esigibilita_iva', '14.0-mig-l10n_it_fiscal_payment_term', '14.0-mig-l10n_it_split_payment' and '14.0-mig-l10n_it_fiscal_document_type'

TheMule71 commented 3 years ago

Aperta #1984 per l10n_it_fatturapa. Occhio che il Grande Fratello (quello di Orwell, non quello della Endemol) vi osserva.

TheMule71 commented 3 years ago

Aperta #1985 per l10n_it_fatturapa_out. C'è ancora da lavorarci.

TheMule71 commented 3 years ago

https://github.com/TheMule71/l10n-italy/tree/14.0-supermerge

aggiornato. Non ho ancora visto i test automatici ma fatturapa-out adesso produce qualcosa: IT06363391001_ocWxU.zip (non so quanto sia corretta al 100% ma pare giusta) 3368.pdf

taifu commented 3 years ago

Si vede (grande!) ma c'è ancora qualche errorino: DOMDocument::schemaValidate() Generated Errors! Error 1871: Element 'Divisa': This element is not expected. Expected is ( TipoDocumento ). on line 2 Error 1871: Element 'DettaglioPagamento': This element is not expected. Expected is ( CondizioniPagamento ). on line 2 Error 1845: Element '{http://ivaservizi.agenziaentrate.gov.it/docs/xsd/fatture/v1.2}FatturaElettronica': No matching global declaration available for the validation root. on line 2

TheMule71 commented 3 years ago

Si alcuni campi hanno cambiato nome, per cui non li trova. L'idea è che a tendere la si validi con l'xsd.

TheMule71 commented 3 years ago

Rettifico, in realtà mancavano dati nella fattura. La funzione che fa le verifiche pre-export va estesa per prevedere tutti i casi, e per estesa intendo che adesso è così:

    def preventive_checks(self):
        # hook for preventive checks. Override and raise exception, in case
        return

Mi pare di capire che prima ci si basasse sugli errori di PyXB, in fase di creazione del documento.

Ho aggiunto la validazione all'export. In pratica è la stessa cosa, riporta un errore in caso di campi mancanti. L'errore è quello generato dalla libreria (lxml) al momento, ovviamente andrebbero effettuati di controlli preventivi magari indicando cosa esattamente manca ed eventualmente come aggiungerlo.

Io in questo caso ho dovuto configurare il tipo di documento e il codice della tipologia di pagamento (del resto i dati demo li ha generati prima che io installassi i nostri moduli). IT06363391001_ojgm7.zip

XML_CATALOG_FILES="/home/marco/.config/xmlcatalog /etc/xml/catalog" xmllint --schema ../src/odoo/OCA/l10n-italy/l10n_it_fatturapa/bindings/xsd/Schema_del_file_xml_FatturaPA_versione_1.2.1.xsd --noout IT06363391001_ojgm7.xml 
IT06363391001_ojgm7.xml validates
TheMule71 commented 3 years ago

Breve elenco di cose da controllare.

[14.0][MIG] l10n_it_causali_pagamento #1987 [14.0][MIG] l10n_it_withholding_tax #1988 [14.0][MIG] l10n_it_withholding_tax_causali #1989 https://github.com/TheMule71/l10n-italy/tree/14.0-mig-l10n_it_fiscal_document_type [14.0][MIG] Migration: l10n_it_fatturapa #1984 [14.0] [MIG] l10n_it_fatturapa_out #1985 [14.0][MIG] l10n_it_fatturapa_in #1990

SimoRubi commented 3 years ago
Borruso commented 3 years ago

l10n_it_fatturapa_out_stamp #2044

dcorio commented 3 years ago

Inizio a lavorare su l10n_it_reverse_charge

eLBati commented 3 years ago

@dcorio :+1: Vedi https://github.com/OCA/l10n-italy/issues/1937 : in base alle nuove specifiche, potrebbe convenire rimuovere il metodo "Integrazione IVA" (e forse semplificare il metodo autofattura riducendo un po' gli scenari) cc @sergiocorato

dcorio commented 3 years ago

1937

Grazie @eLBati , ne discuto con @sergiocorato

eLBati commented 3 years ago

È stato creato il branch https://github.com/odoo-italia/l10n-italy/tree/14.0-supermerge per raccogliere tutte le PR aperte in un unico branch e così testare più agevolmente i moduli sulla 14 (grazie @TheMule71 )

scigghia commented 3 years ago

PR Ricevute Bancarie https://github.com/OmniaGit/l10n-italy/tree/14.0_develop

Borruso commented 3 years ago

l10n_it_fatturapa_export_zip https://github.com/OCA/l10n-italy/pull/2080 l10n_it_abicab https://github.com/OCA/l10n-italy/pull/2079

TheMule71 commented 3 years ago

Ho splittato parte del lavoro di https://github.com/mboscolo all'interno del supermerge in branch separati. Qualche anima pia potrebbe partire da lì per creare le PR corrispondenti...

https://github.com/odoo-italia/l10n-italy/tree/14.0-mig-l10n_it_fatturapa_export_zip attenzione! di questa esiste già #2080 ma non sono parenti... al momento non è possibile mergiare la #2080 nell'attuale supermerge, ma vd. sotto... https://github.com/odoo-italia/l10n-italy/tree/14.0-mig-l10n_it_fatturapa_import_zip https://github.com/odoo-italia/l10n-italy/tree/14.0-mig-l10n_it_ricevute_bancarie

Vd. anche https://github.com/odoo-italia/l10n-italy/blob/14.0-premerge/supermerge.sh

NOTA BENE: l'attuale supermerge è frutto in un'evoluzione che l'ha portato ad avere vita propria (con le sue PR, mergiate e non). In questi giorni mi sono sforzato di riallineare le cose portando le diff a mano nelle varie PR di competenza, ma chiaramente non è "ecosostenibile", così divento io un bottleneck per lo sviluppo.

Idealmente, vorrei rigenerare il supermerge come da script. Al momento (escluso l10n_it_fatturapa_export_zip i cui file collidono brutalmente tra le due varianti e ne va scelta una) non ci sono così tante differenze, la maggior parte delle fix le ho riportate. Se non volete lanciare lo script, il risultato (a oggi) lo vedete qui: https://github.com/TheMule71/l10n-italy/tree/14.0-supermerge-clean

fredzamoabg commented 3 years ago

per quanto riguarda l10n_it_delivery_note_base e l10n_it_delivery_note ho creato 2 PR separate: l10n_it_delivery_note_base https://github.com/OCA/l10n-italy/pull/2140 l10n_it_delivery_note https://github.com/OCA/l10n-italy/pull/2141

questa https://github.com/OCA/l10n-italy/pull/1962 che conteneva entrambi i moduli può essere chiusa

TheMule71 commented 3 years ago

Aggiornato https://github.com/odoo-italia/l10n-italy/blob/14.0-supermerge/supermerge.sh con #2138 #2139 #2140 #2141

TheMule71 commented 3 years ago

[14.0][MIG] l10n_it_account_stamp: Migration to 14.0 #2150 rimossa vd. #2077