[FIX] l10n_it_fatturapa_pec - user without permission to access settings can't send PEC#3933
Conversation
…ngs can send a PEC
michelerusti
force-pushed
the
16.0-fix-l10n_it_fatturapa_pec-sendy-pec-permission
branch
from
d67e9df to
c46ffa6
Compare
SirAionTech
left a comment
SirAionTech
left a comment
There was a problem hiding this comment.
Grazie della PR!
Puoi aggiungere un test? Così si evitano regressioni.
Ho provato a riprodurre l'errore nel runboat di 16.0 ma ottengo
perché non posso confermare il servire di posta in ingresso PEC, sai mica se ci sono dei server di posta da poter configurare per i test? Ho provato a cercare ma trovo solo servizi per testare un nuovo server di posta 😅
| } | ||
| mail = ( | ||
| self.env["mail.mail"] | ||
| .sudo() |
There was a problem hiding this comment.
Senza questo che errore viene sollevato? Nella issue parlavi solo di errori per ir.config_parameter (che immagino vengano corretti dall'altra modifica)
There was a problem hiding this comment.
Creazione dei record "Posta in uscita" (mail.mail) non consentita.
L'operazione è consentita per i seguenti gruppi:
- Amministrazione/Impostazioni
Se necessario, contattare l'amministratore per richiedere l'accesso.
|
|
||
| def check_first_pec_sending(self): | ||
| sdi_address = self.env["ir.config_parameter"].get_param("sdi.pec.first.address") | ||
| sdi_address = ( |
There was a problem hiding this comment.
Questa la metterei dentro l'if altrimenti verrebbe eseguita sempre anche dopo aver ricevuto l'indirizzo finale
| } | ||
| mail = ( | ||
| self.env["mail.mail"] | ||
| .sudo() |
There was a problem hiding this comment.
Mi sembra strano perchè questa necessità non mi è mai capitata anche con utenti senza diritti di amministrazione
There was a problem hiding this comment.
Inizialmente chiede cliccando su "invia e stampa" chiede i diritti di accesso:
Accesso ai record "Modelli" (ir.model) non consentito.
L'operazione è consentita per i seguenti gruppi:
- Amministrazione/Diritti di accesso
Se necessario, contattare l'amministratore per richiedere l'accesso.
In questo runboat non lo fa perchè bisognerebbe fare un rebase comprendendo anche la modifica successiva a questa PR fatta da @monen17 ed @eLBati in cui probabilmente bisognerebbe aggiungere sudo() per la creazione di mail.message: abb391f#diff-c2fcfef6f58895806d38a8668b81bc2174837142537bd2ce7b04e04c977c884aR133
Aggiungendoli crea l'XML, ma cliccando "Invia SDI" all'utente compare quello che ho scritto sopra legato a mail.mail
|
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. |
github-actions
Bot
added
the
stale
5 participants
Risolve #3932 per 16.0