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

limitar caracteres en campos al guardar en base de datos #1637

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

elguitarraverde
Copy link
Contributor

Descripción

Antes se minimizaban los errores de introducir mas caracteres de los que permite el campo en la base de datos con los inputs de formularios.
Cada vez mas se usa la API y plugins que obtienes los datos de API´s y ya no existe limite al guardar en la base de datos, lo que hace que se generen errores al insertar los datos por exceso de caracteres.
Ahora si el campo es varchar se obtiene la longitud del campo y se limita el string a esta longitud evitando así errores.

¿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.

@daniel89fg
Copy link
Contributor

Este pr habría que adaptarlo a la nueva clase model

@elguitarraverde
Copy link
Contributor Author

ok. lo dejamos en standby hasta que este mas avanzado la nueva clase model

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.

2 participants