Projet

Général

Profil

Bug #12569

crash sur rappel depuis le backoffice d'un code de suivi valide mais dont le formulaire a été supprimé

Ajouté par Frédéric Péters il y a plus de 7 ans. Mis à jour il y a plus de 7 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
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

Révision 772b1411 (diff)
Ajouté par Frédéric Péters il y a plus de 7 ans

backoffice: don't crash if tracking code is from a deleted formdata (#12569)

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 7 ans

#2

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é
#3

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)

#4

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)
#5

Mis à jour par Thomas Noël il y a plus de 7 ans

Le 13 juillet 2016 à 16h03, 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.

#6

Mis à jour par Frédéric Péters il y a plus de 7 ans

  • Version cible mis à v1.53
#7

Mis à jour par Frédéric Péters il y a plus de 7 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF