-
-
Notifications
You must be signed in to change notification settings - Fork 305
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[12.0][FIX] migration e-invoice 1.6 #1914
[12.0][FIX] migration e-invoice 1.6 #1914
Conversation
2612285
to
42711d3
Compare
42711d3
to
ccb0e4c
Compare
molto utile ho rilevato anche io il medesimo problema |
openupgrade.logged_query( | ||
env.cr, sql.SQL( | ||
""" | ||
INSERT INTO withholding_data_line |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Considerato che tra questa INSERT e la successiva cambia solo una colonna, potresti provare ad accorparle in una sola utilizzando le espressioni condizionali di PostgreSQL? Es. IF
o NULLIF
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Riesci a fare una PR a riguardo? Dovrei approfondire, al momento sono un po' tirato.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sergiocorato prima di mergiare la mia PR l'hai anche testata?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, ho solo guardato il codice
se quel campo è stato rimosso, cosa lo sostituiscie? possiamo considerarlo sempre null e bypassare quelle due query oppure provocherebbe un disastro? |
… script (DRY principle)
l10n_it_fatturapa_in: avoid duplication in the code of post migration script (DRY principle)
This PR has the |
/ocabot merge patch |
On my way to merge this fine PR! |
It looks like something changed on |
It looks like something changed on |
Congratulations, your PR was merged at 843baac. Thanks a lot for contributing to OCA. ❤️ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Descrizione del problema o della funzionalità: la migrazione dalla 11.0 fallisce perchè manca il campo
ftpa_withholding_amount
aggiunto e tolto sulla 12.0Comportamento attuale prima di questa PR: la migrazione da una versione precedente la 12.0 fallisce
Comportamento desiderato dopo questa PR: la migrazione da una versione precedente la 12.0 va a buon fine
--
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