Hotfix - Formularios - Eliminar mensajes de error innecesarios #448
Conversation
|
Actions executed at: 2024-11-19 15:37:37. |
AlbertoSTIC
requested review from
ManuSinergiaCRM and
jordiSTIC
and removed request for
AlbertoSTIC
There was a problem hiding this comment.
Comprobado que los cambios aplicados evitan que se muestre como error que un formulario no tenga plantilla de email asociada.
Leyendo el issue me surge la duda de qué se ha visto o valorado para, aparéntemente, no abordar los puntos:
-
1.- 2024-09-19 15:56:05 [338463][1][ERROR] Line 281: EventInscriptionBO::checkParams: The person ID [12345678Z ] is not valid. ==> Parece que en la validación del NIF falta un trim y que ese espacio al final de línea provoca un error
-
3.- 2024-09-19 15:56:01 [338467][1][ERROR] Given parameter should contains index "copy_email_addresses"
2024-09-19 15:56:01 [338467][1][ERROR] Email address copy error occured, bean was: stic_Registrations ==> Es un error que se produce dentro de actionCreateRecord de AOW_Actions
|
@ManuSinergiaCRM , sobre
Es un error que se produce dentro de actionCreateRecord de AOW_Actions Sobre el NIF, se descartó porqué el NIF ya debería venir bien del front. Igualmente lo añadimos |
Descripción
Derivado de las trazas detectadas y recogidas en #393 se decide intentar eliminar los mensajes de error que aprecen por el simple hecho de no indicar plantilla en elparámetro defParams de los formualrios.
Se protegen un par de puntos para comprobar si hay o no templateId informado y se mueven algunos mensajes de error a warn, ya que se trata de mensajes redundantes derivados de un error previo que queda reflejado en el log.
Motivation and Context
Se quiere reducir el número de trazas innecesarias que van al log para reducir el log y mejorar también la velocidad de respuesta (aunque sea de manera infinitesimal)
How To Test This
1.- Crear formularios de donación y de eventos sin idicar plantilla de comunicación a usuario
2.- Comprobar en las trazas que no aparecen las líneas
2024-10-21 09:01:47 [988190][1][ERROR] Line 316: WebFormMailer::parseEmailTemplateById: No ID received. 2024-10-21 09:01:47 [988190][1][ERROR] Line 224: EventInscriptionMailer::__sendUserMail: Error parsing the template. 2024-10-21 09:01:47 [988190][1][ERROR] Line 203: EventInscriptionController::sendMails: Unable to send the email to the user. 2024-10-21 09:02:00 [990626][1][FATAL] Attachements not found3.- Crear formularios de eventos y donación, tanto para Personas como Organización
4.- Modificar en los formularios las rutinas de validacion isValidDNI y isValidCif para que siempre devuelvan true
5.- Entrar en los formularios DNIS y CIFs válidos pero con espacios delante y/o detrás
6.- Comrpobar que no aparece traza de error y que el DNI/CIF queda correctamente (sin espacios) registrado en el CRM
2024-09-19 15:56:05 [338463][1][ERROR] Line 281: EventInscriptionBO::checkParams: The person ID [12345678Z ] is not valid.