Enhancement - SinergiaDA - October2/November/December #478
Conversation
|
@BelenFFernandez te he citado en este PR porque aunque no es de SinergiaDA directamente establece límites en el uso de SinergiaDA y necesita ser documentado. Lo comentamos. |
AlbertoSTIC
changed the title
juanSTIC
changed the title
There was a problem hiding this comment.
Lo he probado en alberto.sinergiacrm.org y, aunque parece que las vistas se han creado, aparecen multiples mensajes de error de este tipo:
2024-12-16 11:36:31 [2585984][2][ERROR] Line 1917: ExternalReporting::batchInsertPermissions: Invalid record structure: Array
(
[user_name] => pastic
[table] => sda_stic_work_calendar
[column] => users_id
[stic_permission_source] => ACL_ALLOW_ALL
[global] => 1
)
y
2024-12-16 11:36:32 [2585984][2][FATAL] Mysqli_query failed.
2024-12-16 11:36:32 [2585984][2][FATAL] Query Failed: CREATE OR REPLACE VIEW sda_l_campaigns_notification_prospect_list_ids_dynamic_prospect_ AS : MySQL error 1064: You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '' at line 1
2024-12-16 11:36:32 [2585984][2][ERROR] Line 1610: ExternalReporting::createEnumView: Error has occurred: [ Query Failed: CREATE OR REPLACE VIEW sda_l_campaigns_notification_prospect_list_ids_dynamic_prospect_ AS : MySQL error 1064: You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '' at line 1] running Query: [CREATE OR REPLACE VIEW sda_l_campaigns_notification_prospect_list_ids_dynamic_prospect_ AS ]
2024-12-16 11:36:32 [2585984][2][FATAL] Mysqli_query failed.
2024-12-16 11:36:32 [2585984][2][FATAL] Query Failed: CREATE OR REPLACE VIEW sda_l_campaigns_notification_template_id_dynamic_email_template_ AS : MySQL error 1064: You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '' at line 1
2024-12-16 11:36:32 [2585984][2][ERROR] Line 1610: ExternalReporting::createEnumView: Error has occurred: [ Query Failed: CREATE OR REPLACE VIEW sda_l_campaigns_notification_template_id_dynamic_email_template_ AS : MySQL error 1064: You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '' at line 1] running Query: [CREATE OR REPLACE VIEW sda_l_campaigns_notification_template_id_dynamic_email_template_ AS ]
2024-12-16 11:36:32 [2585984][2][FATAL] Mysqli_query failed.
2024-12-16 11:36:32 [2585984][2][FATAL] Query Failed: CREATE OR REPLACE VIEW sda_l_campaigns_notification_outbound_email_id_dynamic_outbound_ AS : MySQL error 1064: You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '' at line 1
2024-12-16 11:36:32 [2585984][2][ERROR] Line 1610: ExternalReporting::createEnumView: Error has occurred: [ Query Failed: CREATE OR REPLACE VIEW sda_l_campaigns_notification_outbound_email_id_dynamic_outbound_ AS : MySQL error 1064: You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '' at line 1] running Query: [CREATE OR REPLACE VIEW sda_l_campaigns_notification_outbound_email_id_dynamic_outbound_ AS ]
2024-12-16 11:36:32 [2585984][2][FATAL] Mysqli_query failed.
2024-12-16 11:36:32 [2585984][2][FATAL] Query Failed: CREATE OR REPLACE VIEW sda_l_campaigns_notification_inbound_email_id_dynamic_inbound_em AS : MySQL error 1064: You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '' at line 1
2024-12-16 11:36:32 [2585984][2][ERROR] Line 1610: ExternalReporting::createEnumView: Error has occurred: [ Query Failed: CREATE OR REPLACE VIEW sda_l_campaigns_notification_inbound_email_id_dynamic_inbound_em AS : MySQL error 1064: You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '' at line 1] running Query: [CREATE OR REPLACE VIEW sda_l_campaigns_notification_inbound_email_id_dynamic_inbound_em AS ]
Para este caso se ha añadido el valor para el campo group (null) en los dos sitios dónde no se estaba haciendo |
Los logs de este tipo se producen porque algún campo referencia a listas desplegables que no existen, lo que no es habitual en producción, pero más común en entornos de desarrollo. Se corrije haciendo un comprobación específica de si existe o no la lista desplegable, en caso de que no exista se muestra un log con más información y se sale de la función sin intentar hacer la query. |
There was a problem hiding this comment.
Revisados los puntos anteriores y son correctos.
En mi instancia se ve este mensaje en errors.log:
[Tue Dec 17 11:06:24.601405 2024] [proxy_fcgi:error] [pid 2155803:tid 2155822] [remote 79.116.233.62:33474] AH01071: Got error 'PHP message: PHP Warning: unlink(sdaRebuildError.txt): No such file or directory in /usr/home/alberto.sinergiacrm.org/web/SticInclude/SinergiaDARebuild.php on line 66', referer: https://alberto.sinergiacrm.org/index.php?module=Administration&action=sticmanagesdaintegration
|
|
Actions executed at: 2025-01-22 05:19:14. |
1. Añadir ColourPicker y collection como tipos de campo excluidos del procesamiento.
Como validarlo
SinergiaCRM/SticInclude/SinergiaDA.php
Lines 389 to 390 in 3fc9c2b
2. Mejoras en la gestión de tabla sda_def_permissions
Se ha visto que en escenarios con un alto número de permisos (usuarios*grupos) el procesamiento es excesivamente lento si sda_def_permissions es una vista y no una tabla, por lo que se revierte este cambio hecho en la anterior versión. Paralelamente se mejora el proceso de guardado de los registros de esta tabla, que hasta el momento se hacía por cada registro, y ahora se procesan en bloque, haciendo
Se ha establecido una limitación práctica de 100 usuarios no administradores + todos los usuarios administradores de la instancia (los usuarios administradores no añadirán muchos registros ya que no tienen limitaciones de acceso a módulos). Este límite puede ser aumentado o disminuido usando la variable
$sugar_config['stic_sinergiada']['max_users_processed']=<num>en config_override.php.Cómo probarlo
sda_def_permissions, que debe incluir los permisos de los usuarios procesados.3. Incluir solo a usuarios activos en la tabla sda_def_users
Ahora solo se incluyen en la tabla sda_def_users a los usuarios activos con SinergiaDA activado, puesto que esta tabla ya solamente se usa para el proceso de login de los usuarios y no es necesario que estén en la misma, como se creía hasta ahora, los usuario inactivos o con el acceso a SDA desactivado.
Cómo probarlo
Verificar que en tras el rebuild, la tabla sda_def_users y sda_def_users_groups unicamente incluyen a los usuarios activo y con SDA habilitado.
4. Eliminar operaciones DROP redundantes
Se han eliminado las operaciones DROP table redundantes existentes, sobre tablas que generaban errores del tipo [FATAL] en los logs. Aunque estos errores eran inocuos en cuanto a sus consecuencias, se ha corregido esta situación anómala.
Cómo probarlo
Verificar que tras reconstruir, no aparecen en el log entradas como las siguientes:
5. Cambio de motor de tablas de SDA a InnoDB
Ahora las tablas sda_* se crean siempre como InnoDB, en consonancia con el cambio realizado en SinergiaCRM
Cómo probarlo
Verificar que tras reconstruir todas las tablas con prefijo sda_ tienen el motor InnoDB
6. Se corrige la causa de algunas entradas de log indebidas.
Cómo probarlo