Hotfix - Importación - Evitar uso de variable no definida #470
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
AlbertoSTIC
requested review from
jordiSTIC
and removed request for
AlbertoSTIC
|
Actions executed at: 2024-12-19 08:19:59. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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