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

Corregido comprobación campo totales #1683

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Yopli2k
Copy link
Contributor

@Yopli2k Yopli2k commented Oct 24, 2024

Descripción

El modelo JoinModel, a la hora de calcular los totales de final de página y listado comprobaba si el campo a totalizar tenía o contenía un paréntesis. Esto no es del todo correcto puesto que el campo a totalizar puede tener paréntesis por funciones SQL no estadísticas o incluso por operaciones matemáticas que requieran de paréntesis.

Ahora se emplea un nuevo método donde se comprueba que el campo a totalizar no contenga una función estadística, en cuyo caso no se realiza la suma o total.

¿Cómo has probado los cambios?

Toda modificación debe haber sido mínimamente probada. Marca o describe las pruebas que has realizado:

  • He revisado mi código antes de enviarlo.
  • He probado que funciona correctamente en mi PC.
  • He probado que funciona correctamente con una base de datos vacía.
  • He ejecutado los tests unitarios.

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

Successfully merging this pull request may close these issues.

1 participant