Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hotfix - Vistas Personalizadas - Verificación correcta de condiciones numéricas #423

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

jordiSTIC
Copy link
Collaborator

Descripción

Tal y como se comenta en #422, las condiciones no se validan correctamente en ciertos campos. Concretamente, se observa que no se validaban cuando:

  1. El campo tenía asociada una lista mediante la declaración options en el vardefs y su valor no dependía de ella (numérico por ejemplo)
  2. El campo a comparar es numérico

Este comportamiento erróneo se ha observado con el campo "Edad" del módulo "Personas", que en su definición:

  • 'options' => 'numeric_range_search_dom',
  • 'type' => 'int'

Los cambios en el código incluyen:

  1. No usar la lista de posibles valores si el campo no es de tipo enum, multienum o dynamicenum
  2. Convertir los valores a comparar a numérico (si corresponde) cuando se verifique una condición de una VP

Pruebas

  1. Crear una Vista Personalizada para el Módulo Personas - Vista de detalle
  2. Añadir la condición: Edad - Menor que - Valor - 18
  3. Añadir la acción: Campo - Edad - Color de fondo - Rojo
  4. Crear dos Personas y asignar fecha de nacimiento anterior a 18 años y posterior a 18 años
  5. Verificar que el comportamiento de la VP es correcto

Copy link

github-actions bot commented Oct 4, 2024

Actions executed at: 2024-10-04 07:30:03.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants