Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ricevere fatture elettroniche in base64 via PEC #2915

Open
2 of 3 tasks
SirTakobi opened this issue Aug 25, 2022 · 12 comments · Fixed by #3500 · May be fixed by #4420
Open
2 of 3 tasks

Ricevere fatture elettroniche in base64 via PEC #2915

SirTakobi opened this issue Aug 25, 2022 · 12 comments · Fixed by #3500 · May be fixed by #4420
Labels
16.0 bug hotfix no stale Use this label to prevent the automated stale action from closing this PR/Issue. triaged

Comments

@SirTakobi
Copy link
Contributor

SirTakobi commented Aug 25, 2022

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:

(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
Copy link
Contributor

@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
Copy link
Contributor Author

SirTakobi commented Aug 25, 2022

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
Copy link
Contributor

sì mi pare ottimo

@francesco-ooops
Copy link
Contributor

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:

image

potete aggiornare il template?

@odooNextev
Copy link
Contributor

@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
Copy link
Contributor

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

odooNextev added a commit to odooNextev/l10n-italy that referenced this issue Jul 21, 2023
odooNextev pushed a commit to odooNextev/l10n-italy that referenced this issue Jul 21, 2023
odooNextev pushed a commit to odooNextev/l10n-italy that referenced this issue Jul 21, 2023
@TheMule71 TheMule71 added the 16.0 label Jul 21, 2023
odooNextev pushed a commit to odooNextev/l10n-italy that referenced this issue Jul 21, 2023
odooNextev added a commit to odooNextev/l10n-italy that referenced this issue Jul 21, 2023
Co-authored-by: odooNextev <odoo@nextev.it>
stenext pushed a commit to odooNextev/l10n-italy that referenced this issue Jan 8, 2024
Co-authored-by: odooNextev <odoo@nextev.it>
Copy link

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.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jun 23, 2024
@TheMule71 TheMule71 added no stale Use this label to prevent the automated stale action from closing this PR/Issue. and removed stale PR/Issue without recent activity, it'll be soon closed automatically. labels Jun 24, 2024
@micheledic
Copy link
Contributor

Novità su questa issue ?

@matteoopenf
Copy link
Contributor

Novità su questa issue ?

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

@micheledic
Copy link
Contributor

Novità su questa issue ?

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

attachment.name: attachment.datas,
sulla 16.0 non c'è ancora il b64decode , bisognerebbe fare un fwport della patch

@matteoopenf
Copy link
Contributor

Novità su questa issue ?

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

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 pushed a commit to ingegniamo/l10n-italy that referenced this issue Oct 22, 2024
Co-authored-by: odooNextev <odoo@nextev.it>
@micheledic
Copy link
Contributor

Novità su questa issue ?

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
16.0 bug hotfix no stale Use this label to prevent the automated stale action from closing this PR/Issue. triaged
Projects
None yet
7 participants