Bug #12569
crash sur rappel depuis le backoffice d'un code de suivi valide mais dont le formulaire a été supprimé
Début:
13 juillet 2016
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
Exception: type = '<type 'exceptions.KeyError'>', value = '' ... File "/usr/lib/python2.7/dist-packages/wcs/backoffice/management.py", line 481, in code 479 get_session().message = ('error', _('No such code')) 480 return redirect(get_request().form.get('back') or '.') > 481 formdata = tracking_code.formdata 482 get_session().mark_anonymous_formdata(formdata) 483 return redirect(formdata.get_url(backoffice=True))
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 7 ans
- Fichier 0001-backoffice-don-t-crash-if-tracking-code-is-from-a-de.patch 0001-backoffice-don-t-crash-if-tracking-code-is-from-a-de.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 7 ans
- Sujet changé de crash sur rappel depusi le backoffice d'un code de suivi valide mais dont le formulaire a été supprimé à crash sur rappel depuis le backoffice d'un code de suivi valide mais dont le formulaire a été supprimé
Mis à jour par Thomas Noël il y a plus de 7 ans
Ack.
Me demandant comment c'était possible, j'ai l'impression que la suppression d'un formdata ne supprime par le formdata.tracking_code attaché quand il existe. C'est voulu ? (je ne me rappelle plus si on avait déjà évoqué la chose)
Mis à jour par Frédéric Péters il y a plus de 7 ans
- Statut changé de En cours à Résolu (à déployer)
Ce n'est pas nécessairement voulu ou réfléchi. (sur le vif je dirais que ça a comme avantage de ne pas réutiliser de codes de suivi).
commit 772b141178cdf1776061fc95959638a5626ea619 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Wed Jul 13 15:36:40 2016 +0200 backoffice: don't crash if tracking code is from a deleted formdata (#12569)
Mis à jour par Thomas Noël il y a plus de 7 ans
Le 13 juillet 2016 à 16h03, redmine@entrouvert.com a écrit :
Ce n'est pas nécessairement voulu ou réfléchi. (sur le vif je dirais que ça a comme avantage de ne pas réutiliser de codes de suivi).
Et c'est donc la bonne raison qu'on avait dû discuter (vague souvenir) ;
laissons comme cela, très bien.
backoffice: don't crash if tracking code is from a deleted formdata (#12569)