Migrazione con OpenUpgrade di moduli presenti in apriori non funzionante

[sergiocorato]

Questo codice viene richiamato sempre da OpenUpgrade per eseguire la migrazione:

l10n-italy/l10n_it_appointment_code/migrations/14.0.1.0.0/pre-migrate.py

Line 11 in 8a18ee5

# Used by OpenUpgrade when module is in `apriori`

per cui nel caso in cui il modulo sia già stato rinominato tramite l'inserimento nel file apriori.py in

https://github.com/OCA/OpenUpgrade/blob/11000ece9ea3254deea8f20aaf526b94ed95d439/openupgrade_scripts/apriori.py#L66

non viene più eseguita la funzione

l10n-italy/l10n_it_appointment_code/hooks.py

Line 9 in 2d6bbe2

if not openupgrade.is_module_installed(cr, "l10n_it_codici_carica"):

in quanto non risulta più installato.

Per utilizzare apriori.py serve quindi bypassare il controllo sullo stato di installazione del modulo e poi evitare eventualmente di eseguire la rinomina del modulo, già fatta.

Affected versions:

• 14.0

Moduli da correggere:

• l10n_it_payment_reason [FIX] migration script for l10n_it_payment_reason with OpenUpgrade #3604
• l10n_it_appointment_code [FIX][14.0] set migration script of l10n_it_appointment_code to same logic of other #3598
• l10n_it_withholding_tax_reason [14.0][FIX] l10n_it_withholding_tax_reason migration script compatibile con apriori #3495

[SirAionTech]

Puoi seguire la struttura di https://github.com/OCA/l10n-italy/tree/2d6bbe27311c876c290d83adf15c8c46bcecd21c/l10n_it_financial_statements_report: il metodo che esegue la migrazione è migrate_old_module.
Nel hook di installazione c'è rinomina + migrazione, se il vecchio modulo è installato:

l10n-italy/l10n_it_financial_statements_report/hooks.py

Lines 123 to 132 in 2d6bbe2

	def pre_absorb_old_module(cr):
	if openupgrade.is_module_installed(cr, OLD_MODULE_NAME):
	openupgrade.update_module_names(
	cr,
	[
	(OLD_MODULE_NAME, NEW_MODULE_NAME),
	],
	merge_modules=True,
	)
	migrate_old_module(cr)

mentre nella migrazione c'è la chiamata diretta al metodo perché il modulo è stato rinominato da https://github.com/OCA/OpenUpgrade/blob/11000ece9ea3254deea8f20aaf526b94ed95d439/openupgrade_scripts/apriori.py#L64C19-L64C19:

l10n-italy/l10n_it_financial_statements_report/migrations/14.0.1.0.0/pre-migrate.py

Lines 7 to 9 in 2d6bbe2

	def migrate(cr, installed_version):
	# Used by OpenUpgrade when module is in `apriori`
	hooks.migrate_old_module(cr)

EDIT: Maggiori dettagli in https://github.com/OCA/l10n-italy/wiki/Migrazione-con-rinomina-modulo,-compatibile-con-OpenUpgrade-e-Odoo-SA.

[sergiocorato]

Puoi seguire la struttura di https://github.com/OCA/l10n-italy/tree/2d6bbe27311c876c290d83adf15c8c46bcecd21c/l10n_it_financial_statements_report: il metodo che esegue la migrazione è migrate_old_module. Nel hook di installazione c'è rinomina + migrazione, se il vecchio modulo è installato:

l10n-italy/l10n_it_financial_statements_report/hooks.py

Lines 123 to 132 in 2d6bbe2

	def pre_absorb_old_module(cr):
	if openupgrade.is_module_installed(cr, OLD_MODULE_NAME):
	openupgrade.update_module_names(
	cr,
	[
	(OLD_MODULE_NAME, NEW_MODULE_NAME),
	],
	merge_modules=True,
	)
	migrate_old_module(cr)

mentre nella migrazione c'è la chiamata diretta al metodo perché il modulo è stato rinominato da https://github.com/OCA/OpenUpgrade/blob/11000ece9ea3254deea8f20aaf526b94ed95d439/openupgrade_scripts/apriori.py#L64C19-L64C19:

l10n-italy/l10n_it_financial_statements_report/migrations/14.0.1.0.0/pre-migrate.py

Lines 7 to 9 in 2d6bbe2

	def migrate(cr, installed_version):
	# Used by OpenUpgrade when module is in `apriori`
	hooks.migrate_old_module(cr)

Ottimo, faccio così allora.

[sergiocorato]

Ottimo, faccio così allora.

Grazie.

[SirAionTech]

Ho riportato la struttura descritta in #3574 (comment) su https://github.com/OCA/l10n-italy/wiki/Migrazione-con-rinomina-modulo,-compatibile-con-OpenUpgrade-e-Odoo-SA.

[francesco-ooops]

@sergiocorato @SirAionTech c'è altro da fare in questa issue o chiudiamo?

[SirAionTech]

@sergiocorato @SirAionTech c'è altro da fare in questa issue o chiudiamo?

Non lo so, io ho solo suggerito un modo per risolvere.
@sergiocorato ho trovato #3598 che forse è per risolvere questa issue, ce ne sono altre?

[sergiocorato]

@sergiocorato @SirAionTech c'è altro da fare in questa issue o chiudiamo?

Non lo so, io ho solo suggerito un modo per risolvere. @sergiocorato ho trovato #3598 che forse è per risolvere questa issue, ce ne sono altre?

Si può chiudere, ho aggiornato la descrizione.