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

Incidencia - Validación de importación - Eliminar validación no necesaria #510

Closed
ManuSinergiaCRM opened this issue Dec 9, 2024 · 0 comments · Fixed by #511
Closed
Assignees
Labels
bug Something isn't working Importación

Comments

@ManuSinergiaCRM
Copy link
Collaborator

ManuSinergiaCRM commented Dec 9, 2024

Descripción del problema
Se detecta que una de las validaciones que se realiza en el proceso de validación de importación (https://github.com/SinergiaTIC/SinergiaCRM/blob/develop/modules/stic_Import_Validation/ImporterValidationFunctions.php#L54-L57) puede ser omitida ya que el CRM dispone de una LH en el módulo de Personas
(https://github.com/SinergiaTIC/SinergiaCRM/blob/develop/custom/modules/Contacts/SticLogicHooksCode.php#L58-L75) que calcula el valor del campo Tipo de identificación en caso de que este no venga indicado y sí lo esté el número de identificación.

En consecuencia, al no ser necesaria dicha validación, se propone comentar la lógica que informaba del error e indicar una breve descripción y la URL al PR para que quede documentado por que se tomo esta decisión.

Cómo reproducir el problema

  1. Crear una persona con Tipo de identificación y Número de identificación
  2. Exportarlo y eliminar todas las columnas menos las de Apellido, Tipo de identificación y Número de identificación
  3. Eliminar el valor del campo Tipo de identificación
  4. Realizar la validación del fichero de importación y comprobar que el proceso informa a la entidad, a través del fichero enriquecido que genera el proceso, de que hay un error con el campo Tipo de identificación y que se debería indicar un valor
  5. Importar el fichero y comprobar que el CRM, durante la importación, ha indicado el campo Tipo de identificación aunque en el fichero de importación no estuviera indicado.

Solución propuesta
Comentar la lógica que informaba del error e indicar una breve descripción y la URL al PR para que quede documentado por que se tomo esta decisión.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Importación
Projects
None yet
2 participants