Hotfix - Emails - Mostrar notas adjuntas en el subpanel de Datos Adjuntos del email

[ManuSinergiaCRM]

• Closes Incidencia - Emails - No se muestran los datos adjuntos en el subpanel de Datos Adjuntos #35

Descripción

Pendiente de valorar la solución propuesta

Además de lo indicado en el issue, se detecta que tampoco se crea la relación entre el email y la nota que genera el CRM cuando se adjunta un registro del módulo de Documentos al correo electrónico.

También se observa que:

• Existe una relación N-M entre Emails y Notas (en Estudio: emails_notes_rel) que es la que genera el subpanel de Datos Adjuntos y que no se usa para relacionar la nota con el Email. Esta relación la modificamos en este PR https://github.com/SinergiaTIC/SinergiaCRM-SuiteCRM/pull/1081

• Para relacionar la nota con el email se utiliza un campo (parent_type) de tipo Posible relacionado con .

Durante la implementación del PR se intentó crear el registro vía beans en el mismo punto donde ahora está el INSERT:

$note->load_relationship('notes');
$note->notes->add($this->id);

pero daba error al encontrar el lado derecho de la relación.

Se optó por no investigar dicho error e implementarlo vía SQL ya que si creábamos los registros en la tabla de BBDD de la relación aparecían las notas en el subpanel de Datos Adjuntos.

Pruebas

1. Acceder al módulo de Emails y enviar un email con uno o más ficheros y documentos adjuntos
2. Acceder al email enviado y comprobar que se muestran todas las notas generadas

[enricsinergia]

He probado en SuiteCRM y parece que ahí sí crea la relación con la nota automáticamente, por lo que quizás, el cambio que introduje para corregir el problema del envío de e-mails sobre notas de Organizaciones introdujo este problema.
Quizás habría que replantear la solución a uno y otro PR para garantizar que ambas funcionalidades funcionan correctamente.

[ManuSinergiaCRM]

Se confirma que deshaciendo el cambio generado en el PR salesagility#1081 se guardan las notas en el subpanel de Datos Adjuntos pero, al mismo tiempo, volvemos a tener un campo posiblemente relacionado que será sobreescrito por la operativa de adjuntar ficheros a un email en caso de que se relacione el email con la nota.

Se ha intentado:

1. Cambiar la label del campo Posiblemente Relacionado con a Correo electrónico en vez de Relacionado con
2. Por JS, esconder el input donde seleccionar el módulo (parent_name)
3. Crear otro campo Posiblemente relacionado con para mantener el comportamiento actual y poder relacionar la nota con cualquier módulo.

Pero este último paso no es posible realizarlo ya que desde la interfaz no se puede añadir más de un campo de este tipo por módulo, y al intentar implementarlo mediante código, no se consigue ya que hay código del core que trabajará con este tipo de campo que sigue relacionando al campo nuevo con el campo ya existente. En vez de utilizar parent_type_1 sigue utilizando parent_type. Se han realizado cambios, mas allá de os lógicos, en diferentes propiedades de los 3 campos (parent_name_1, parent_type_1, parent_id_1) que se necesitan para crear este tipo de campo y aún así se sigue dando la relación entre el campo nuevo y el existente.

En consecuencia, se decide no seguir investigando esta nueva propuesta y volver a la primera solución que teníamos. Es decir, no revertir los cambios aplicados en el PR salesagility#1081 y crear el registro de relación entre el email y sus posibles ficheros y documentos del CRM relacionados a través de la base de datos.

[enricsinergia]

(A)probado
Comprobado que funciona correctamente.
Se acepta la decisión de no seguir urgando en la definición de las relaciones, ya que parece fruto de la evolución histórica de las relaciones de base.

[ainaraRT]

(A)Probado