NemesiX / odoo

Odoo. Open Source Apps To Grow Your Business.
https://www.odoo.com
Other
0 stars 0 forks source link

Nnuova versione di Odoo da repository: Errore nel form sequenze. #3

Closed CodiceMatrix closed 2 years ago

CodiceMatrix commented 2 years ago

Ciao @giorgioborri , in produzione stiamo usando questo progetto odoo https://github.com/NemesiX/odoo/tree/7.0 quindi occorre che ti allenai anche tu.

ti ho inviato un file tramite wetransfer per facilitare le verifiche del caso.

All'interno del file hai la cartella odoo "VECCHIA" e "NUOVA" ovveo quella che si scarica da questo repository.

Schermata 2022-02-15 alle 15 18 51

Come puoi vedere mi sono fatto un LINK in modo da passare velocemente da una versione ad un altra. Ultimamente @rpiva ha fatto delle modifiche alla versione base di odoo per permettere alle nostre App sviluppate in flutter di collegarsi a odoo.

A parte questo il problema che vorrei che tu controllassi è quello che compare quando tento di entrare nel menù delle sequenze .

Schermata 2022-02-15 alle 15 37 16

Questo screenshot è preso da demo.nempos.com

CodiceMatrix commented 2 years ago

Questo è il link WeTransfer per scaricare direttamente la cartella odoo che c'è in laboratorio contenente la nuova versione e la vecchia del ramo 7.0 --> https://we.tl/t-iPByFg731N

giorgioborri commented 2 years ago

mi dici : modificato odoo per permettere alle nostre App sviluppate in flutter di collegarsi a odoo.

per il collegamento dall'esterno era sufficiente la libreria jsonrpc in dart. perché avete dovuto modificare odoo?

Il giorno mar 15 feb 2022 alle ore 15:38 Zizzo @.***> ha scritto:

Assigned #3 https://github.com/NemesiX/odoo/issues/3 to @giorgioborri https://github.com/giorgioborri.

— Reply to this email directly, view it on GitHub https://github.com/NemesiX/odoo/issues/3#event-6075463962, or unsubscribe https://github.com/notifications/unsubscribe-auth/APPR5ZUCWRMNSI5FPQKADH3U3JQNXANCNFSM5OOXB5BQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you were assigned.Message ID: @.***>

CodiceMatrix commented 2 years ago

Scusa , le WEB APP sviluppate in flutter! Le app già funzionavano anzi funzionano. Con la parte WEB ci sono dei limiti che ha aggirato @rpiva con le modifiche che ha fatto sul ramo ufficiale ma a parte questo avevamo già fatto un ramo della 7 e dovevamo usare quello perchè odoo 7 non è più manutenuto quindi è giusto farsi un ramo e continuare noi su quello. Però sembra esserci un problema sul ramo ufficiale.

giorgioborri commented 2 years ago

Per le sequenze era stata fatta qualche correzione perchè la gestione automatica dei progressivi fornita da postgres era cambiata passando dal 9.5 a 9.6 o a 10 (sicuramente nella versione 10 era cambiato) per cui era stato aggiornata la libreria base. se provate a confrontare questa nei 2 progetti new e old e a rimettere in new la modifica dovrebbe essere ciò che basta a far funzionare il sistema

giorgioborri commented 2 years ago

la libreria jsonrpc in dart dava problemi con odoo 7?

Il giorno mar 15 feb 2022 alle ore 16:12 Zizzo @.***> ha scritto:

Scusa , le WEB APP sviluppate in flutter! Le app già funzionavano anzi funzionano. Con la parte WEB ci sono dei limiti che ha aggirato @rpiva https://github.com/rpiva con le modifiche che ha fatto sul ramo ufficiale ma a parte questo avevamo già fatto un ramo della 7 e dovevamo usare quello perchè odoo 7 non è più manutenuto quindi è giusto farsi un ramo e continuare noi su quello. Però sembra esserci un problema sul ramo ufficiale.

