Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hotfix - General - Añadir relación entre actividades y personas desde subpanel #447

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

PaulaaSTIC
Copy link
Collaborator

@PaulaaSTIC PaulaaSTIC commented Oct 21, 2024

Descripción

Los cambios realizados en el PR https://github.com/SinergiaTIC/SinergiaCRM-SuiteCRM/pull/849 provocan el issue #444

Solución propuesta

Eliminar una de las condiciones incluidas en el PR sobre la función

|| (empty($this->{$new_rel_link}) && !empty($this->parent_id) && $new_rel_id !=$this->parent_id)
.
Esta condición provoca que, si el valor del registro relacionado indicado en el formulario no es igual al ID del registro padre (es decir, el registro desde el cual se mostraba el subpanel y se abrió el formulario de edición/creación), no se cree la relación entre el registro creado en el subpanel y el registro padre.

Pruebas

  1. Ir al módulo de personas.
  2. Abrir el subpanel de Actividades de cualquier persona.
  3. Crear una nueva reunión.
  4. En el campo "Relacionado con", seleccionar un registro de otro módulo.
  5. Guardar la reunión.
  6. Verificar que la reunión queda asociada a la persona en la que se ha creado.

Probar que se mantienen las comprobaciones realizadas en el PR849

  1. Crear dos personas.
  2. Ir a una de ellas y crear una relación con persona desde el subpanel.
  3. Desde el subpanel, editar la relación con persona y cambiar varios campos, incluida la persona.
  4. Comprobar que se han cambiado la relación.
  5. Realizar pruebas con subpaneles N-M.

Copy link

github-actions bot commented Oct 21, 2024

Actions executed at: 2024-10-23 05:14:58.

@PaulaaSTIC PaulaaSTIC marked this pull request as ready for review October 21, 2024 15:37
@PaulaaSTIC PaulaaSTIC changed the title Fix the relationship contacts/activities from subpanel Hotfix - General - AÑadir relación entre actividades y personas desde subpanel Oct 21, 2024
@PaulaaSTIC PaulaaSTIC changed the title Hotfix - General - AÑadir relación entre actividades y personas desde subpanel Hotfix - General - Añadir relación entre actividades y personas desde subpanel Oct 21, 2024
Copy link
Collaborator

@enricsinergia enricsinergia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(A)probado

ManuSinergiaCRM

This comment was marked as outdated.

Copy link
Collaborator

@ManuSinergiaCRM ManuSinergiaCRM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

La línea de código que se está comentando solucionaba la incidencia para relaciones especiales como member_accounts donde parent_id no es igual al ID del registro principal y ahora vuelve a ocurrir el error.

Se propone investigar y averiguar si se puede añadir alguna condición más que permita solucionar ambas incidencias.

Si no es sencillo, y por lo que recuerdo no lo será ya que la función set_relationship_info() tenía complejidad, una opción alternativa y sencilla podría ser quitar la opción de Editar en los subpaneles de aquellas relaciones especiales como member_accounts

Paula, si lo consideras, podemos comentarlo :)

Como reproducir la incidencia

  1. Crear dos organizaciones.
  2. Ir a una de ellas y seleccionar a la otra en el subpanel de Organizaciones miembro
  3. Desde el mismo subpanel, editar el registro y cambiar varios campos, incluido el campo Miembro de
  4. Comprobar que en la organización se han actualizado los campos modificados excepto el de Miembro de

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants