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.

Sign up for GitHub

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

Jump to bottom

[12.0] Errore durante l'importazione fattura elettronica #2093

Closed
3 tasks done
Borruso opened this issue Feb 3, 2021 · 5 comments
Closed
3 tasks done

[12.0] Errore durante l'importazione fattura elettronica #2093

Borruso opened this issue Feb 3, 2021 · 5 comments
Labels
14.0 tracking

Comments

@Borruso
Copy link
Contributor

Borruso commented Feb 3, 2021 •
edited by TheMule71
Loading

Buonasera,
per un cliente ci è arrivata una fattura elettronica con arrotondamento a -0.003.
Se questa fattura la importo con il minimo va in errore poiché cerca di prendere il max() delle righe in fattura, che al minimo non esistono, e invece che 0 va in errore per mancanza di argomento.
Ora bisogna impostare a 0 se le righe non ci sono o, dato che è al minimo, non aggiungere neanche la riga di arrotondamento??

Questo è il file XML:
IT08940791216_00325.zip
@SimoRubi
Copy link
Member

SimoRubi commented Feb 5, 2021

Verificato in runbot v12, dopo aver impostato round up/down account.
Facendo importazione minima, l'errore è:

[...]
  File "/home/odoo/build/OCA/l10n-italy/setup/l10n_it_fatturapa_in/odoo/addons/l10n_it_fatturapa_in/wizard/wizard_import_fatturapa.py", line 1488, in importFatturaPA
    fatt, fatturapa_attachment, fattura, partner_id)
  File "/home/odoo/build/OCA/l10n-italy/setup/l10n_it_fatturapa_in/odoo/addons/l10n_it_fatturapa_in/wizard/wizard_import_fatturapa.py", line 1020, in invoiceCreate
    self.set_roundings(FatturaBody, invoice)
  File "/home/odoo/build/OCA/l10n-italy/setup/l10n_it_fatturapa_in/odoo/addons/l10n_it_fatturapa_in/wizard/wizard_import_fatturapa.py", line 1133, in set_roundings
    line_sequence = max(invoice.invoice_line_ids.mapped('sequence'))
ValueError: max() arg is an empty sequence

@TheMule71
Copy link
Contributor

Stesso problema pure io:
image
image
IT05979361218_016.zip

TheMule71 added a commit to TheMule71/l10n-italy that referenced this issue Feb 9, 2021
@TheMule71
Risolve OCA#2093.
31626ad 
Quando viene importata con dettaglio minimo la fattura non ha righe,
la max() fallisce su recordset vuoto.
@TheMule71 TheMule71 mentioned this issue Feb 9, 2021
Merged
@TheMule71
Copy link
Contributor

Giuseppe e Simone, se potete, testate la PR.

TheMule71 added a commit to TheMule71/l10n-italy that referenced this issue Feb 24, 2021
@TheMule71
[FIX] l10n_it_fatturapa_in: fattura con arrotondamento importata con …
164ed6f 
…dettaglio minimo

Risolve OCA#2093.

Quando viene importata con dettaglio minimo la fattura non ha righe,
la max() fallisce su recordset vuoto.
TheMule71 added a commit to TheMule71/l10n-italy that referenced this issue Feb 24, 2021
@TheMule71
[FIX] l10n_it_fatturapa_in: fattura con arrotondamento importata con …
eb67d3f 
…dettaglio minimo

Risolve OCA#2093.

Quando viene importata con dettaglio minimo la fattura non ha righe,
la max() fallisce su recordset vuoto.
@eLBati
Copy link
Member

eLBati commented Feb 27, 2021

In seguito alla chiusura della #2107 , questa si può chiudere?

tafaRU pushed a commit to tafaRU/l10n-italy that referenced this issue Mar 23, 2021
@TheMule71 @tafaRU
[FIX] l10n_it_fatturapa_in: fattura con arrotondamento importata con …
cb2a111 
…dettaglio minimo

Risolve OCA#2093.

Quando viene importata con dettaglio minimo la fattura non ha righe,
la max() fallisce su recordset vuoto.
@tafaRU tafaRU mentioned this issue Mar 23, 2021
Merged
@tafaRU tafaRU closed this as completed Mar 24, 2021
@eLBati eLBati reopened this May 28, 2021
@TheMule71
Copy link
Contributor

Chiudo perché nella 14.0 è risolto.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
14.0 tracking
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@TheMule71 @eLBati @tafaRU @SimoRubi @Borruso