search

SinergiaTIC / SinergiaDA

SinergiaDA is an open source analytics tool that fully integrates with SinergiaCRM
GNU Affero General Public License v3.0
4 stars 0 forks source link

Hotfix - APP - Restaurado de filtro no nulo en paneles #165

Closed jortilles closed 2 months ago

jortilles commented 3 months ago

Descripción del Cambio

Restaurado del filtro no nulo en los paneles

Issue(s) resuelto(s)

Pruebas a realizar para validar el cambio

Realizar un informe con un panel y hacer un filtro de no nulo ni vacio.

jortilles commented 3 months ago

Hemos separdo los issues. Este es solo reltativo al filtro no nulo. Si usas una auto-relación en los filtros de la consulta falla. Ese es el otro issue que estamos resolviendo.

jortilles commented 3 months ago

Los campos de tipo numérico con alguna agregación dan error al intentar filtrar valores no nulos. Este error no se produce en reporting.

num_no_nulos num_no_nulos

MmMm... claro con los números no tiene sentido. Tenemos 3 opciones:

  1. Los campos numéricos y de fecha no tienen las opciones "no nulo ni vacío"
  2. Los campos numéricos y de fecha se hace un cast a string ( esto afecta al rendimiento )
  3. Los campos numéricos y de fecha... si llega un "no nulo ni vacío" se convierte a "no nulo" puesto que no tienen cadenas vacías '' Yo opto por la opción 3. Creo que es la mejor. Por favor. Confirmadme la decisión e implementamos eso. @PaulaaSTIC @juanSTIC
PaulaaSTIC commented 3 months ago

MmMm... claro con los números no tiene sentido. Tenemos 3 opciones:

  1. Los campos numéricos y de fecha no tienen las opciones "no nulo ni vacío"
  2. Los campos numéricos y de fecha se hace un cast a string ( esto afecta al rendimiento )
  3. Los campos numéricos y de fecha... si llega un "no nulo ni vacío" se convierte a "no nulo" puesto que no tienen cadenas vacías '' Yo opto por la opción 3. Creo que es la mejor. Por favor. Confirmadme la decisión e implementamos eso. @PaulaaSTIC @juanSTIC

Los campos de texto, una vez tienen una agregación (cuenta o cuenta valores únicos), son tratados como numéricos. ¿Estos sí tendrán vacíos y nulos? ¿O se tratan todos como nulos? Porque si vamos a coger la opción 3, debemos tenerlo en cuenta.

Por otro lado, he visto que los filtros de no nulos no funcionan en las fechas; no se inyecta la condición en la query.

jortilles commented 2 months ago

Hola. He revisado los errores detectados ayer y creo que ahora ya está todo bien. Podéis volver a revisar? Gracias.