Ricevere fatture elettroniche in base64 via PEC

[SirTakobi]

Module

l10n_it_fatturapa_pec

Describe the bug

La fattura elettronica viene creata e presenta l'errore

Xml 'NoneType' object has no attribute 'iter' non valido.

To Reproduce

Affected versions:

• 12.0: [FIX] Ricevere fatture elettroniche in base64 via PEC #2916
• 14.0: [14.0] [FIX] Ricevere fatture elettroniche in base64 via PEC #2915 #3500
• 16.0: [FW][FIX] l10n_it_fatturapa_pec: Receive base64 attachments OCA#2915 #4420

(per la 16 si attendeva il merge di #3785 )

Steps to reproduce the behavior:

1. Nella PEC, ricevere un'email contenente un file codificato in base64

Expected behavior
La fattura elettronica viene creata senza errori

Additional context
Non credo che #2854 sia collegata perché la fattura allegata non è codificata
Leggendo meglio la #2854 si parla di

Le fatture elettroniche firmate digitalmente, con il allegato il file criptato p7m NON vengono parsate!

quindi potrebbe essere collegata.

[francesco-ooops]

@SirTakobi @OCA/local-italy-maintainers una questione riguardante tutte le issues: quando in "affected versions" c'è solo una versione segnalata e non l'altra (considerando 12 e 14 come quelle mantenute da Italy), molto spesso ci sono dei commenti che chiedono se il bug non è presente nell'altra versione o se non è stato verificato se è presente o meno

sarebbe possibile includere sempre entrambi i checkbox per la 12 e la 14 sotto "affected versions"?
Poi le opzioni possibili per popolarli sarebbero:

• Non verificata la presenza del bug
• Bug presente
• Bug non presente
• PR (quando viene aperta)

Credo questo aiuterebbe a far chiarezza sullo status delle altre versioni, che ne pensate?

[SirTakobi]

Per le versioni che non sono scritte, io di solito suppongo che non sia stato verificato (in questo caso è così).

Se è stato fatto qualcosa nelle altre versioni (verifica o meno del bug, PR o altro), penso sia normale scriverlo nell'apposita casellina.

Per non lasciare nulla di sottinteso si potrebbe modificare il template impostando le versioni supportate a

Non verificata la presenza del bug

EDIT: O anche un più semplice

Non verificato

visto che il titoletto Affected versions già suppone si stia parlando del bug in questione

[francesco-ooops]

sì mi pare ottimo

[francesco-ooops]

Per le versioni che non sono scritte, io di solito suppongo che non sia stato verificato (in questo caso è così).

Se è stato fatto qualcosa nelle altre versioni (verifica o meno del bug, PR o altro), penso sia normale scriverlo nell'apposita casellina.

Per non lasciare nulla di sottinteso si potrebbe modificare il template impostando le versioni supportate a

Non verificata la presenza del bug

EDIT: O anche un più semplice

Non verificato

visto che il titoletto Affected versions già suppone si stia parlando del bug in questione

anche solo così è più che chiaro:

potete aggiornare il template?

[odooNextev]

@SirTakobi si verifica anche nella v14
https://github.com/OCA/l10n-italy/blob/14.0/l10n_it_fatturapa_pec/models/mail_thread.py#L223

[odooNextev]

@SirTakobi ho aperto una PR copiando ed incollando esattamente quanto avevi fatto: #3500

[github-actions]

There hasn't been any activity on this issue in the past 6 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this issue to never become stale, please ask a PSC member to apply the "no stale" label.

[micheledic]

Novità su questa issue ?

[matteoopenf]

Novità su questa issue ?

le pr sono state mergiate hai ancora il problema con l'ultima versione??

[micheledic]

Novità su questa issue ?

le pr sono state mergiate hai ancora il problema con l'ultima versione??

l10n-italy/l10n_it_fatturapa_pec/models/mail_thread.py

Line 225 in 080f9b4

attachment.name: attachment.datas,

sulla 16.0 non c'è ancora il b64decode , bisognerebbe fare un fwport della patch

[matteoopenf]

Novità su questa issue ?

le pr sono state mergiate hai ancora il problema con l'ultima versione??

l10n-italy/l10n_it_fatturapa_pec/models/mail_thread.py

Line 225 in 080f9b4

attachment.name: attachment.datas,

sulla 16.0 non c'è ancora il b64decode , bisognerebbe fare un fwport della patch

puoi fare una pr in proposito?

[micheledic]

Novità su questa issue ?

le pr sono state mergiate hai ancora il problema con l'ultima versione??

l10n-italy/l10n_it_fatturapa_pec/models/mail_thread.py

Line 225 in 080f9b4

attachment.name: attachment.datas,

sulla 16.0 non c'è ancora il b64decode , bisognerebbe fare un fwport della patch

puoi fare una pr in proposito?

#4420 ho cherry-pickato la patch