[16.0][FIX] Fixed invoice ref number for passive invoices in vat registries #4206
Conversation
|
Hi @eLBati, |
There was a problem hiding this comment.
Grazie della PR!
Potresti modificare il messaggio del commit per seguire https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#71commit-message?
In particolare:
please check if the commit message is cut with ellipsis
Potresti separare i due commit in modo che ogni commit modifichi un solo modulo? Anche questo fa parte delle linee guida linkate sopra:
Try to split into different commits where impacted modules are different
| @@ -26,14 +36,13 @@ | |||
| <t t-set="original_supplier" t-value="original_invoice.partner_id" /> | |||
| <t | |||
| t-set="reference" | |||
| t-value="original_invoice.ref or original_invoice.name or ''" | |||
| t-value="(original_invoice.ref or original_invoice.name or '') + ' - ' + (move.rc_purchase_invoice_id.ref or move.rc_purchase_invoice_id.name or '')" | |||
There was a problem hiding this comment.
Se non ci sono né .ref né .name, viene aggiunto solo " - ": è possibile non aggiungere nulla in questo caso?
There was a problem hiding this comment.
Non mi risultano casi dove name di account.move possa essere vuoto, quindi potremmo non preoccuparcene
There was a problem hiding this comment.
Non mi risultano casi dove
namediaccount.movepossa essere vuoto, quindi potremmo non preoccuparcene
Il campo account.move.name non è required:
name = fields.Char(
string='Number',
compute='_compute_name', inverse='_inverse_name', readonly=False, store=True,
copy=False,
tracking=True,
index='trigram',
)quindi può essere vuoto.
Può succedere ad esempio quando l'utente svuota il campo manualmente, come viene suggerito da Odoo nei casi in cui la sequenza non è valida:
"You will need to clear the %(model)s's %(sequence_field)s to proceed.\n"
There was a problem hiding this comment.
Intendo che in questo caso sto stampando un'autofattura e recuperando i dati della fattura originale. Se esiste l'autofattura vuol dire che la fattura originale è validata e una fattura validata non mi risulta possa non avere un numero
|
/ocabot rebase |
…number of passive self invoice in description of self invoice
|
Congratulations, PR rebased to 16.0. |
OCA-git-bot
force-pushed
the
16.0-imp-l10n_it_vat_registries
branch
from
e8d0cec
to
0179d60
Compare
#4205