— Reply to this email directly, view it on GitHub https://github.com/NemesiX/odoo/issues/3#issuecomment-1040387620, or unsubscribe https://github.com/notifications/unsubscribe-auth/APPR5ZQE5B5AHNF3XPOBI43U3JUONANCNFSM5OOXB5BQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you were mentioned.Message ID: @.***>

giorgioborri commented 2 years ago

ritornando alla sequenza: controllate i file 7.0/openerp/addons/base/ir/ir_sequence.py

rpiva commented 2 years ago

Sono stato costretto a fare delle modifiche per configurare il CORS necessario per far funzionare NemPOS Delivery V2 sottoforma di web app, altrimenti essendo NemPOS Delivery pubblicato sotto un altro dominio rispetto a NemPOS Backoffice il browser non gli faceva accedere a XMLRPC e JSONRPC.

Saluti, Roberto

Il giorno mar 15 feb 2022 alle ore 16:21 giorgioborri < @.***> ha scritto:

la libreria jsonrpc in dart dava problemi con odoo 7?

Il giorno mar 15 feb 2022 alle ore 16:12 Zizzo @.***> ha scritto:

Scusa , le WEB APP sviluppate in flutter! Le app già funzionavano anzi funzionano. Con la parte WEB ci sono dei limiti che ha aggirato @rpiva https://github.com/rpiva con le modifiche che ha fatto sul ramo ufficiale ma a parte questo avevamo già fatto un ramo della 7 e dovevamo usare quello perchè odoo 7 non è più manutenuto quindi è giusto farsi un ramo e continuare noi su quello. Però sembra esserci un problema sul ramo ufficiale.

— Reply to this email directly, view it on GitHub https://github.com/NemesiX/odoo/issues/3#issuecomment-1040387620, or unsubscribe < https://github.com/notifications/unsubscribe-auth/APPR5ZQE5B5AHNF3XPOBI43U3JUONANCNFSM5OOXB5BQ

. Triage notifications on the go with GitHub Mobile for iOS < https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675

or Android < https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .

You are receiving this because you were mentioned.Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/NemesiX/odoo/issues/3#issuecomment-1040399968, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABJETPGNQCKNPQQLWWJOGDDU3JVNVANCNFSM5OOXB5BQ . You are receiving this because you were mentioned.Message ID: @.***>

rpiva commented 2 years ago

Dal momento che Odoo 7.0 non è più manutenuto ho fatto un fork ed ho già fatto delle modifiche direttamente al codice "base" perchè non mi è parso che fosse possibile modificare tramite modulo la base di XMLRPC e JSONRPC

Saluti, Roberto

Il giorno mar 15 feb 2022 alle ore 16:23 giorgioborri < @.***> ha scritto:

ritornando alla sequenza: controllate i file 7.0/openerp/addons/base/ir/ir_sequence.py

— Reply to this email directly, view it on GitHub https://github.com/NemesiX/odoo/issues/3#issuecomment-1040406857, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABJETPDXZMPSI44PTXUPYWLU3JVYNANCNFSM5OOXB5BQ . You are receiving this because you were mentioned.Message ID: @.***>

CodiceMatrix commented 2 years ago

Ciao @giorgioborri , ho visto che le differenze sono in questi file:

root@Odoo-Lab-Lenovo:/opt/odoo# diff -r --brief --exclude=*.pyc 7.0.NEW 7.0.OLD
I file 7.0.NEW/addons/web/http.py e 7.0.OLD/addons/web/http.py sono diversi
I file 7.0.NEW/addons/web/static/src/js/pyeval.js e 7.0.OLD/addons/web/static/src/js/pyeval.js sono diversi
Solo in 7.0.OLD/addons/web/static/src/js: pyeval.js.ori
I file 7.0.NEW/addons/web/static/src/js/view_form.js e 7.0.OLD/addons/web/static/src/js/view_form.js sono diversi
Solo in 7.0.NEW: .git
I file 7.0.NEW/openerp/addons/base/ir/ir_sequence.py e 7.0.OLD/openerp/addons/base/ir/ir_sequence.py sono diversi
I file 7.0.NEW/openerp/service/wsgi_server.py e 7.0.OLD/openerp/service/wsgi_server.py sono diversi
rpiva commented 2 years ago

