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 - Importación - Evitar uso de variable no definida #470

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

Conversation

enricsinergia
Copy link
Collaborator

@enricsinergia enricsinergia commented Oct 31, 2024

Description

Se resuelven las 2 trazas explicadas en #451.
En el primer caso se estaba usando $key enla expresión $this->$key sin comprobar que $key estuviera informado, lo que generaba el pertiennte warning.
En el segundo caso se realizaban acciones que sólo debían hacerse en creación del registro. Para ello se comparaban los datos con fetched_row pero sin comprobar si éste estaba informado.
Al corregir los primeros warnings se ha detectado tambiénq ue en Incorpora relacionado con el campo de localización había un problema parecido, que también se ha corregido.

Motivation and Context

Derivado del proceso realizado para eliminar trazas innecesarias del log, se detectan estas 2 situaciones que se producen en la importación.

How To Test This

1.- Realizar una importación de Personas
1.1.- Incluir personas con fecha de nacimiento y sin fecha de nacimiento
1.2.- Incluir personas con stic_postal_mail_return_reason_c informado y vacío
1.3.- Incluir personas con stic_incorpora_locations_id_c informado y vacío
1.4.- Comprobar el log de errores (suitecrm.log)
2.- Cambiar la localización de Incorpora
2.1.- Pasar de nulo a informado
2.2.- Cambiar un valor por otro
2.3.- Pasar de valor informado a nulo
2.4.- Comprobar el log de errores (suitecrm.log)
2.5.- Comprobar que los campos calculados a partir de la localización se actualizan correctamente

@enricsinergia enricsinergia self-assigned this Oct 31, 2024
@enricsinergia enricsinergia marked this pull request as ready for review October 31, 2024 11:31
@AlbertoSTIC AlbertoSTIC requested review from jordiSTIC and removed request for AlbertoSTIC November 18, 2024 09:53
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 not urgent
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incidencia - Importador - Mensajes de error en log
1 participant