[16.0] new module l10n_it_fatturapa_out_dn

[TheMule71]

Fixes #4005

Modulo di collegamento tra l10n_it_fatturapa_out e l10n_it_delivery_note
vd. anche #2202

[MaurizioPellegrinet]

LGTM

[francesco-ooops]

/ocabot rebase

[OCA-git-bot]

@francesco-ooops The rebase process failed, because command git rebase origin/16.0 failed with output:

Rebasing (1/2)
Auto-merging l10n_it_fatturapa_out/data/invoice_it_template.xml
CONFLICT (content): Merge conflict in l10n_it_fatturapa_out/data/invoice_it_template.xml
error: could not apply 05ae68b5a... [REF] l10n_it_fatturapa_out: Allow to exclude lines from e-invoice
hint: Resolve all conflicts manually, mark them as resolved with
hint: "git add/rm <conflicted_files>", then run "git rebase --continue".
hint: You can instead skip this commit: run "git rebase --skip".
hint: To abort and get back to the state before "git rebase", run "git rebase --abort".
Could not apply 05ae68b5a... [REF] l10n_it_fatturapa_out: Allow to exclude lines from e-invoice

[francesco-ooops]

@TheMule71 puoi fare rebase?

[TheMule71]

ValueError: External ID not found in the system: l10n_it.l10n_it_chart_template_generic

La modifica è stata introdotta con #4176. Per caso è cambiato qualcosa nell'ambiente di esecuzione dei test (e l10n_it non è più presente?)

In ogni caso prob. va fixato visto che non deve dipendere da l10n_it.

[odooNextev]

Ho cambiato questa parte così per mettere il riferimentonumerolinea solo se c'è più di un DN collegato e togliere le righe nota/sezione:

if len(invoice.delivery_note_ids) > 1:
    # RiferimentoNumeroLinea should not be populated
    # if all the lines of the invoice
    # are linked to this delivery_note
    e_invoice_lines = self.get_e_invoice_lines(invoice)
    e_invoice_delivery_note_lines = e_invoice_lines.filtered(
        lambda eil, delivery_note=delivery_note:
        eil.delivery_note_id == delivery_note and eil.display_type == 'product')

    e_invoice_lines_list = list(e_invoice_lines)
    lines_refs_list = [
        e_invoice_lines_list.index(line) + 1  # NumeroLinea is 1-based
        for line in e_invoice_delivery_note_lines
    ]

    ddt_data.update(
        {
            "_invoice_lines": e_invoice_delivery_note_lines,
            "RiferimentoNumeroLinea": lines_refs_list,
        }
    )

    dati_ddt_list.append(ddt_data)

[TheMule71]

Non seguo la logica, in che senso se len(invoice.delivery_note_ids) > 1 allora tutte le righe fattura sono collegate allo stesso DDT?
Potrebbero esserci righe che non sono collegate al DDT.

È vero che probabilmente la maggior parte delle volte non sarebbe un problema perché tali righe saranno magari per servizi e quindi va da sé che non riguardano il DDT. Però è una considerazione di pratica, in teoria nulla vieta questo scenario:

crei SO -> consegni -> crei DDT -> crei fattura
ora aggiungi uno o più prodotti alla fattura

prima avrebbe riportato nell'XML i riferimenti alle sole righe derivate del DDT;
se non sbaglio, con questa modifica, riporterebbe l'intera fattura riferita al quel DDT, e se uno va a cercare i prodotti aggiunti dopo in fattura nel DDT non li trova.

[odooNextev]

Non seguo la logica, in che senso se len(invoice.delivery_note_ids) > 1 allora tutte le righe fattura sono collegate allo stesso DDT?

No, collego solo quelle che appartengono ai DN come si vede in questo passaggio:

e_invoice_delivery_note_lines = e_invoice_lines.filtered(
        lambda eil, delivery_note=delivery_note:
        eil.delivery_note_id == delivery_note and eil.display_type == 'product')

se non sbaglio, con questa modifica, riporterebbe l'intera fattura riferita al quel DDT, e se uno va a cercare i prodotti aggiunti dopo in fattura nel DDT non li trova.

se committi queste modifiche le si può provare anche nel runboat e dovresti vedere che funziona
lo stiamo usando in produzione da un cliente, ho fatto un test con il flusso che hai indicato e correttamente la riga aggiunta post in fattura non viene collegata a DN

[TheMule71]

Ok, ma quello lo fai se len(invoice.delivery_note_ids) > 1.

Il caso che ho portato era per len(invoice.delivery_note_ids) == 1.

Rileggendo nel mio commento precedente non è chiaro, riformulo:

Non seguo la logica, in che senso se len(invoice.delivery_note_ids) == 1 allora tutte le righe fattura sono collegate allo stesso DDT?

[TheMule71]

Ok ho rivisto alcune cose, adesso delivery_note_id è popolato anche per le righe prodotto, per cui è possibile controllare quando aggiungere i riferimenti alle righe oppure no nella stanza DatiDDT

[TheMule71]

Ok, come non detto, delivery_note_id NON è popolato per le righe prodotto.

Si torna a 173564b

l10n_it_delivery_note non è ancora pronto per la patch, ma ai suoi figli piacerà

@odooNextev

e_invoice_delivery_note_lines = e_invoice_lines.filtered(
        lambda eil, delivery_note=delivery_note:
        eil.delivery_note_id == delivery_note and eil.display_type == 'product')

è sempre vuoto.

[sergiocorato]

(faccio minor perchè c'è un'aggiunta su altro modulo)
/ocabot merge minor

[OCA-git-bot]

What a great day to merge this nice PR. Let's do it!
Prepared branch 16.0-ocabot-merge-pr-4054-by-sergiocorato-bump-minor, awaiting test results.

[OCA-git-bot]

@sergiocorato your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-4054-by-sergiocorato-bump-minor.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

[sergiocorato]

@TheMule71 manca una configurazione sul pagamento su un test, non so come non fosse stata rilevata prima, se puoi correggerla

[monen17]

I test dovrebbero fallire anche qui nella PR se il codice fosse più aggiornato, lo aggiorno.
/ocabot rebase

[OCA-git-bot]

Congratulations, PR rebased to 16.0.

[monen17]

@TheMule71 puoi correggere i test? Altrimenti non si può mergiare

[TheMule71]

@TheMule71 manca una configurazione sul pagamento su un test, non so come non fosse stata rilevata prima, se puoi correggerla

Tra l'altro non ho ben capito come mai i test si sono rotti, erano verdi prima, deve essere cambiato qualcosa: magari check più aggressivi opppure un default che prima non c'era. Comunque ho impostato il metodo di pagamento corretto e aggiornato l'XML del test di conseguenza.

[monen17]

Grazie della PR!
Il modulo è stato mergiato in 14.0 con #2202 quindi questa è una migrazione, puoi riscrivere i commit come indicato in https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-16.0? Così è più chiaro cosa è cambiato da 14.0 a 16.0.

[monen17]

/ocabot migration l10n_it_fatturapa_out_dn

[TheMule71]

Veramente le due PR sono andate avanti in parallelo, tant'è che questa è stata creata un anno prima che l'altra venisse mergiata. I commit sono stati portati a mano.

Comunque si può simulare una migrazione e riportare le modifiche:

#5154