[15][BUG] l10n_es_pos - Error en la validación del límite de factura simplificada

[fcvalgar]

Buenas tardes,

Hemos detectado algunos pequeños detalles que pueden ser un problema para el usuario final en el POS de V15 con el módulo l10n_es_pos.

Os lo explico en este video: https://www.loom.com/share/e84b5948b2c940b09adf1fdf4b18fdb2

Detectamos:

1. El mensaje de error al intentar realizar una operación sin cliente e imprimir una simplificada por encima del límite configurado es un poco confuso.
2. Se sigue permitiendo la impresiónde un ticket aunque legalmente no es posible.
3. Al no ser posible se ha optado por dar por perdido el pedido.

Proponemos como solución la planteada por @danielduqma en V16 con:

1. Validación mediante el botón de validar que obligue por exceso de límite fijado a selecciona un cliente y bloquea el botón de factura.

@manuelregidor @HaraldPanten ¿Qué os parece?

Gracias.

[HaraldPanten]

Hola Fran,

Gracias por la aportación. Me parece interesante lo que comentas. Efectivamente; tal y como lo planteas el comportamiento tiene lagunas...

Podrías referenciar el hilo donde se está hablando del planteamiento para V16?

Me tomo la libertad de etiquetar a @chienandalu , quien tiene bastante experiencia en este aspecto y seguramente pueda aportar en este caso.

Saludos.

[fcvalgar]

Harald, no veo ningún Issue con modificación del planteamiento.

Probe el PR #2615

Y puede que la mejora sea este FIX 8b6615c

Igual @danielduqma nos lo puede aclarar. El proceso de validación mejora notablemente.

Saludos.

[chienandalu]

Hola, Fran. No tenemos aún ningún punto de venta en v15, pero si ese el fix (tiene sentido) proponedlo en PR y de mil amores lo reviso si me haces ping 🙂

[danielduqma]

Hola a todos,

Empezamos a trabajar en la migración a 16.0 y, posteriormente, se mergeó en la 14.0 la asignación de número de factura simplificada en caso de pérdida de conexión en 1baf50b, a raíz de #2812 (secuencias por dispositivo). Cuando integré estos cambios a nuestra migración a la 16.0 no entendí bien el cambio del método validateOrder a _finalizeValidation. De hecho hicimos pruebas en 16.0 con ambos métodos y constatamos el error que reportas @fcvalgar, pero no validamos en 15.0 por lo que no lo pudimos confirmar y reportar para esta versión. ¿@chienandalu nos puedes comentar por favor qué motivaba ese cambio? Si no abrimos PR con el commit que comenta @fcvalgar aplicado en 15.0, ¿en 14.0 no se da el error?

El mensaje de error es del core y sigue así en 16.0

Muchas gracias a todos, un saludo.

[HaraldPanten]

Tiene sentido. Si en V14 funciona correctamente, en V15 no, pero en V16 está bien planteado... Vendrá de ahí. Esperamos los comentarios de David, a ver si puede arrojar algo más de luz.

[fcvalgar]

Tiene sentido. Si en V14 funciona correctamente, en V15 no, pero en V16 está bien planteado... Vendrá de ahí. Esperamos los comentarios de David, a ver si puede arrojar algo más de luz.

En V14 tiene el mismo sistema de validación que en V15:

• El mensaje de error al intentar realizar una operación sin cliente e imprimir una simplificada por encima del límite configurado es un poco confuso.
• Se sigue permitiendo la impresiónde un ticket aunque legalmente no es posible.
• Al no ser posible se ha optado por dar por perdido el pedido.

[fcvalgar]

Hola a todos,

Empezamos a trabajar en la migración a 16.0 y, posteriormente, se mergeó en la 14.0 la asignación de número de factura simplificada en caso de pérdida de conexión en 1baf50b, a raíz de #2812 (secuencias por dispositivo). Cuando integré estos cambios a nuestra migración a la 16.0 no entendí bien el cambio del método validateOrder a _finalizeValidation. De hecho hicimos pruebas en 16.0 con ambos métodos y constatamos el error que reportas @fcvalgar, pero no validamos en 15.0 por lo que no lo pudimos confirmar y reportar para esta versión. ¿@chienandalu nos puedes comentar por favor qué motivaba ese cambio? Si no abrimos PR con el commit que comenta @fcvalgar aplicado en 15.0, ¿en 14.0 no se da el error?

El mensaje de error es del core y sigue así en 16.0

Muchas gracias a todos, un saludo.

El mensaje de validación es completamente diferente porque el proceso es difetente. En el caso de 14/15 es un error relacionado con el límite asignado a la simplificada y en 16 al modificar el sistema de validación el sistema lanza un mensaje sobre la asignación al cliente.

@danielduqma si pudieras solucionar los errores del PR #2615 para poder ser mergeado, mi compañero @Shide haría el backport.

Muchas gracias.

[chienandalu]

He podido sacar un poco de tiempo para revisarlo y tenéis razón. Ahora mismo no sabría decir qué justificación tenía aquel cambio pero lo he revertido en #3026

Gracias a todos 😄 👍

[fcvalgar]

He podido sacar un poco de tiempo para revisarlo y tenéis razón. Ahora mismo no sabría decir qué justificación tenía aquel cambio pero lo he revertido en #3026

Gracias a todos smile +1

Gracias a ti @chienandalu

[yajo]

Fix para v15: #3029. ¿Podéis revisarlo porfa?

[HaraldPanten]

Pues con esto podríamos cerrar el asunto, correcto @fcvalgar ?

Saludos.

[fcvalgar]

Pues con esto podríamos cerrar el asunto, correcto @fcvalgar ?

Saludos.

Si @HaraldPanten , gracias.