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 - Flujos de trabajo - Tratamiento del is_null en condiciones sobre camposnuméricos #2

Closed
wants to merge 1 commit into from

Conversation

enricsinergia
Copy link
Collaborator

@enricsinergia enricsinergia commented Dec 29, 2023

Descripción

A la hora de evaluar el valor nulo en lasd condiciones de los flujos de trabajo, se ha visto que los campos de tipo texto o fecha se recuperan como "", sin embargo los campos numéricos se estaban seteando a NULL. A la hora de comprobar si un campo era NULL, la condición miraba si el campo estaba como "". Se ha cambiado esta condición para que si el tipo es numérico y tiene valor NULL se altere el valor previo a la condición a "", sin afectar el valor en el resto del código (sólo para la evaluación de la condición), de manera que no haya efectos colaterales.

Pruebas

  1. Crear flujo de trabajo, añadiendo una condición donde se requiera que un importe sea nulo
  2. Comprobar que tanto en ejecucion "on save" como en el planificador, se ejecuta el flujo cuando y sólo cuando el campo numérico es NULL.

@enricsinergia enricsinergia self-assigned this Dec 29, 2023
@enricsinergia enricsinergia changed the title Null for decimals patched Hotfix - Flujos de trabajo - Tratamiento del is_null en condiciones sobre camposnuméricos Dec 29, 2023
@enricsinergia enricsinergia deleted the bugs/WorkflowsDecimalNull branch January 2, 2024 10:54
Copy link

github-actions bot commented Jan 2, 2024

Actions executed at: 2024-01-02 10:54:52.

PaulaaSTIC pushed a commit that referenced this pull request Feb 26, 2024
# This is the 1st commit message:

Fix salesagility#10364 - Adding now option in Datetime fields

# This is the commit message #2:

Fix salesagility#10364 - Adding now option in Datetime fields

# This is the commit message #3:

Fix salesagility#10364 - Adding now option in Datetime fields

# This is the commit message #4:

Fix salesagility#10364 - Adding now option in Datetime fields

# This is the commit message #5:

Fix salesagility#10364 - Adding now option in Datetime fields

# This is the commit message #6:

Fix salesagility#10364 - Adding now option in Datetime fields

# This is the commit message #7:

Fix salesagility#10364 - Adding now option in Datetime fields

# This is the commit message #8:

Fix salesagility#10364 - Adding now option in Datetime fields

# This is the commit message #9:

Fix salesagility#10364 - Adding now option in Datetime fields
PaulaaSTIC added a commit that referenced this pull request Apr 12, 2024
# This is the 1st commit message:

Sidebar changes

# This is the commit message #2:

Remove setting

# This is the commit message #3:

Fix setting sidebar

# This is the commit message #4:

Fix sidebar dark mode
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incidencia - Flujos de trabajo - Error al evaluar condición en tipo decimal
1 participant