search

OCA / l10n-italy

Odoo Italian localization
https://www.odoo-italia.org
GNU Affero General Public License v3.0
153 stars 305 forks source link

[14.0][FIX] migration script l10n_it_delivery_note_base #4002

Open sergiocorato opened 8 months ago

OCA-git-bot commented 8 months ago

Hi @MarcoCalcagni, some modules you are maintaining are being modified, check this out!

sergiocorato commented 8 months ago

@odooNextev Ho recuperato il codice che ti avevo bocciato (sigh) dalla precedente PR, poi ho forzato anche in fase di aggiornamento la verifica/creazione delle sequence/tipi ddt dal post-migrate.py ~, impostando la ricerca sul nome non tradotto per evitare delle creazioni doppie.~

odooNextev commented 8 months ago

@odooNextev Ho recuperato il codice che ti avevo bocciato (sigh) dalla precedente PR, poi ho forzato anche in fase di aggiornamento la verifica/creazione delle sequence/tipi ddt dal post-migrate.py, impostando la ricerca sul nome non tradotto per evitare delle creazioni doppie.

Ok, la settimana prossima devo testare una migrazione e provo la fix

sergiocorato commented 8 months ago

Ho rimosso la parte in cui cerco di evitare duplicazioni, in quanto in caso di un ulteriore tentativo di migrazione si blocca sembra.

Cercando solo sul nome tradotto possono esserci dei duplicati da rimuovere a mano, nel caso l'utente abbia cambiato la traduzione o ci sia una traduzione non aggiornata.

stenext commented 7 months ago

@sergiocorato ho appena testato il tuo branch durante l'aggiornamento dei moduli di l10n-italy post-migrazione di Odoo SA. Il DB inizialmente era alla 10.0, poi migrandolo alla 12.0 sono passato da l10n_it_ddt a l10n_it_delivery_note ed aggiornando i moduli alla 14.0 ottengo questo errore:

Traceback (most recent call last):
  ........
  File "/OCA/l10n-italy/l10n_it_delivery_note_base/migrations/14.0.1.0.3/post-migrate.py", line 44, in migrate
    post_init_hook(env.cr, env)
  File "/OCA/l10n-italy/l10n_it_delivery_note_base/hooks.py", line 13, in post_init_hook
    env["stock.delivery.note.type"].create_dn_types(company)
  File "/OCA/l10n-italy/l10n_it_delivery_note_base/models/stock_delivery_note_type.py", line 204, in create_dn_types
    self.with_context(lang=lang)._prepare_dn_types_vals(company_id)
  File "/OCA/l10n-italy/l10n_it_delivery_note_base/models/stock_delivery_note_type.py", line 174, in _prepare_dn_types_vals
    self._set_or_create_dn_types(
  File "/OCA/l10n-italy/l10n_it_delivery_note_base/models/stock_delivery_note_type.py", line 153, in _set_or_create_dn_types
    self.env["stock.delivery.note.type"].create(
......
psycopg2.errors.UniqueViolation: duplicate key value violates unique constraint "stock_delivery_note_type_name_uniq"
DETAIL:  Key (name, company_id)=(Outgoing, 1) already exists.

Senza le modifiche di questa PR, la migrazione non si blocca.

github-actions[bot] commented 2 months ago

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.

francesco-ooops commented 2 months ago

@sergiocorato questa alla fine era buona o da abbandonare?