Io ho modificato solo http.py e wsgi_server.py gli altri non so cosa siano

Ci sarebbe da andare più nel dettaglio del ir_sequence.py per capire cosa ci sia di diverso e valutare se portare la versione "old" nel repo nuovo. Stessa cosa vale per i js

Saluti, Roberto

Il giorno mar 15 feb 2022 alle ore 16:32 Zizzo @.***> ha scritto:

Ciao @giorgioborri https://github.com/giorgioborri , ho visto che le differenze sono in questi file:

@.**:/opt/odoo# diff -r --brief --exclude=.pyc 7.0.NEW 7.0.OLD I file 7.0.NEW/addons/web/http.py e 7.0.OLD/addons/web/http.py sono diversi I file 7.0.NEW/addons/web/static/src/js/pyeval.js e 7.0.OLD/addons/web/static/src/js/pyeval.js sono diversi Solo in 7.0.OLD/addons/web/static/src/js: pyeval.js.ori I file 7.0.NEW/addons/web/static/src/js/view_form.js e 7.0.OLD/addons/web/static/src/js/view_form.js sono diversi Solo in 7.0.NEW: .git I file 7.0.NEW/openerp/addons/base/ir/ir_sequence.py e 7.0.OLD/openerp/addons/base/ir/ir_sequence.py sono diversi I file 7.0.NEW/openerp/service/wsgi_server.py e 7.0.OLD/openerp/service/wsgi_server.py sono diversi

— Reply to this email directly, view it on GitHub https://github.com/NemesiX/odoo/issues/3#issuecomment-1040416715, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABJETPDDB2OLJKBBE7A6BKLU3JWYBANCNFSM5OOXB5BQ . You are receiving this because you were mentioned.Message ID: @.***>

giorgioborri commented 2 years ago

ho visto i commit che avete fatto: nei file js ci sono correzioni per la dimensione del upload ammesso ora fino a 100mb

il file ir_sequence in old invece contenere le correzioni aggiunte per gestire i seriali nel nuovo modo di postgres 9.6 o 10 sicuramente

Il giorno mar 15 feb 2022 alle ore 16:44 Roberto :-p < @.***> ha scritto:

Io ho modificato solo http.py e wsgi_server.py gli altri non so cosa siano

Ci sarebbe da andare più nel dettaglio del ir_sequence.py per capire cosa ci sia di diverso e valutare se portare la versione "old" nel repo nuovo. Stessa cosa vale per i js

Saluti, Roberto

Il giorno mar 15 feb 2022 alle ore 16:32 Zizzo @.***> ha scritto:

Ciao @giorgioborri https://github.com/giorgioborri , ho visto che le differenze sono in questi file:

@.**:/opt/odoo# diff -r --brief --exclude=.pyc 7.0.NEW 7.0.OLD I file 7.0.NEW/addons/web/http.py e 7.0.OLD/addons/web/http.py sono diversi I file 7.0.NEW/addons/web/static/src/js/pyeval.js e 7.0.OLD/addons/web/static/src/js/pyeval.js sono diversi Solo in 7.0.OLD/addons/web/static/src/js: pyeval.js.ori I file 7.0.NEW/addons/web/static/src/js/view_form.js e 7.0.OLD/addons/web/static/src/js/view_form.js sono diversi Solo in 7.0.NEW: .git I file 7.0.NEW/openerp/addons/base/ir/ir_sequence.py e 7.0.OLD/openerp/addons/base/ir/ir_sequence.py sono diversi I file 7.0.NEW/openerp/service/wsgi_server.py e 7.0.OLD/openerp/service/wsgi_server.py sono diversi

