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

Formato data fatture importate (fpa in) #4580

Open
2 tasks
TheMule71 opened this issue Jan 28, 2025 · 0 comments
Open
2 tasks

Formato data fatture importate (fpa in) #4580

TheMule71 opened this issue Jan 28, 2025 · 0 comments

Comments

@TheMule71
Copy link
Contributor

Module

l10n_it_fatturapa_in

Describe the bug

Vengono salvate date in un formato dipendente dalla lingua dell'utente che lo fa, in quel momento.

To Reproduce

Affected versions:

  • 16.0 sì
  • 14.0 ? (da controllare)

Steps to reproduce the behavior:

  1. Imposta una lingua che abbia un formato data MM/DD/YYYY
  2. Importa una f.e.
  3. Imposta una lingua con un formato data DD/MM/YYYY
  4. Nel DB sono presenti date con formati diversi nello stesso campo (invoices_date)

Expected behavior
Le date su DB dovrebbero avere un formato consistente, possibilmente non ambiguo (stile ISO, YYYY/MM/DD)

Additional context
Questo il codice incriminato:

document_date = invoice_body.DatiGenerali.DatiGeneraliDocumento.Data
invoice_date = format_date(
att.with_context(lang=att.env.user.lang).env,
fields.Date.from_string(document_date),
)

La cosa è stata verificata nel mondo reale, abbiamo un cliente che ha metà allegati con date DD/MM/YYYY e metà con MM/DD/YYYY. Nel DB sono presenti sia 10/31/2024 sia 30/11/2024.

La cosa è particolarmente fastidiosa perché alcune sono ambigue e l'unico modo per sistemare è osservare i record con ID successivi e precedenti e cercare di capire con quale formato sono state importate. Per es. avevamo 10/11/2024. Abbiamo dedotto che le fatture in ottobre sono state importate MM/DD/YYYY e quelle di novembre DD/MM/YYYY ma non basta... quella fattura potrebbe essere entrambe. Solo grazie all'ID (era stata importata nel batch di ottobre) l'abbiamo dedotto.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant