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

#2475: evitar guardar patrones que generen códigos de más de 20 caracteres. #1486

Closed
wants to merge 2 commits into from

Conversation

castoragra
Copy link
Contributor

Descripción

  • Comprueba al guardar la secuencia que un documento con ese patrón, longitud de número, ejercicio y fecha no generen un código de más de 20 caracteres.

¿Cómo has probado los cambios?

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

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

@NeoRazorX
Copy link
Owner

Se ha mezclado código con el filtro del grupo de clientes. Por eso es necesario hacer una rama distinta cada vez que se quiere enviar algo:

@elguitarraverde
Copy link
Contributor

Dejar la función var_dump en el código puede generar salidas no deseadas o parar el funcionamiento del script.

El hook precommit de git te puede ayudar a evitar esta situación.

https://www.google.com/search?q=var_dump+precommit

@castoragra castoragra closed this Dec 23, 2023
@castoragra castoragra deleted the #2475 branch December 23, 2023 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants