[14.0][MIG] l10n_it_account_stamp_sale: Migration to 14.0

[TheMule71]

--
Confermo di aver firmato il CLA https://odoo-community.org/page/cla e di aver letto le linee guida su https://odoo-community.org/page/contributing

[andreampiovesana]

go

[TheMule71]

Mi sono fermato con questo modulo perché mi chiedo se sulla 14 serva. Abbiamo:

l10n-italy/l10n_it_account_stamp/models/account_move.py

Lines 46 to 53 in 67f8f89

	def _compute_tax_stamp(self):
	for invoice in self:
	invoice.tax_stamp = False
	if invoice.auto_compute_stamp:
	invoice.tax_stamp = invoice.is_tax_stamp_applicable()
	else:
	if invoice.manually_apply_tax_stamp:
	invoice.tax_stamp = True

direttamente nel modulo l10n_it_account_stamp.

In questo modulo facciamo:

    def action_invoice_create(self, grouped=False, final=False):
        invoice_ids = super(SaleOrder, self).action_invoice_create(
            grouped=grouped, final=final
        )
        invoice_model = self.env["account.move"]
        for invoice in invoice_model.browse(invoice_ids):
            invoice.tax_stamp = invoice.is_tax_stamp_applicable()
        return invoice_ids

Ma la _compute che viene invocata in fase di creazione delle fatture dovrebbe avere già settato invoice.tax_stamp

[primes2h]

Metto questa PR come promemoria #3105.

[primes2h]

Mi sono fermato con questo modulo perché mi chiedo se sulla 14 serva. Abbiamo:

l10n-italy/l10n_it_account_stamp/models/account_move.py

Lines 46 to 53 in 67f8f89

	def _compute_tax_stamp(self):
	for invoice in self:
	invoice.tax_stamp = False
	if invoice.auto_compute_stamp:
	invoice.tax_stamp = invoice.is_tax_stamp_applicable()
	else:
	if invoice.manually_apply_tax_stamp:
	invoice.tax_stamp = True

direttamente nel modulo l10n_it_account_stamp.

In questo modulo facciamo:

    def action_invoice_create(self, grouped=False, final=False):
        invoice_ids = super(SaleOrder, self).action_invoice_create(
            grouped=grouped, final=final
        )
        invoice_model = self.env["account.move"]
        for invoice in invoice_model.browse(invoice_ids):
            invoice.tax_stamp = invoice.is_tax_stamp_applicable()
        return invoice_ids

Ma la _compute che viene invocata in fase di creazione delle fatture dovrebbe avere già settato invoice.tax_stamp

In effetti è proprio così.
L'ho anche testato su runboat, alle fatture create da ordini di vendita viene applicata la marca da bollo.

[primes2h]

Mi sono fermato con questo modulo perché mi chiedo se sulla 14 serva. Abbiamo:

l10n-italy/l10n_it_account_stamp/models/account_move.py

Lines 46 to 53 in 67f8f89

	def _compute_tax_stamp(self):
	for invoice in self:
	invoice.tax_stamp = False
	if invoice.auto_compute_stamp:
	invoice.tax_stamp = invoice.is_tax_stamp_applicable()
	else:
	if invoice.manually_apply_tax_stamp:
	invoice.tax_stamp = True

direttamente nel modulo l10n_it_account_stamp.
In questo modulo facciamo:

    def action_invoice_create(self, grouped=False, final=False):
        invoice_ids = super(SaleOrder, self).action_invoice_create(
            grouped=grouped, final=final
        )
        invoice_model = self.env["account.move"]
        for invoice in invoice_model.browse(invoice_ids):
            invoice.tax_stamp = invoice.is_tax_stamp_applicable()
        return invoice_ids

Ma la _compute che viene invocata in fase di creazione delle fatture dovrebbe avere già settato invoice.tax_stamp

In effetti è proprio così. L'ho anche testato su runboat, alle fatture create da ordini di vendita viene applicata la marca da bollo.

Mi spiego meglio, il test l'ho fatto senza questo modulo installato.

[tafaRU]

Ok grazie @primes2h, pertanto si può chiudere la PR?

[primes2h]

Ok grazie @primes2h, pertanto si può chiudere la PR?

Beh, a questo punto direi di si.
Che ne pensi @TheMule71 ?

P.S.: ho dato un'occhiata e ho provato anche la 12.0, disinstallando il modulo.
Stessa cosa, il modulo in pratica non serve.