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

Erreur 500 lors de l'édition malencontreuse d'un message du forum #2633

Closed
firm1 opened this issue May 5, 2015 · 9 comments
Closed

Erreur 500 lors de l'édition malencontreuse d'un message du forum #2633

firm1 opened this issue May 5, 2015 · 9 comments
Assignees
Labels
C-Back Concerne le back-end Django Facile Bon ticket pour débuter pour rejoindre le développement ! S-BUG Corrige un problème
Milestone

Comments

@firm1
Copy link
Contributor

firm1 commented May 5, 2015

url incriminée : http://zestedesavoir.com/forums/message/editer/?message=plazza/

On tombe sur une erreur 50. La bienséance voudrait qu'on n'ait jamais d'erreur 500 dans la vie.

@firm1 firm1 added S-BUG Corrige un problème Facile Bon ticket pour débuter pour rejoindre le développement ! C-Back Concerne le back-end Django labels May 5, 2015
@Eskimon
Copy link
Contributor

Eskimon commented May 5, 2015

Ca se passe ici: https://github.com/zestedesavoir/zds-site/blob/dev/zds/forum/views.py#L665

Il manque une conversion en int

La même chose pourrait être faite ici d'ailleurs : https://github.com/zestedesavoir/zds-site/blob/dev/zds/forum/views.py#L625

@Eskimon
Copy link
Contributor

Eskimon commented May 5, 2015

Je prends

@Eskimon Eskimon self-assigned this May 5, 2015
@SpaceFox
Copy link
Contributor

SpaceFox commented May 5, 2015

C'est une vraie erreur que tu as obtenue en appuyant sur un bouton ou tu as fait n'importe quoi avec les paramètres ?

@Eskimon
Copy link
Contributor

Eskimon commented May 5, 2015

C'est une vraie erreur que tu as obtenue en appuyant sur un bouton ou tu as fait n'importe quoi avec les paramètres ?

Il faut bricoler les paramètres. Mais la remarque "ne pas avoir de 500" reste vrai tout de même.

@SpaceFox
Copy link
Contributor

SpaceFox commented May 5, 2015

Ouais mais ça change complètement la priorité du truc : si c'est en appuyant sur un bouton c'est urgent, si c'est en jouant avec les paramètres on s'en fout, ça ne met pas en danger le site et l'utilisateur n'a qu'à pas jouer avec les paramètres :)

@Eskimon
Copy link
Contributor

Eskimon commented May 5, 2015

Tout a fait :) Mais j'ai juste dit je prends car je sais très bien comment corriger et faire le TU, le tout en 5 minutes ^^

@firm1
Copy link
Contributor Author

firm1 commented May 5, 2015

Comme le souligne Eskimon, on ne tombe pas dessus par hasard, faut le chercher. Donc oui, en priorité faible, après si c'est faisable rapidement, c'est toujours ça de corrigé

@Eskimon
Copy link
Contributor

Eskimon commented May 5, 2015

Voila.

@Situphen
Copy link
Member

Corrigé. Merci @Eskimon !

@Situphen Situphen added this to the Version de développement milestone May 10, 2015
@SpaceFox SpaceFox modified the milestones: Version 15.5.1, Version de développement May 26, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django Facile Bon ticket pour débuter pour rejoindre le développement ! S-BUG Corrige un problème
Projects
None yet
Development

No branches or pull requests

4 participants