— Reply to this email directly, view it on GitHub https://github.com/NemesiX/odoo/issues/3#issuecomment-1040416715, or unsubscribe < https://github.com/notifications/unsubscribe-auth/ABJETPDDB2OLJKBBE7A6BKLU3JWYBANCNFSM5OOXB5BQ

. You are receiving this because you were mentioned.Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/NemesiX/odoo/issues/3#issuecomment-1040430488, or unsubscribe https://github.com/notifications/unsubscribe-auth/APPR5ZQWP3ZEPVPMGPG26HDU3JYHDANCNFSM5OOXB5BQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you were mentioned.Message ID: @.***>

CodiceMatrix commented 2 years ago

Ciao @rpiva , ho trovato questa vecchia mail. Ti ricorda qualcosa? Perchè a questo punto basta rimettere il file ir_sequence.py precedente che è comunque quello più aggiornato.

Schermata 2022-02-15 alle 17 58 30

In allegato alla tua mail del 18 dic 2018 alle 15:54 il file era questo :

ir_sequence.py.zip

rpiva commented 2 years ago

Azz Allora ci sono da tenere buoni i file .js e ir_sequence.py della versione vecchia. poi push e via di nuova immagine.

Saluti, Roberto

Il giorno mar 15 feb 2022 alle ore 18:01 Zizzo @.***> ha scritto:

Ciao @rpiva https://github.com/rpiva , ho trovato questa vecchia mail. Ti ricorda qualcosa? Perchè a questo punto basta rimettere il file ir_sequence.py precedente che è comunque quello più aggiornato.

[image: Schermata 2022-02-15 alle 17 58 30] https://user-images.githubusercontent.com/63808967/154110921-ed50f1af-7fa1-4920-bf9c-40aa911a75ce.png

In allegato alla tua mail del 18 dic 2018 alle 15:54 il file era questo :

ir_sequence.py.zip https://github.com/NemesiX/odoo/files/8073045/ir_sequence.py.zip

— Reply to this email directly, view it on GitHub https://github.com/NemesiX/odoo/issues/3#issuecomment-1040528970, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABJETPGLLC2PVCYWKLMG22DU3KBG7ANCNFSM5OOXB5BQ . You are receiving this because you were mentioned.Message ID: @.***>

CodiceMatrix commented 2 years ago

Preso file ir_sequence.py della versione vecchia e fatto commit e push.

giorgioborri commented 2 years ago

hai trovato le differenze nei file

I file 7.0.NEW/addons/web/static/src/js/pyeval.js e 7.0.OLD/addons/web/static/src/js/pyeval.js sono diversi

Questo file in particolare era stato corretto per permettere di semplificare i calcoli di date in nx_hr -> interfaccia turni ed anche in alcuni filtri search

si dovrebbe copiare la versioni in old e mettere in new

Il giorno mar 15 feb 2022 alle ore 16:32 Zizzo @.***> ha scritto:

Ciao @giorgioborri https://github.com/giorgioborri , ho visto che le differenze sono in questi file:

@.**:/opt/odoo# diff -r --brief --exclude=.pyc 7.0.NEW 7.0.OLD I file 7.0.NEW/addons/web/http.py e 7.0.OLD/addons/web/http.py sono diversi I file 7.0.NEW/addons/web/static/src/js/pyeval.js e 7.0.OLD/addons/web/static/src/js/pyeval.js sono diversi Solo in 7.0.OLD/addons/web/static/src/js: pyeval.js.ori I file 7.0.NEW/addons/web/static/src/js/view_form.js e 7.0.OLD/addons/web/static/src/js/view_form.js sono diversi Solo in 7.0.NEW: .git I file 7.0.NEW/openerp/addons/base/ir/ir_sequence.py e 7.0.OLD/openerp/addons/base/ir/ir_sequence.py sono diversi I file 7.0.NEW/openerp/service/wsgi_server.py e 7.0.OLD/openerp/service/wsgi_server.py sono diversi

