Skip to content

[12.0][IMP] new module l10n_it_fatturapa_out_di#2518

Merged
OCA-git-bot merged 1 commit into
OCA:12.0from
TheMule71:12.0-l10n_it_fatturapa_out_di
Dec 13, 2021
Merged

[12.0][IMP] new module l10n_it_fatturapa_out_di#2518
OCA-git-bot merged 1 commit into
OCA:12.0from
TheMule71:12.0-l10n_it_fatturapa_out_di

Conversation

@TheMule71

Copy link
Copy Markdown
Contributor

Modulo ponte tra l10n_it_dichiarazione_intento e l10n_it_fatturapa_out

vd #2516

@TheMule71

Copy link
Copy Markdown
Contributor Author

Sembra che il set_sequences() non funzioni a dovere nei test. Se qualcuno ricorda come funziona...

Comment thread l10n_it_fatturapa_out_di/wizard/wizard_export_fatturapa.py
Comment thread l10n_it_fatturapa_out_di/tests/data/IT06363391001_00004.xml
Comment thread l10n_it_fatturapa_out_di/tests/data/IT06363391001_00004.xml Outdated
Comment thread l10n_it_fatturapa_out_di/tests/test_fatturapa_xml_validation.py Outdated
Comment thread l10n_it_fatturapa_out_di/tests/test_fatturapa_xml_validation.py Outdated
Comment thread l10n_it_fatturapa_out_di/tests/test_fatturapa_xml_validation.py Outdated
Comment thread l10n_it_fatturapa_out_di/tests/test_fatturapa_xml_validation.py Outdated
Comment thread l10n_it_fatturapa_out_di/tests/test_fatturapa_xml_validation.py Outdated
Comment thread l10n_it_fatturapa_out_di/tests/data/IT06363391001_00004.xml
Comment thread l10n_it_fatturapa_out_di/tests/data/IT06363391001_00004.xml
Comment thread l10n_it_fatturapa_out_di/tests/data/IT06363391001_00004.xml
@TheMule71

TheMule71 commented Nov 27, 2021

Copy link
Copy Markdown
Contributor Author

Sto facendo anche una ricerca su come fanno altri gestionali.
Un'alternativa, in effetti, potrebbe essere quella di indicare, invece che riga per riga, <AltriDatiGestionali>, aggiungere una riga note alla fine della fattura, avente N <AltriDatiGestionali> con le varie lettere d'intento.

@sergiocorato

Copy link
Copy Markdown
Contributor

Sto facendo anche una ricerca su come fanno altri gestionali. Un'alternativa, in effetti, potrebbe essere quella di indicare, invece che riga per riga, <AltriDatiGestionali>, aggiungere una riga note alla fine della fattura, avente N <AltriDatiGestionali> con le varie lettere d'intento.

E la fattura passa il controllo? Dalla documentazione non si capisce, l'intero tag viene definito come facoltativo, però nel Provvedimento antifrode del 28/10/21 scrivono genericamente deve essere compilato un blocco 2.2.1.16 <AltriDatiGestionali> per ogni dichiarazione d’intento quindi direi che è fattibile.

@TheMule71 TheMule71 force-pushed the 12.0-l10n_it_fatturapa_out_di branch from a49af6f to a4cb78d Compare November 27, 2021 17:24
@TheMule71

Copy link
Copy Markdown
Contributor Author

Si potrebbe implementare la seguente logica:

  • le DI collegate alle righe in modo esplicito le riportiamo con <AltriCampiGestionali> nella riga di competenza
  • le DI collegate alla fattura (meno quelle esplicite sopra) le riportiamo con <AltriCampiGestionali> in una riga nota alla fine della fattura

@sergiocorato

Copy link
Copy Markdown
Contributor

Si potrebbe implementare la seguente logica:

* le DI collegate alle righe in modo esplicito le riportiamo con `<AltriCampiGestionali>` nella riga di competenza

* le DI collegate alla fattura (meno quelle esplicite sopra) le riportiamo con `<AltriCampiGestionali>` in una riga nota alla fine della fattura

Così è molto preciso, ottimo.

@sergiocorato

Copy link
Copy Markdown
Contributor

Sembra che il set_sequences() non funzioni a dovere nei test. Se qualcuno ricorda come funziona...

A me pare che funzioni come lo hai impostato

@TheMule71 TheMule71 force-pushed the 12.0-l10n_it_fatturapa_out_di branch from a4cb78d to e75bc13 Compare November 27, 2021 18:12
@TheMule71

TheMule71 commented Nov 27, 2021

Copy link
Copy Markdown
Contributor Author

Si potrebbe implementare la seguente logica:

* le DI collegate alle righe in modo esplicito le riportiamo con `<AltriCampiGestionali>` nella riga di competenza

* le DI collegate alla fattura (meno quelle esplicite sopra) le riportiamo con `<AltriCampiGestionali>` in una riga nota alla fine della fattura

Così è molto preciso, ottimo.

Ho appena pushato una bozza. Nel file XML del test c'è una sola causale. Ora, il modulo base (dichiarazione_intento) dovrebbe popolare lui il campo commento della fattura coi dati giusti. Se c'è da correggere il comportamento va fatto di là, credo. (vd test_3)

Qui ci limitiamo a generare gli <AltriDatiGestionali> che servono, al posto giusto.

@TheMule71

Copy link
Copy Markdown
Contributor Author

Sembra che il set_sequences() non funzioni a dovere nei test. Se qualcuno ricorda come funziona...

A me pare che funzioni come lo hai impostato

Si, probabilmente ero io cotto l'altra volta.

@TheMule71

Copy link
Copy Markdown
Contributor Author

Ah proposito, i file XML andrebbero testati con lo SdI, ovviamente.

@TheMule71 TheMule71 force-pushed the 12.0-l10n_it_fatturapa_out_di branch from e75bc13 to ecc414c Compare November 27, 2021 18:24
@TheMule71 TheMule71 marked this pull request as ready for review November 27, 2021 18:26
@TheMule71 TheMule71 force-pushed the 12.0-l10n_it_fatturapa_out_di branch from ecc414c to ada2ed3 Compare November 27, 2021 19:15
Comment thread l10n_it_fatturapa_out_di/tests/test_fatturapa_xml_validation.py Outdated
Comment thread l10n_it_fatturapa_out_di/tests/test_fatturapa_xml_validation.py
@TheMule71 TheMule71 force-pushed the 12.0-l10n_it_fatturapa_out_di branch from ada2ed3 to ce6f705 Compare December 1, 2021 23:07
@TheMule71 TheMule71 force-pushed the 12.0-l10n_it_fatturapa_out_di branch 2 times, most recently from 5dc6de9 to a2a3870 Compare December 10, 2021 10:27
Comment thread l10n_it_fatturapa_out_di/wizard/wizard_export_fatturapa.py Outdated
Comment thread l10n_it_fatturapa_out_di/wizard/wizard_export_fatturapa.py Outdated
Modulo ponte tra l10n_it_dichiarazione_intento e l10n_it_fatturapa_out
@TheMule71 TheMule71 force-pushed the 12.0-l10n_it_fatturapa_out_di branch from a2a3870 to b53d26c Compare December 10, 2021 15:45
@sergiocorato

Copy link
Copy Markdown
Contributor

Grazie

@sergiocorato

Copy link
Copy Markdown
Contributor

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 12.0-ocabot-merge-pr-2518-by-sergiocorato-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 429f54e into OCA:12.0 Dec 13, 2021
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at dc7367d. Thanks a lot for contributing to OCA. ❤️

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.

3 participants