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

52 or more characters filenames support (Soporte de nombre de archivos de mas de 52 caracteres) #17

Open
jmunoz298 opened this issue Nov 28, 2013 · 3 comments

Comments

@jmunoz298
Copy link

Issue creado por Alen en Git-Tryolabs

1234567890123456789012345678901234567890123456789012.docx SI
12345678901234567890123456789012345678901234567890123.docx NO
Da el error:
Warning at /project/7/document/add/
Data truncated for column 'file' at row 1
Request Method: POST
Request URL: http://demo.libreqda.edu.uy/project/7/document/add/
Django Version: 1.4.3
Exception Type: Warning
Exception Value:

Data truncated for column 'file' at row 1
Exception Location: /usr/local/lib/python2.7/dist-packages/MySQLdb/cursors.py in warningcheck, line 117
Python Executable: /usr/bin/python
Python Version: 2.7.3
Python Path:

['/usr/share/libreqda',
'/usr/local/lib/python2.7/site-packages',
'/usr/share/libreqda/src/docx',
'/usr/lib/python2.7',
'/usr/lib/python2.7/plat-linux2',
'/usr/lib/python2.7/lib-tk',
'/usr/lib/python2.7/lib-old',
'/usr/lib/python2.7/lib-dynload',
'/usr/local/lib/python2.7/dist-packages',
'/usr/local/lib/python2.7/dist-packages/PIL',
'/usr/lib/python2.7/dist-packages']
Server time: mar, 2 Abr 2013 00:39:53 -0300

@jmunoz298
Copy link
Author

AMARTINEZUY commented:

Es correcto, la restricción actual es la siguiente:

Largo máximo de nombre de archivo = 100 - LARGO(ruta a carpeta donde se guardan los archivos)

Es posible aumentar el largo máximo (el actual es el que Django pone por defecto). Es importante entender que este cambio implica modificar los modelos de la aplicación. Para eso deberíamos agregar South a los requerimientos del proyecto.

@jmunoz298
Copy link
Author

Considero que si hay que tocar el modelo, podríamos asumir esta restricción

@marcbria
Copy link

Estoy con Juan.
En cualquier caso, aunque el funcionamiento sea el mismo, un mensaje como "El nombre del fichero es demasiado largo" da mucha más tranquilidad al usuario final.

¿Podemos cambiar el issue por "Derivar los errores estrepitosos a un log y informar al usuario del problema detectado"?

La idea que propongo NO es pedir a los desarrolladores que implementen una mejor gestión de errores, sino que entre todos reportemos aquellos errores habituales (que se informan como si fuese el fin del mundo) y se cambien por mensajes comprensibles.

Juan ¿este "reportar errores habituales con mensajes comprensibles" te parece algo que deba entrar en la 1.0?
Si la respuesta es si, deberíamos hacer una lista de errores a filtrar (ahora me miro el resto de bugs a ver si ya lo contemplan).

@lupa18 lupa18 changed the title No soporta nombres de Documentos de mas de 52 caracteres 52 or more characters filenames support (Soporte de nombre de archivos de mas de 52 caracteres) May 27, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants