[11.0][FIX] l10n_es_facturae: The bank code should be obtained from the mandate in direct debits

[eantones]

The bank code is being acquired from the field partner_bank_id of the invoice but this field can be null.

The bank code must be acquired from the same field partner_bank_id but from the mandate mandate_id just as the other bank data as IBAN and BIC.

Can you review it please?

cc @etobella @pedrobaeza

[etobella]

No sé si debería serinvoice.partner_bank_id.bank_id.code or invoice.mandate_id.partner_bank_id.bank_id.code.
Aunque personalmente creo que debe ser exactamente lo mismo, ya que el mandato generalmente es para una única cuenta.
:LGTM: Por mi parte creo que es mejor dejarlo así.

[etobella]

Por cierto, @eantones en este repositorio de la OCA se puede escribir en español, ya que es una localización 😉

[eantones]

Muchas gracias @etobella

No creo que sea lo mismo ya que en una factura con método de pago "Adeudo directo" el campo invoice.partner_bank_id no tiene porqué estar informado.

Saludos :-)

[pedrobaeza]

Cuando se cambia el mandato, se cambia partner_bank_id en la factura, así que yo tampoco veo qué aporta este cambio y en qué caso está desincronizado.

[etobella]

Realmente no cambia nada, así que por mi parte lo podemos dejar así

[eantones]

Lo que aporta es que no devuelva un error al crear la factura electrónica cuando el campo invoice.partner_bank_id no esta informado que es el caso de los "Adeudos directos".

Adicionalmente, por coherencia, si todos los campos del banco se cogen del partner_bank_id del mandato, porque coger uno del partner_bank_id de la factura:

l10n-spain/l10n_es_facturae/views/report_facturae.xml

Lines 337 to 340 in ee1736a

	<IBAN t-minlength="5" t-length="34" t-esc="''.join(invoice.mandate_id.partner_bank_id.acc_number.split())"/>
	<BankCode t-length="60" t-esc="invoice.partner_bank_id.bank_id.code"/>
	<BranchCode t-length="60" t-if="False"/>
	<BIC t-minlength="11" t-length="11" t-esc="invoice.mandate_id.partner_bank_id.bank_id.bic"/>

También, en la validación previa, únicamente valida el partner_bank_iddel mandato y no el de la factura:

l10n-spain/l10n_es_facturae/models/account_invoice.py

Lines 214 to 215 in e6ce6f2

	if not self.mandate_id.partner_bank_id:
	raise ValidationError(_('Partner bank in mandate is missing'))

[github-actions]

There hasn't been any activity on this pull request in the past 4 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 PR to never become stale, please ask a PSC member to apply the "no stale" label.