[FIX][l10n_it_fatturapa_in] fix import with type minimo, and add che…

[matteoopenf]

Risolve #4228

[francesco-ooops]

@matteoopenf puoi sistemare il nome del commit secondo le guidelines? grazie!

[matteoopenf]

@matteoopenf puoi sistemare il nome del commit secondo le guidelines? grazie!

Il problema è solo il nome del modulo tra parentesi esatto?
Fammi sapere così provo ad arrivarci entro fine giornata

[MaurizioPellegrinet]

LGTM

[OCA-git-bot]

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

[matteoopenf]

@francesco-ooops fatto merge?

[francesco-ooops]

@matteoopenf so che è una pipponata e non so darti indicazioni pratiche perchè non uso github, ma metterei nel nome commit:

[FIX]l10n_it_fatturapa_in: fix import with type minimo

E nella descrizione commit:

Added check if invoice_line_ids is valorize to check the price_unit, if is not valorize stop the method because is not possibile assume the line is negative

[matteoopenf]

@matteoopenf so che è una pipponata e non so darti indicazioni pratiche perchè non uso github, ma metterei nel nome commit:

[FIX]l10n_it_fatturapa_in: fix import with type minimo

E nella descrizione commit:

Added check if invoice_line_ids is valorize to check the price_unit, if is not valorize stop the method because is not possibile assume the line is negative

arrivo

[francesco-ooops]

@OCA/local-italy-maintainers mergiamo?

[SirAionTech]

Grazie della PR!
Ho provato in runboat e confermo che il problema si verifica in 14.0 con il file https://github.com/OCA/l10n-italy/blob/1c0e8323ec675550e84129bdf9292ccd5962fb6d/l10n_it_fatturapa_in/tests/data/IT05979361218_001.xml:

(da http://oca-l10n-italy-14-0-d2bffbf6a8d1.runboat.odoo-community.org/web#id=15&active_id=1&model=account.move&view_type=form&cids=&menu_id=134).
Il problema viene corretto da questa PR (importando lo stesso file):

(da http://oca-l10n-italy-14-0-pr4229-9ecd104a85f3.runboat.odoo-community.org/web#id=15&active_id=1&model=account.move&view_type=form&cids=&menu_id=134)

Puoi aggiungere o modificare un test esistente in modo che fallisca senza questa correzione? Basterebbe anche solo aggiungere un controllo sul tipo di move nel test che usa il dettaglio minimo.

[matteoopenf]

Grazie della PR!

Ho provato in runboat e confermo che il problema si verifica in 14.0 con il file https://github.com/OCA/l10n-italy/blob/1c0e8323ec675550e84129bdf9292ccd5962fb6d/l10n_it_fatturapa_in/tests/data/IT05979361218_001.xml:

(da http://oca-l10n-italy-14-0-d2bffbf6a8d1.runboat.odoo-community.org/web#id=15&active_id=1&model=account.move&view_type=form&cids=&menu_id=134).

Il problema viene corretto da questa PR (importando lo stesso file):

(da http://oca-l10n-italy-14-0-pr4229-9ecd104a85f3.runboat.odoo-community.org/web#id=15&active_id=1&model=account.move&view_type=form&cids=&menu_id=134)

Puoi aggiungere o modificare un test esistente in modo che fallisca senza questa correzione? Basterebbe anche solo aggiungere un controllo sul tipo di move nel test che usa il dettaglio minimo.

Provo a guardare come torno sul problema per fare la pr anche per 16.0

[eLBati]

LGTM

[matteoopenf]

@SirAionTech scusami sto tentando di fare il test "opposto" al 54, ma non capisco come dire di importare al minimo la fattura anziche' al massimo.
Per caso sai dirmi come fare?

[matteoopenf]

ho provato a creare il test vediamo come va

[matteoopenf]

@SirAionTech sto ottenendo questo errore tentando di forzare l'import al minimo

Per caso sai darmi un aiuto cosi da concludere la cosa?

[matteoopenf]

oppure come in altri test, prima devo creare il partner e settare il livello di import e poi fare girare il wizard?

[SirAionTech]

Il test test_16_xml_import importa con dettaglio minimo:

l10n-italy/l10n_it_fatturapa_in/tests/test_import_fatturapa_xml.py

Lines 443 to 444 in 0aa4def

	# change Livello di dettaglio Fatture elettroniche to Minimo
	partner.e_invoice_detail_level = "0"

lo imposta sul partner prima di importare la fattura, puoi fare allo stesso modo.
Oppure, forse sarebbe più semplice

aggiungere un controllo sul tipo di move nel test che usa il dettaglio minimo.

(da #4229 (review))

[matteoopenf]

Il test test_16_xml_import importa con dettaglio minimo:

l10n-italy/l10n_it_fatturapa_in/tests/test_import_fatturapa_xml.py

Lines 443 to 444 in 0aa4def

	# change Livello di dettaglio Fatture elettroniche to Minimo
	partner.e_invoice_detail_level = "0"

lo imposta sul partner prima di importare la fattura, puoi fare allo stesso modo.
Oppure, forse sarebbe più semplice

aggiungere un controllo sul tipo di move nel test che usa il dettaglio minimo.

(da #4229 (review))

ok provo ad aggiustare li, grazie mille

[matteoopenf]

@SirAionTech ho aggiunto il test e ci dovremmo essere ora, ho anche fatto il porting sulla pr 16.0

[SirAionTech]

@SirAionTech ho aggiunto il test e ci dovremmo essere ora, ho anche fatto il porting sulla pr 16.0

Grazie, potresti includere il test nello stesso commit della sua correzione?
Così è chiaro che le due parti di codice sono collegate, senza dover risalire fino a questa PR.

[matteoopenf]

@SirAionTech ho aggiunto il test e ci dovremmo essere ora, ho anche fatto il porting sulla pr 16.0

Grazie, potresti includere il test nello stesso commit della sua correzione? Così è chiaro che le due parti di codice sono collegate, senza dover risalire fino a questa PR.

ok due minuti e li unisco

[matteoopenf]

@SirAionTech fatto

[SirAionTech]

/ocabot merge patch

[OCA-git-bot]

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-4229-by-SirAionTech-bump-patch, awaiting test results.

[OCA-git-bot]

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