Projet

Général

Profil

Bug #3369

crash sortie JSON en cas de formulaire sans statut visible

Ajouté par Thomas Noël il y a presque 11 ans. Mis à jour il y a plus de 10 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
24 juillet 2013
Echéance:
% réalisé:

100%

Temps estimé:
Patch proposed:
Planning:

Description

Trace reçue :

Exception:
  type = '<type 'exceptions.AttributeError'>', value = ''NoneType' object has no attribute 'name''

Stack trace (most recent call first):
  File "/var/vhosts/calvados.dev.au-quotidien.com/src/auquotidien.git/extra/modules/myspace.ptl", line 393, in forms
   391                                      'id': x.get_display_id(),
   392                                      'status': x.get_visible_status(user=self.user).name},
>  393                             'url': x.get_url()} for x in user_forms])
   394
   395

Il faut vérifier que x.get_visible_status(user=self.user) n'est pas à None, ça peut arriver pour des demandes qui sont dans un statut non visible par l'utilisateur, genre.


Fichiers

Révisions associées

Révision ee0df521 (diff)
Ajouté par Benjamin Dauvergne il y a plus de 10 ans

myspace: in json forn listing, skip form without a visible status (fixes #3369)

Historique

#1

Mis à jour par Benjamin Dauvergne il y a presque 11 ans

  • Fichier 0001-myspace-in-json-forn-listing-skip-form-without-a-vis.patch ajouté

Patch pas testé; j'en ai profité pour rendre le code un peu plus lisible.

#2

Mis à jour par Benjamin Dauvergne il y a presque 11 ans

  • Statut changé de Nouveau à Résolu (à déployer)
#3

Mis à jour par Frédéric Péters il y a presque 11 ans

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

Attention, les dictionnaires, plutôt {'xxx': yyy} que dict(xxx= yyy).

#4

Mis à jour par Benjamin Dauvergne il y a presque 11 ans

  • Fichier 0001-myspace-in-json-forn-listing-skip-form-without-a-vis.patch supprimé
#6

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

Ca a l'air ok, sauf le "=tatus" qui traine ligne 402

#7

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

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

Je viens d'appliquer ce patch et de pousser ça dans le dépôt git.

#8

Mis à jour par Benjamin Dauvergne il y a plus de 10 ans

  • % réalisé changé de 0 à 100
#9

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

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

Formats disponibles : Atom PDF