rename module from l10n_it_riba to l10n_it_riba_oca #4520

Borruso · 2025-01-03T12:14:59Z

MaurizioPellegrinet

... se tento di installare riba_oca mi segnala errore:

Traceback (most recent call last):
File "/opt/odoo/odoo/http.py", line 1653, in _serve_db
return service_model.retrying(self._serve_ir_http, self.env)
File "/opt/odoo/odoo/service/model.py", line 133, in retrying
result = func()
File "/opt/odoo/odoo/http.py", line 1680, in _serve_ir_http
response = self.dispatcher.dispatch(rule.endpoint, args)
File "/opt/odoo/odoo/http.py", line 1884, in dispatch
result = self.request.registry['ir.http']._dispatch(endpoint)
File "/opt/odoo/odoo/addons/base/models/ir_http.py", line 154, in _dispatch
result = endpoint(**request.params)
File "/opt/odoo/odoo/http.py", line 734, in route_wrapper
result = endpoint(self, *args, **params_ok)
File "/opt/odoo/addons/web/controllers/dataset.py", line 46, in call_button
action = self._call_kw(model, method, args, kwargs)
File "/opt/odoo/addons/web/controllers/dataset.py", line 33, in _call_kw
return call_kw(request.env[model], method, args, kwargs)
File "/opt/odoo/odoo/api.py", line 468, in call_kw
result = _call_kw_multi(method, model, args, kwargs)
File "/opt/odoo/odoo/api.py", line 453, in _call_kw_multi
result = method(recs, *args, **kwargs)
File "", line 2, in button_immediate_install
File "/opt/odoo/odoo/addons/base/models/ir_module.py", line 74, in check_and_log
return method(self, *args, **kwargs)
File "/opt/odoo/odoo/addons/base/models/ir_module.py", line 478, in button_immediate_install
return self._button_immediate_function(self.env.registry[self._name].button_install)
File "/opt/odoo/odoo/addons/base/models/ir_module.py", line 602, in _button_immediate_function
registry = modules.registry.Registry.new(self._cr.dbname, update_module=True)
File "", line 2, in new
File "/opt/odoo/odoo/tools/func.py", line 87, in locked
return func(inst, *args, **kwargs)
File "/opt/odoo/odoo/modules/registry.py", line 87, in new
odoo.modules.load_modules(registry, force_demo, status, update_module)
File "/opt/odoo/odoo/modules/loading.py", line 489, in load_modules
processed_modules += load_marked_modules(cr, graph,
File "/opt/odoo/odoo/modules/loading.py", line 373, in load_marked_modules
loaded, processed = load_module_graph(
File "/opt/odoo/odoo/modules/loading.py", line 196, in load_module_graph
getattr(py_module, pre_init)(cr)
File "/mnt/data/odoo-addons-dir/l10n_it_riba_oca/hooks.py", line 73, in pre_absorb_old_module
cr.execute(
File "/opt/odoo/odoo/sql_db.py", line 321, in execute
res = self._obj.execute(query, params)
IndexError: tuple index out of range

The above server error caused the following client error:
RPC_ERROR: Odoo Server Error
RPC_ERROR
at makeErrorFromResponse (http://oca-l10n-italy-16-0-pr4520-10e76e965d5e.runboat.odoo-community.org/web/assets/413-c765628/web.assets_backend.min.js:1001:163)
at XMLHttpRequest. (http://oca-l10n-italy-16-0-pr4520-10e76e965d5e.runboat.odoo-community.org/web/assets/413-c765628/web.assets_backend.min.js:1009:13)

Borruso · 2025-01-03T13:32:27Z

... se tento di installare riba_oca mi segnala errore:

scusami stavo testando, ora puoi provare

MaurizioPellegrinet

Test funzionale: OK
Mancano traduzioni

francesco-ooops · 2025-01-08T11:22:20Z

Mancano traduzioni

penso vada rinominato il file .pot @Borruso

primes2h · 2025-01-08T14:13:44Z

Mancano traduzioni

penso vada rinominato il file .pot @Borruso

Non è così semplice, andrebbero cambiati anche tutti i riferimenti interni al modulo per ciascuna stringa.

primes2h

Come ho già indicato qui non credo sia opportuno procedere alla modifica del nome di un modulo in una versione stabile già rilasciata. (vedi anche OCA/manufacture#559 (comment)).

altissimo rischio di spaccare installazioni di produzione esistenti con un semplice update del DB.
La rinomina corrisponde a:

aggiungere un nuovo modulo l10n_it_riba_oca
eliminare il modulo esistente l10n_riba

Il modulo nuovo non può mantenere la stessa versione del modulo vecchio (dato che sono due moduli diversi), quindi dovrebbe ripartire da 16.0.1.0.0.
Eliminare un modulo da un repository non mi sembra per nulla una buona idea, data la complessità dei meccanismi automatici che vengono avviati ad ogni modifica. Per esempio cosa succede con i sistemi di CI di OCA? Cosa succede con Weblate?

L'alternativa sarebbe quella di aggiungere un nuovo modulo l10n_it_riba_oca lasciando anche quello esistente l10_it_riba.
Questa soluzione in ogni caso non risolverebbe il problema di chi usa la versione Enterprise, perché andrebbe comunque modificato l'ordine dei path delle cartelle (alternativa descritta nella issue).
A questo punto tanto vale lasciare la 16.0 così com'è e cambiare solo l'ordine dei path.

github-actions · 2025-05-11T12:38:18Z

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.

monen17

In giornata ti propongo una PR per aggiungere lo script di migrazione.

Intanto ho notato queste cose, puoi correggere?
Il concetto di base è che servono per chi migrerà dalla 14.0 a 16.0 dopo che questa PR viene mergiata.

P.S.: la PR è Borruso#35

monen17 · 2025-07-25T10:57:42Z

l10n_it_riba_oca/hooks.py

+OLD_MODULE_NAME = "l10n_it_riba"
+NEW_MODULE_NAME = "l10n_it_riba_oca"


Suggested change

OLD_MODULE_NAME = "l10n_it_riba"

NEW_MODULE_NAME = "l10n_it_riba_oca"

OLD_MODULE_NAME = "l10n_it_ricevute_bancarie"

NEW_MODULE_NAME = "l10n_it_riba_oca"

Questo hook serve per chi migra da 14.0, dove il modulo si chiama l10n_it_ricevute_bancarie, alla 16.0 dove il modulo, dopo questa PR, si chiamerà l10n_it_riba_oca.

Quindi anche qui va solo fatta la sostituzione l10n_it_riba -> l10n_it_riba_oca.

Per la migrazione dati ti sto proponendo io una PR come abbiamo detto poco fa in chiamata.

monen17 · 2025-07-25T11:03:49Z

l10n_it_riba/hooks.py

Il titolo del commit non segue https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#71commit-message, puoi correggere?
Ad esempio potrebbe essere

[MOV] l10n_it_riba: Rename to l10n_it_riba_oca

E ricorda di usare git mv per spostare i file, altrimenti non li vedremo nella commit history del nuovo modulo:

[MOV] for moving files: use git move and do not change content of moved file otherwise Git may loose track and history of the file; also used when moving code from one file to another;

(da https://www.odoo.com/documentation/16.0/contributing/development/git_guidelines.html#tag-and-module-name)

monen17 · 2025-07-25T11:10:14Z

l10n_it_riba_oca/migrations/16.0.1.8.0/pre-migrate.py

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


Questo è nuovo, quando dovrebbe essere eseguito?

monen17 · 2025-07-25T11:13:52Z

l10n_it_riba_oca/migrations/16.0.1.0.0/post-migrate.py

Qui dentro c'è ancora un riferimento a l10n_it_riba, puoi rinominarlo in l10n_it_riba_oca?

monen17 · 2025-07-25T11:14:15Z

l10n_it_riba_oca/migrations/16.0.1.0.0/data/noupdate.xml

Qui dentro ci sono ancora riferimenti a l10n_it_riba, puoi rinominarli in l10n_it_riba_oca?

monen17 · 2025-07-25T11:15:26Z

l10n_it_riba_oca/tests/test_riba.py

@@ -278,7 +278,7 @@ def test_riba_incasso_all_paid(self):
        self.invoice.action_post()
        self.assertEqual(self.invoice.state, "posted")

-        to_issue_action = self.env.ref("l10n_it_riba.action_riba_to_issue")
+        to_issue_action = self.env.ref("l10n_it_riba_oca.action_riba_to_issue")
        to_issue_model = self.env[to_issue_action.res_model]
        to_issue_domain = safe_eval.safe_eval(to_issue_action.domain)
        to_issue_records = (


Qui sotto (riga 334) c'è ancora un riferimento a l10n_it_riba, puoi rinominarlo in l10n_it_riba_oca?
È il motivo per cui stanno fallendo i test.

monen17 · 2025-07-25T13:13:36Z

Come ho già indicato qui non credo sia opportuno procedere alla modifica del nome di un modulo in una versione stabile già rilasciata. (vedi anche OCA/manufacture#559 (comment)).
[...]

(da #4520 (review))
@primes2h grazie della revisione, purtroppo recentemente la CI ha iniziato a fallire a causa del modulo enterprise (vedi #4854 e #4519 (comment)) e sembra che l'unica possibilità che abbiamo è rinominare questo modulo.

Ne abbiamo parlato stamattina in chiamata e le alternative che sono state proposte (moduli fittizi, disattivare il check-license, ...) non reggono il confronto con una rinomina, che permette di lasciare il codice senza soluzioni temporanee.

Sappiamo che eventuali moduli custom dipendenti da l10n_it_riba dovranno essere ritoccati, ma come scrivevo non abbiamo trovato alternative valide.

Se hai alternative da proporre siamo tutt'orecchi, ma per il momento siamo orientati verso la rinomina.

primes2h · 2025-07-25T14:18:55Z

Come ho già indicato qui non credo sia opportuno procedere alla modifica del nome di un modulo in una versione stabile già rilasciata. (vedi anche OCA/manufacture#559 (comment)).
[...]

(da #4520 (review)) @primes2h grazie della revisione, purtroppo recentemente la CI ha iniziato a fallire a causa del modulo enterprise (vedi #4854 e #4519 (comment)) e sembra che l'unica possibilità che abbiamo è rinominare questo modulo.

Ne abbiamo parlato stamattina in chiamata e le alternative che sono state proposte (moduli fittizi, disattivare il check-license, ...) non reggono il confronto con una rinomina, che permette di lasciare il codice senza soluzioni temporanee.

Sappiamo che eventuali moduli custom dipendenti da l10n_it_riba dovranno essere ritoccati, ma come scrivevo non abbiamo trovato alternative valide.

Se hai alternative da proporre siamo tutt'orecchi, ma per il momento siamo orientati verso la rinomina.

Sinceramente al momento non mi vengono in mente altre alternative valide.
Data la situazione, mi pare che sia l'unica attuabile senza ricorrere ad accrocchi vari. Resta il fatto che è comunque la più rischiosa e andrà testata per bene prima del rilascio. Credo che sarebbe anche meglio aggiungere un avviso bello grande nel README che spieghi la situazione e i relativi rischi, indicando cosa fare se in caso di moduli custom che dipendono da l10n_it_riba.

monen17 · 2025-07-25T16:20:56Z

Prendo in carico in #4856

Borruso force-pushed the 16.0-l10n_it_riba-rename branch 2 times, most recently from 10e76e9 to 56e941e Compare January 3, 2025 13:16

MaurizioPellegrinet reviewed Jan 3, 2025

View reviewed changes

Borruso force-pushed the 16.0-l10n_it_riba-rename branch from 56e941e to 8e05bfd Compare January 3, 2025 13:24

Borruso requested a review from MaurizioPellegrinet January 3, 2025 13:35

SirAionTech mentioned this pull request Jan 3, 2025

rinomina modulo ‎l10n_it_riba #4519

Open

2 tasks

SirAionTech linked an issue Jan 3, 2025 that may be closed by this pull request

rinomina modulo ‎l10n_it_riba #4519

Open

2 tasks

MaurizioPellegrinet approved these changes Jan 3, 2025

View reviewed changes

primes2h requested changes Jan 8, 2025

View reviewed changes

Borruso force-pushed the 16.0-l10n_it_riba-rename branch from 8e05bfd to 84d9ed8 Compare January 10, 2025 09:15

github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label May 11, 2025

github-actions bot closed this Jul 20, 2025

monen17 reopened this Jul 25, 2025

rename module from l10n_it_riba to l10n_it_riba_oca

65d43b3

Borruso force-pushed the 16.0-l10n_it_riba-rename branch from 84d9ed8 to 65d43b3 Compare July 25, 2025 10:36

monen17 requested changes Jul 25, 2025

View reviewed changes

This was referenced Jul 25, 2025

[MIG] l10n_it_riba: Rename to l10n_it_riba_oca OCA/OpenUpgrade#5199

Draft

Script per migrazione da l10n_it_riba a l10n_it_riba_oca Borruso/l10n-italy#35

Closed

monen17 requested a review from primes2h July 25, 2025 13:13

monen17 mentioned this pull request Jul 25, 2025

Rinomina modulo ‎l10n_it_riba -> l10n_it_riba_oca #4856

Open

monen17 closed this Jul 25, 2025

		OLD_MODULE_NAME = "l10n_it_riba"
		NEW_MODULE_NAME = "l10n_it_riba_oca"

Uh oh!

rename module from l10n_it_riba to l10n_it_riba_oca #4520

rename module from l10n_it_riba to l10n_it_riba_oca #4520

Uh oh!

Conversation

Borruso commented Jan 3, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

MaurizioPellegrinet left a comment

Choose a reason for hiding this comment

Uh oh!

Borruso commented Jan 3, 2025

Uh oh!

MaurizioPellegrinet left a comment

Choose a reason for hiding this comment

Uh oh!

francesco-ooops commented Jan 8, 2025

Uh oh!

primes2h commented Jan 8, 2025

Uh oh!

primes2h left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented May 11, 2025

Uh oh!

monen17 left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

monen17 Jul 25, 2025

Choose a reason for hiding this comment

Uh oh!

monen17 Jul 25, 2025

Choose a reason for hiding this comment

Uh oh!

monen17 Jul 25, 2025

Choose a reason for hiding this comment

Uh oh!

monen17 Jul 25, 2025

Choose a reason for hiding this comment

Uh oh!

monen17 Jul 25, 2025

Choose a reason for hiding this comment

Uh oh!

monen17 Jul 25, 2025

Choose a reason for hiding this comment

Uh oh!

monen17 commented Jul 25, 2025

Uh oh!

primes2h commented Jul 25, 2025

Uh oh!

monen17 commented Jul 25, 2025

Uh oh!

Uh oh!

Borruso commented Jan 3, 2025 •

edited

Loading

primes2h left a comment •

edited

Loading

monen17 left a comment •

edited

Loading