Projet

Général

Profil

Development #32643

crash sur champ fichier ajouté dans le profil usager, si celui-ci n'a pas de fichier

Ajouté par Thomas Noël il y a environ 5 ans. Mis à jour il y a environ 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
26 avril 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

parce que sur un usager qui n'a pas de fichier :

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

Stack trace (most recent call first):
  File "/usr/lib/python2.7/dist-packages/wcs/fields.py", line 940, in get_view_value
   938         t = TemplateIO(html=True)
   939         t += htmltext('<div class="file-field">')
>  940         t += htmltext('<a download="%s" href="[download]?f=%s">') % (value.base_filename, self.id)
   941         if include_image_thumbnail and can_thumbnail(value.content_type):
   942             t += htmltext('<img alt="" src="[download]?f=%s&thumbnail=1"/>') % self.id

  locals:
     include_image_thumbnail = True
     self = <fields.FileField object at 0x7f5bbab5ec10>
     t = <TemplateIO object at 0x7f5bba7f06b0>
     value = None

  File "/usr/lib/python2.7/dist-packages/wcs/admin/users.py", line 153, in _q_index
   151                     r += htmltext('</div>')
   152                     r += htmltext('<div class="StringWidget content">')
>  153                     r += field.get_view_value(value)
   154                     r += htmltext('</div>')
   155


Fichiers

Révisions associées

Révision 142ef103 (diff)
Ajouté par Thomas Noël il y a environ 5 ans

admin: handle empty values on user page (#32643)

Historique

#1

Mis à jour par Thomas Noël il y a environ 5 ans

  • Sujet changé de crash sur champ fichier ajouté dans le profil usager à crash sur champ fichier ajouté dans le profil usager, si celui-ci n'a pas de fichier
#2

Mis à jour par Thomas Noël il y a environ 5 ans

Je propose simplement d'afficher un "Non renseigné" sur tous les champs None

#4

Mis à jour par Frédéric Péters il y a environ 5 ans

  • Statut changé de Solution proposée à Solution validée

Ok mais pas sûr que le get_view_value() suffise quand il y a un fichier. (il y aurait un [download] à peut-être substituer, mais surtout pas sûr qu'ensuite le fichier soit accessible via une URL).

#5

Mis à jour par Thomas Noël il y a environ 5 ans

  • Statut changé de Solution validée à Résolu (à déployer)

Frédéric Péters a écrit :

Ok mais pas sûr que le get_view_value() suffise quand il y a un fichier. (il y aurait un [download] à peut-être substituer, mais surtout pas sûr qu'ensuite le fichier soit accessible via une URL).

J'avoue que j'ai même pas testé

commit 142ef10355db4d36662137a9e27bbfc60917cc87
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Fri Apr 26 12:10:45 2019 +0200

    admin: handle empty values on user page (#32643)

#6

Mis à jour par Frédéric Péters il y a environ 5 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF