l10n_es_payment_order_confirming_aef: Bankinter no toma una fecha de emisión correcta

[giarve]

Module

l10n_es_payment_order_confirming_aef

Describe the bug

Bankinter usa la fecha de proceso del fichero AEF que actualmente es incorrecta (tomamos el primer pago y su fecha). Los demás bancos usan la fecha de subida del fichero a sus plataformas. Por ende, es probable que la mejor solución sea poner el campo de fecha de generación del fichero como "fecha proceso".

Si se pone una fecha de vencimiento del primer pago (criterio para hacer esto? Ninguno), nunca se habilita la remesa en el sistema para que los proveedores lo decidan usar o no.

l10n-spain/l10n_es_payment_order_confirming_aef/models/confirming_aef.py

Lines 114 to 116 in 0feaf74

	# 67 - 74 Fecha proceso
	fecha_proceso = fields.first(self.record.payment_line_ids).date
	text += self._aef_convert_text(str(fecha_proceso).replace("-", ""), 8)

To Reproduce

Affected versions: 14.0,15.0,16.0

Steps to reproduce the behavior:

1. Crear remesa de pago
2. Generar fichero
3. Subir fichero a Bankinter

Expected behavior
La fecha de generación del fichero se establece aquí:

l10n-spain/l10n_es_payment_order_confirming_aef/models/confirming_aef.py

Lines 114 to 116 in 0feaf74

	# 67 - 74 Fecha proceso
	fecha_proceso = fields.first(self.record.payment_line_ids).date
	text += self._aef_convert_text(str(fecha_proceso).replace("-", ""), 8)

[pedrobaeza]

No termino de entender el problema, pero lo que puedo decir es que la fecha de pago depende de lo que se elija en la orden de pago. Puede ser una fecha fija o puede ser la fecha de hoy. Obviamente, si se escoge fecha de vencimiento, y la primera línea tiene una fecha en el futuro posterior al de otras líneas, esa fecha no será representativa.

De todas, ¿para qué utilizan los bancos ese dato de fecha proceso? Cada línea luego lleva información de su propia fecha:

l10n-spain/l10n_es_payment_order_confirming_aef/models/confirming_aef.py

Line 258 in 0feaf74

text += str(line.date).replace("-", "").replace("/", "")

con lo que es un dato bastante testimonial.

[giarve]

No termino de entender el problema, pero lo que puedo decir es que la fecha de pago depende de lo que se elija en la orden de pago. Puede ser una fecha fija o puede ser la fecha de hoy. Obviamente, si se escoge fecha de vencimiento, y la primera línea tiene una fecha en el futuro posterior al de otras líneas, esa fecha no será representativa.

De todas, ¿para qué utilizan los bancos ese dato de fecha proceso? Cada línea luego lleva información de su propia fecha:

l10n-spain/l10n_es_payment_order_confirming_aef/models/confirming_aef.py

Line 258 in 0feaf74

text += str(line.date).replace("-", "").replace("/", "")

con lo que es un dato bastante testimonial.

Los demás bancos, para nada. Bankinter para procesar el fichero, si no, ni crea los confirming hasta esa fecha, y es como si nunca se hubiera subido el fichero.

Según me han explicado los usuarios, es para dar planificar una disponibilidad del confirming desde fecha previa a la subida real del fichero (los otros bancos no tendrían esta posibilidad).

Solución:
Se pone la fecha de generación del fichero. No afecta a ningún otro banco y soluciona el caso de Bankinter. La verdad es que dudo que lo use alguien como planificador, sinó habrá que crear un campo explícito para especificar esa fecha, pero ya digo que solo pasa con Bankinter y ni necesitamos eso en nuestro caso de uso (sirve con poner la fecha de generación del fichero).

[pedrobaeza]

Vale, como no tiene ninguna función especial, se puede utilizar la fecha que comentas. ¿Puedes hacer un PR con el cambio?