[16.0] [FIX] l10n_it_fatturapa_out: attach report to a xml invoice pops an error on "report_print_menu" field #4156
Conversation
…ps an error on "report_print_menu" field
micheledic
force-pushed
the
16.0-fix_attach_report
branch
from
8e3ca22
to
32ad2c4
Compare
|
andato in fail il precommit, qual è il problema? |
Te lo dice la action stessa: In generale, installa pre-commit in locale e fallo girare lì prima di pushare così vedi gli errori (molti dei quali risolve black autonomamente cambiando il codice) |
|
@micheledic ciao, puoi aprire una issue per tracciare il problema con i passi per riprodurre? |
#4159 ecco a te Principalmente credo sia un problema di contesto, visto che i values dipendono dal contesto se si perde il contesto nel ping-pong di chiamate allora non riesce a validare il value |
| report_print_menu = fields.Selection( | ||
| selection="_get_selection", | ||
| help="This report will be automatically included in the created XML", | ||
| report_print_menu = fields.Many2one( | ||
| "ir.actions.report", |
There was a problem hiding this comment.
Sembra un rollback di 0315ba5, @TheMule71 @michelerusti cosa ne pensate?
There was a problem hiding this comment.
Ho visto anche la PR che ha portato da Many2one a Selection , ma secondo il mio parere ha più senso sia una many2one visto anche che nel caso di Selection, viene riportato errore se viene settato quel campo senza contesto active_model ....
Cioè gli attributi dell'entità del wizard, non dovrebbero essere strict rispetto al contesto ma il contesto dovrebbe solamente suggerire il report da utilizzare. Poi usando widget=selection anche la many2one diventa come selection e quindi trasparente per ,l'utente finale
Poi chi dovrà provvedere ad esportare le fatture, avrà sicuramente anche accesso al modello ir.actions.report quindi non dovrebbero esserci problemi di permessi d'accesso
Ovviamente parere mio tecnico, potrei anche sbagliarmi
SirAionTech
added
the
needs fixing
Quando provi ad allegare un report ad un XML di fattura, esce un errore perchè è stata usata fields.Selection .
Inoltre non funziona la domain.
Con questa PR vengono fixate entrambe le cose