search

OCA / l10n-italy

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

[l10n_it_fatturapa_in] Errore visualizzazione campo o2m in vista wizard "collegamento fattura esistente" #3573

Closed CiroBoxHub closed 11 months ago

CiroBoxHub commented 1 year ago

C'è un errore di visualizzazione del campo o2m nella finestra wizard della funzione "collega a fattura esistente"

Module

l10n_it_fatturapa_in

Describe the bug

screenshot-oca-l10n-italy-16-0-fb5911852a87 runboat odoo-community org-2023 09 13-17_05_23

To Reproduce

CPT2309131706-1462x783

Affected versions:

mlopesbjj commented 1 year ago

Buongiorno, Siccome non conosco tanto bene ODOO, ma sono un sviluppatore, se qualcuno mi dice dove posso fare la modifica posso provare a sistemare.

Grazie.

FrancescoBallerini commented 1 year ago

@mlopesbjj Buongiorno a te, dovrebbe essere sufficiente sistemare l'xml del campo one2many line_ids che trovi qui https://github.com/OCA/l10n-italy/blob/9dd1526f38c6399543c82f010992193d6f10d12d/l10n_it_fatturapa_in/wizard/link_to_existing_invoice.xml#L11-L27

Grazie

mlopesbjj commented 1 year ago

Grazie Francesco, però ho provato modificare ma non cambia. Ho provato mettere width o colspan, ma non ho avuto successo, forse perque non conosco bene l'interfaccia di odoo.

FrancescoBallerini commented 1 year ago

@mlopesbjj @CiroBoxHub Ho fatto qualche prova modificando le viste dal runbot della 16: da quello che ho capito in Odoo16 i campi one2many (many2many non testati) hanno questo problema in determinate condizioni. Condizione comune che causa il problema sembra la seguente:

Se si verificano queste due condizioni non sembra possibile influire sulla width neanche riassegnando manualmente gli attributi col e colspan. Poiché il div non sembra causare questo inconveniente è possibile gestire la width via div, se necessario.

image

image

Girando un pò per le viste ho inoltre riscontrato questo problema in altri campi, ad esempio la vista https://github.com/OCA/l10n-italy/blob/9dd1526f38c6399543c82f010992193d6f10d12d/l10n_it_fatturapa_out/views/account_view.xml#L15

ha il campo related_documents racchiuso nel <group/> e risente dello stesso inconveniente. Anche in questo campo si ripristina il <group/> per ripristinare la width.

Nella stessa form, anche il campo is_pdf_invoice_print ha lo stesso problema, però sono da runbot quindi non sono riuscito ad individuare bene la vista che la implementa.

Mi sembra un inconveniente rilevabile su più viste diverse. Forse nel processo di migrazione è stato lasciato su alcuni campi, può essere? Proporrei di tenere questa issue (o farne una più generica) come riferimento, risolvere il problema in un'unica PR, aggiungendo altri fix se vengono segnalate altre viste/campi, io per ora ho trovato questi 2 oltre a quello segnalato.

OpenCode commented 11 months ago

@CiroBoxHub Va bene per te se aggiungiamo qiesta tua segnalazione come punto specifico della issue https://github.com/OCA/l10n-italy/issues/3646 e la chiudiamo per evitare ridondanze?

CiroBoxHub commented 11 months ago

@CiroBoxHub Va bene per te se aggiungiamo qiesta tua segnalazione come punto specifico della issue #3646 e la chiudiamo per evitare ridondanze?

Si @OpenCode nessun problema. Se la problematica è strettamente dipendente dalla #3646 possiamo pure chiudere e inserirla come punto.

OpenCode commented 11 months ago

Chiusa in favore di https://github.com/OCA/l10n-italy/issues/3646