[MIG] l10n_it_fatturapa_out_triple_discount by SirAionTech · Pull Request #3720 · OCA/l10n-italy

SirAionTech · 2023-11-15T15:09:47Z

Migrazione da https://github.com/OCA/l10n-italy/tree/8937e1406eb42768a820539a0f0d9e5e8f305188/l10n_it_fatturapa_out_triple_discount (12.0).

francesco-ooops · 2023-11-15T17:42:48Z

@SirAionTech capisco la necessità di fare la migrazione alla versione che interessa, ma dato che stiamo lavorando proprio ora sulla migrazione per la 14 (#3681) potrebbe essere utile unire le forze per mergiare una delle due versioni e successivamente fare il backporting o fw-porting dell'altra in modo da essere allineati, che ne dici?

soprattutto perchè queste migrazioni hanno fatto uscire fuori magagne su account_triple_discount (e al momento sulla 14 hanno rotto i test di sale_triple_discount, cfr OCA/account-invoicing#1605 ), per cui non proprio una passeggiata di salute 😅

SirAionTech · 2023-11-16T08:58:41Z

@SirAionTech capisco la necessità di fare la migrazione alla versione che interessa, ma dato che stiamo lavorando proprio ora sulla migrazione per la 14 (#3681) potrebbe essere utile unire le forze per mergiare una delle due versioni e successivamente fare il backporting o fw-porting dell'altra in modo da essere allineati, che ne dici?

soprattutto perchè queste migrazioni hanno fatto uscire fuori magagne su account_triple_discount (e al momento sulla 14 hanno rotto i test di sale_triple_discount, cfr OCA/account-invoicing#1605 ), per cui non proprio una passeggiata di salute 😅

Certo, ho fatto #3681 (review) per evidenziare cosa secondo me non va bene in quella migrazione.

francesco-ooops · 2023-11-16T09:20:08Z

@SirAionTech molto apprezzato <3

Se la tua PR è in stato più avanzato e sono presenti i test possiamo anche concentrarci su questa e fare il backporting, basta che il tutto sia celere ecco 😅

eLBati · 2023-11-27T11:32:16Z

+        self.env.company.partner_id.country_id = self.env.ref("base.it").id
+        self.env.company.fatturapa_fiscal_position_id = self.env.ref(
+            "l10n_it_fatturapa.fatturapa_RF01"
+        ).id


@SirAionTech queste modifiche a cosa servono?

Servono ad avere un'azienda che produca lo stesso file presente nei test.
Ad esempio viene impostato il regime fiscale a RF01 perché deve essere lo stesso di https://github.com/OCA/l10n-italy/pull/3720/files#diff-b4442d0a23dcdee7b95d10e0baed8c8266eaac33d66499c6cd97f11266027b04R29.

Per modificare il meno possibile il file di test esistente?
L'importante penso sia avere un XML valido per la parte rilevante per le funzionalità di questo modulo, poi se il resto del file XML contiene i dati che contengono tutti gli altri XML di test , non vedo problemi a modificarlo

O modifico il setup dei test (come ho fatto), o modifico il file XML: ho preferito modificare il setup dei test per ottenere la stessa fattura elettronica perché penso che il test debba verificare che il modulo possa produrre esattamente quella fattura elettronica.

se il resto del file XML contiene i dati che contengono tutti gli altri XML di test , non vedo problemi a modificarlo

Tutti gli altri XML di test contengono proprio questi dati: le modifiche fatte qui sono le stesse che furono necessarie per la migrazione a 14.0 di l10n_it_fatturapa_out:

l10n-italy/l10n_it_fatturapa_out/tests/test_fatturapa_xml_validation.py

Lines 45 to 62 in 8ae41d3

# XXX - a company named "YourCompany" alread exists

# we move it out of the way but we should do better here

self.env.company.sudo().search([("name", "=", "YourCompany")]).write(

{"name": "YourCompany_"}

)

self.env.company.name = "YourCompany"

self.env.company.vat = "IT06363391001"

self.env.company.fatturapa_art73 = True

self.env.company.partner_id.street = "Via Milano, 1"

self.env.company.partner_id.city = "Roma"

self.env.company.partner_id.state_id = self.env.ref("base.state_us_2").id

self.env.company.partner_id.zip = "00100"

self.env.company.partner_id.phone = "06543534343"

self.env.company.email = "info@yourcompany.example.com"

self.env.company.partner_id.country_id = self.env.ref("base.it").id

self.env.company.fatturapa_fiscal_position_id = self.env.ref(

"l10n_it_fatturapa.fatturapa_RF01"

).id

in modo che le fatture elettroniche non fossero da cambiare nella migrazione.
Esiste già la issue #2507 per evitare di ripetere questo setup ma deve ancora essere risolta.

Quindi pensi sia meglio che modifichi il file della fattura elettronica generato dai test?
Così facendo conterrà dati diversi da tutti gli altri XML di test.

matteoopenf · 2023-11-27T16:18:01Z

Concordo Matteo ***@***.*** Il giorno 27 nov 2023, alle ore 17:12, Lorenzo Battistini ***@***.***> ha scritto: @eLBati commented on this pull request. In l10n_it_fatturapa_out_triple_discount/tests/fatturapa_common.py:

+ {"name": "YourCompany_"}

+ ) + + self.env.company.name = "YourCompany" + self.env.company.vat = "IT06363391001" + self.env.company.fatturapa_art73 = True + self.env.company.partner_id.street = "Via Milano, 1" + self.env.company.partner_id.city = "Roma" + self.env.company.partner_id.state_id = self.env.ref("base.state_us_2").id + self.env.company.partner_id.zip = "00100" + self.env.company.partner_id.phone = "06543534343" + self.env.company.email = ***@***.***" + self.env.company.partner_id.country_id = self.env.ref("base.it").id + self.env.company.fatturapa_fiscal_position_id = self.env.ref( + "l10n_it_fatturapa.fatturapa_RF01" + ).id Per modificare il meno possibile il file di test esistente? L'importante penso sia avere un XML valido per la parte rilevante per le funzionalità di questo modulo, poi se il resto del file XML contiene i dati che contengono tutti gli altri XML di test , non vedo problemi a modificarlo —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: ***@***.***> Openforce Srls Unipersonale | www.openforce.it | 02 21119472 | Via Tiziano,32 20145 Milano

SirAionTech · 2023-11-27T17:04:09Z

#3720 (comment)

…

On Mon, 27 Nov 2023 at 17:18, Matteo Mircoli ***@***.***> wrote: Concordo Matteo ***@***.*** Il giorno 27 nov 2023, alle ore 17:12, Lorenzo Battistini ***@***.***> ha scritto: @eLBati commented on this pull request. In l10n_it_fatturapa_out_triple_discount/tests/fatturapa_common.py: > + {"name": "YourCompany_"} + ) + + self.env.company.name = "YourCompany" + self.env.company.vat = "IT06363391001" + self.env.company.fatturapa_art73 = True + self.env.company.partner_id.street = "Via Milano, 1" + self.env.company.partner_id.city = "Roma" + self.env.company.partner_id.state_id = self.env.ref("base.state_us_2").id + self.env.company.partner_id.zip = "00100" + self.env.company.partner_id.phone = "06543534343" + self.env.company.email = ***@***.***" + self.env.company.partner_id.country_id = self.env.ref("base.it").id + self.env.company.fatturapa_fiscal_position_id = self.env.ref( + "l10n_it_fatturapa.fatturapa_RF01" + ).id Per modificare il meno possibile il file di test esistente? L'importante penso sia avere un XML valido per la parte rilevante per le funzionalità di questo modulo, poi se il resto del file XML contiene i dati che contengono tutti gli altri XML di test , non vedo problemi a modificarlo —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: ***@***.***> Openforce Srls Unipersonale | www.openforce.it | 02 21119472 | Via Tiziano,32 20145 Milano — Reply to this email directly, view it on GitHub <#3720 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/BCJAXQVS4U2XBRQPENZWYU3YGS4MLAVCNFSM6AAAAAA7MUVTZSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRYGE3DANZRGM> . You are receiving this because you were mentioned.Message ID: ***@***.***>

eLBati · 2023-11-28T13:32:18Z

+  -->
+<odoo>
+    <template
+        id="account_invoice_it_dati_bollo"


Qua meglio un id tipo account_invoice_it_triple_discount

Ah questo mi era scappato grazie, l'ho chiamato come il template da cui eredita come suggerito in https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#253inherited-xml.

SirAionTech · 2023-12-07T15:54:12Z

Aggiornamento: la PR analoga per 14.0 #3743 è stata mergiata

francesco-ooops · 2023-12-07T16:10:45Z

@OCA/local-italy-maintainers aggiungere a #2967

OvertakeMattia

Testato il modulo e controllato emissione di una decina di documenti con sconto 1, 2, 3 anche in varie combinazioni di presenza /assenza e funziona correttamente

SirAionTech · 2024-06-14T13:51:20Z

/ocabot migration l10n_it_fatturapa_out_triple_discount

SirAionTech · 2024-11-12T09:30:02Z

/ocabot rebase

…API and to new file names

it fixes: - removes xs:dateTime if bogus and not mandatory i.e. 0001-01-01T00:00:00.000+02:00 where python raises OverflowError - removes timezone from xs:date to make pyxb able to compare with 1-1-1970, it also removes the need of patching pyxb - removes space only strings if not mandatory, else replace with a dash breaking change: modules needs to import binding.fatturapa instead of bindings.fatturapa_v_1_2, this would be asl useful for new specs

…e now rounded to the nearest even result instead of away from zero". See https://docs.python.org/3/whatsnew/3.0.html#builtins Examples: >>> '%.2f' % 75.845 '75.84' >>> '%.2f' % 75.855 '75.86' >>> round(2.5) 2 >>> round(3.5) 4

…scount: modifica del type di Importo in ScontoMaggiorazioneType (OCA#1875) * [IMP] l10n_it_fatturapa_out: change precision_rounding of ScontoMaggiorazioneType according to e-invoicing specs 1.6 * [IMP] l10n_it_fatturapa_out_triple_discount: change precision_rounding of ScontoMaggiorazioneType according to e-invoicing specs 1.6

… discard

OCA-git-bot · 2024-11-12T09:30:24Z

Congratulations, PR rebased to 16.0.

github-actions · 2025-03-16T12:35:19Z

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

SirAionTech force-pushed the 16.0-mig-l10n_it_fatturapa_out_triple_discount branch from 49f7e3d to a0e4c8d Compare November 15, 2023 15:15

This comment was marked as resolved.

Sign in to view

SirAionTech marked this pull request as ready for review November 15, 2023 15:44

SirAionTech force-pushed the 16.0-mig-l10n_it_fatturapa_out_triple_discount branch from a0e4c8d to c79be06 Compare November 15, 2023 17:03

SirAionTech marked this pull request as draft November 15, 2023 17:05

SirAionTech marked this pull request as ready for review November 15, 2023 17:14

SirAionTech mentioned this pull request Nov 16, 2023

[14.0][MIG] l10n_it_fatturapa_out_triple_discount #3681

Closed

SirAionTech mentioned this pull request Nov 16, 2023

[14.0] FIX account_invoice_triple_discount - _recompute_tax_lines involving only relevant lines: invoice_line_ids OCA/account-invoicing#1592

Merged

SirAionTech force-pushed the 16.0-mig-l10n_it_fatturapa_out_triple_discount branch from c79be06 to 476cd6a Compare November 24, 2023 12:04

eLBati reviewed Nov 27, 2023

View reviewed changes

SirAionTech requested a review from eLBati November 27, 2023 15:39

eLBati requested changes Nov 28, 2023

View reviewed changes

eLBati mentioned this pull request Nov 28, 2023

[14.0][MIG] l10n_it_fatturapa_out_triple_discount #3743

Merged

SirAionTech force-pushed the 16.0-mig-l10n_it_fatturapa_out_triple_discount branch from 476cd6a to 143696e Compare November 28, 2023 14:53

SirAionTech requested a review from eLBati November 28, 2023 14:54

SirAionTech force-pushed the 16.0-mig-l10n_it_fatturapa_out_triple_discount branch from 143696e to 89aafa0 Compare December 1, 2023 09:26

francesco-ooops added migration 16.0 labels Dec 19, 2023

francesco-ooops mentioned this pull request Dec 29, 2023

Migration to version 16.0 #2967

Open

81 tasks

DinamicheAziendali mentioned this pull request Feb 29, 2024

PR aggiunte v16 DinamicheAziendali/l10n-italy#1

Open

14 tasks

SirAionTech force-pushed the 16.0-mig-l10n_it_fatturapa_out_triple_discount branch from 89aafa0 to a622fec Compare March 21, 2024 13:19

OvertakeMattia approved these changes Jun 14, 2024

View reviewed changes

OCA-git-bot added this to the 16.0 milestone Jun 14, 2024

SirAionTech force-pushed the 16.0-mig-l10n_it_fatturapa_out_triple_discount branch from a622fec to 444e4e9 Compare August 6, 2024 15:12

SimoRubi and others added 15 commits November 12, 2024 09:30

[ADD] l10n_it_fatturapa_out_triple_discount

f46b957

FIX l10n_it_fatturapa_out_triple_discount adapt to new set_sequences …

1023bfa

…API and to new file names

[MIG] 10n_it_fatturapa_out_triple_discount: Migration to 12.0

2c83511

l10n_it_fatturapa_out_triple_discount: disable tests because unstable

be05103

merge PR 1264 1717 1728 1737 1739

c19cde8

FIX license LGPL-3 conflicts

64c8c1b

[FIX][l10n_it_fatturapa_out_triple_discount] change rounding to avoid…

ab8b85c

… discard

Manually fix manifest website key

cf1a842

Added translation using Weblate (Italian)

81e2bbd

[IMP] l10n_it_fatturapa_out_triple_discount: pre-commit execution

adb38d5

[MIG] l10n_it_fatturapa_out_triple_discount: Migration to 16.0

8062081

Updated from template

a6ea8d2

OCA-git-bot force-pushed the 16.0-mig-l10n_it_fatturapa_out_triple_discount branch from 444e4e9 to a6ea8d2 Compare November 12, 2024 09:30

github-actions Bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Mar 16, 2025

github-actions Bot closed this Apr 20, 2025

SirPyTech mentioned this pull request Aug 27, 2025

[MIG] l10n_it_fatturapa_out_triple_discount: Migration to v16 #4879

Merged

	# XXX - a company named "YourCompany" alread exists
	# we move it out of the way but we should do better here
	self.env.company.sudo().search([("name", "=", "YourCompany")]).write(
	{"name": "YourCompany_"}
	)
	self.env.company.name = "YourCompany"
	self.env.company.vat = "IT06363391001"
	self.env.company.fatturapa_art73 = True
	self.env.company.partner_id.street = "Via Milano, 1"
	self.env.company.partner_id.city = "Roma"
	self.env.company.partner_id.state_id = self.env.ref("base.state_us_2").id
	self.env.company.partner_id.zip = "00100"
	self.env.company.partner_id.phone = "06543534343"
	self.env.company.email = "info@yourcompany.example.com"
	self.env.company.partner_id.country_id = self.env.ref("base.it").id
	self.env.company.fatturapa_fiscal_position_id = self.env.ref(
	"l10n_it_fatturapa.fatturapa_RF01"
	).id

Uh oh!

Conversation

SirAionTech commented Nov 15, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

This comment was marked as resolved.

francesco-ooops commented Nov 15, 2023

Uh oh!

SirAionTech commented Nov 16, 2023

Uh oh!

francesco-ooops commented Nov 16, 2023

Uh oh!

eLBati Nov 27, 2023

Choose a reason for hiding this comment

Uh oh!

SirAionTech Nov 27, 2023

Choose a reason for hiding this comment

Uh oh!

eLBati Nov 27, 2023

Choose a reason for hiding this comment

Uh oh!

SirAionTech Nov 27, 2023

Choose a reason for hiding this comment

Uh oh!

matteoopenf commented Nov 27, 2023 via email

Uh oh!

SirAionTech commented Nov 27, 2023 via email

Uh oh!

eLBati Nov 28, 2023

Choose a reason for hiding this comment

Uh oh!

SirAionTech Nov 28, 2023

Choose a reason for hiding this comment

Uh oh!

SirAionTech commented Dec 7, 2023

Uh oh!

francesco-ooops commented Dec 7, 2023

Uh oh!

OvertakeMattia left a comment

Choose a reason for hiding this comment

Uh oh!

SirAionTech commented Jun 14, 2024

Uh oh!

SirAionTech commented Nov 12, 2024

Uh oh!

OCA-git-bot commented Nov 12, 2024

Uh oh!

github-actions Bot commented Mar 16, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

11 participants

SirAionTech commented Nov 15, 2023 •

edited

Loading