Hotfix - APP + API - Corrección paneles SQL y autorelaciones #151
Conversation
# Conflicts: # eda/eda_app/src/app/shared/components/sidebar/sidebar.component.ts
Control sobre los grupos de seguridad para que no fallen si ya no existe una tabla en el modelo.
este error daba un bug en la modificación / eliminación de permisos
# Conflicts: # .gitignore # eda/eda_api/lib/module/dashboard/dashboard.controller.ts # eda/eda_app/src/app/module/components/eda-panels/eda-blank-panel/panel-utils/query-utils.ts # eda/eda_app/src/app/module/components/eda-table/eda-table.component.html # eda/eda_app/src/locale/messages.ca.xlf # eda/eda_app/src/locale/messages.en.xlf 1. Afegit filtres numerics. 2. Afegit control sobre no data. 3. Revisión de la ordenació per defecte 4. Anadido display
juanSTIC
changed the title
|
Ya he quitado el duplicado de las relaciones para las relaciones. El pantallazo anterior ahora es correcto:
que responde a:
|
There was a problem hiding this comment.
Error en el uso de las autorelaciones
Aunque las autorelaciones aparecen correctamente representadas en el árbol, cuando hay más de una autorelación (lo cual es una situación frecuente) solamente es posible usar una de ellas, ya que, aunque hagamos click en la otra, solamente se utiliza una. Se puede comprobar como en la imagen, aunque se haga click en subevento1, siempre se muestran los datos de subevento2
There was a problem hiding this comment.
Hemos visto que da error siempre que comencemos el árbol en un módulo diferente del que contiene la autorelación.
En este caso se construye incorrectamente la SQL, usando un nombre de columna inexistente:
SELECT `sda_stic_registrations`.`name` as `Nombre`, `sda_stic_events`.`name` as `Nombre_1`, `sda_stic_registrations.stic_registrations_stic_eventsstic_events_ida`.`name` as `Nombre_2`
FROM sda_stic_registrations
inner JOIN `sda_stic_events` ON `sda_stic_registrations`.`stic_registrations_stic_eventsstic_events_ida` = `sda_stic_events`.`id`
inner JOIN `sda_stic_events` `sda_stic_eventsstic_events_id_c` ON `sda_stic_events`.`stic_events_id_c` = `sda_stic_eventsstic_events_id_c`.`id`
group by `sda_stic_registrations`.`name`, `sda_stic_events`.`name`, `sda_stic_registrations.stic_registrations_stic_eventsstic_events_ida`.`name`
#1054 - Unknown column 'sda_stic_registrations.stic_registrations_stic_eventsstic_events_ida.name' in 'field list'
|
Con los últimos cambios tenemos un entorno "estable". Para poder explorar las auto-relaciones necesitaremos un poco más de esfuerzo |
There was a problem hiding this comment.
Las autorelaciones parecen funcionar correctamente según lo esperado, sin embargo al guardar el informe y recargar solamente se guardan los campos del módulo principal. Esto es lo mismo que ya ocurrió en las primera validaciones del PR del árbol y que se reporto aquí #140 (comment)
Es similar a lo ya reportado por @PaulaaSTIC aqui #151 (review)
Cambio en la app al comprobar el tipo de panel para los paneles SQL.
Issue(s) resuelto(s)
Pruebas a realizar para validar el cambio
Este PR requiere de regenerar el modelo de datos por lo que para probarlo hay que:
Para probar el issue 148 se puede hacer un infomre con un panel SQL y un gráfico de barras. Al recargar el informe sigue funcionando correctamente.
Para probar el issue 150 se puede hacer un nuevo informe que incluya alguno de los campos auto-relacionados. sda_contacts, sda_users, sda_stic_events.
Este PR supone alguna re-escritura de las relaciones del arbol. Por lo que requiere que se hagan algunas pruebas "generales" del arbol.
Información Adicional
El error se debia al hacer la comprobación para mantener la compatibilidad con versiones antiguas.
En cuanto a las auto-relaciones. Contempla un nivel de profundidad. No hay más recursividad.