— Reply to this email directly, view it on GitHub https://github.com/NemesiX/odoo/issues/3#issuecomment-1040416715, or unsubscribe https://github.com/notifications/unsubscribe-auth/APPR5ZVB2U6CM7HC4UX5WADU3JWYBANCNFSM5OOXB5BQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you were mentioned.Message ID: @.***>

CodiceMatrix commented 2 years ago

Si perfetto. Abbiamo già preso il vecchio file ir_sequence.py e messo i questo repository che è poi quello che viene scaricato durante la creazione di un Docker. ( c'è un mio commit ) Bisogna che in sviluppo cominciamo ad usare questo repository , c'è solo un problema che dobbiamo risolvere quando si usa http invece di https ma risolto quello poi dobbiamo usarlo in sviluppo.

giorgioborri commented 2 years ago

ti ho indicato un altro file: pyeval.js anche questo va considerato dal old a new pensavo che lo controllassi ieri dato che ti è risultato con la ricerca di differenze

Il giorno mer 16 feb 2022 alle ore 11:26 Zizzo @.***> ha scritto:

Si perfetto. Abbiamo già preso il vecchio file ir_sequence.py e messo i questo repository che è poi quello che viene scaricato durante la creazione di un Docker. ( c'è un mio commit ) Bisogna che in sviluppo cominciamo ad usare questo repository , c'è solo un problema che dobbiamo risolvere quando si usa http invece di https ma risolto quello poi dobbiamo usarlo in sviluppo.

— Reply to this email directly, view it on GitHub https://github.com/NemesiX/odoo/issues/3#issuecomment-1041337046, or unsubscribe https://github.com/notifications/unsubscribe-auth/APPR5ZRFDYYBH6HGCJES5A3U3N3VLANCNFSM5OOXB5BQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you were mentioned.Message ID: @.***>

CodiceMatrix commented 2 years ago

Ciao @giorgioborri , non l'ho fatto perchè ero convinto che fosse per il file della dimensione del Upload ma non è così. Queste sono le differenze qualcuno si ricorda perchè è stata fatta questa modifica?

Schermata 2022-02-16 alle 12 24 30

giorgioborri commented 2 years ago

ti avevo già indicato nella prima mail cosa era stato fatto: sono stati aggiunti dei parametri in alcune funzioni da essere usate nel modulo nx_hr interfaccia turni

Il giorno mer 16 feb 2022 alle ore 12:26 Zizzo @.***> ha scritto:

Ciao @giorgioborri https://github.com/giorgioborri , non l'ho fatto perchè ero convinto che fosse per il file della dimensione del Upload ma non è così. Queste sono le differenze qualcuno si ricordo perchè è stata fatta questa modifica?

[image: Schermata 2022-02-16 alle 12 24 30] https://user-images.githubusercontent.com/63808967/154255275-74add077-fa65-45e0-b88a-38ae80606d7f.png

— Reply to this email directly, view it on GitHub https://github.com/NemesiX/odoo/issues/3#issuecomment-1041388811, or unsubscribe https://github.com/notifications/unsubscribe-auth/APPR5ZQBI3ZNPZEYE2RP4GLU3OCU5ANCNFSM5OOXB5BQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you were mentioned.Message ID: @.***>

CodiceMatrix commented 2 years ago

OK @giorgioborri , ho creato il commit f974dd675087201c5455f7a8e61f17a721dc5d13 che contiene il file pyeval.js corretto per HR.

CodiceMatrix commented 2 years ago

@rpiva alla prossima immagine docker ricordati che c'è anche da pullare la cartella odoo.

rpiva commented 2 years ago

Meglio che tu lo faccia direttamente piuttosto che ci dimentichiamo

CodiceMatrix commented 2 years ago

Ho fatto la cartella 7.0. Quali altre cartelle devo fare?

Schermata 2022-02-18 alle 18 51 45