Projet

Général

Profil

Bug #12848

erreur lors d'un export vers d'un tableau de demandes avec une colonne "canal" rajoutée

Ajouté par Serghei Mihai (congés, retour 15/05) 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:
09 août 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Trace lors d'une tentative d'export en tableur d'un tableau de demandes avec une colonne supplémentaire rajoutée:

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

Stack trace (most recent call first):
  File "/usr/lib/python2.7/dist-packages/wcs/backoffice/management.py", line 1218, in csv_tuple
  1216             else:
  1217                 element = data.data.get(field.id, '') or ''
> 1218                 if field.store_display_value:  
  1219                     display_value = data.data.get('%s_display' % field.id) or ''
  1220             elements.extend(field.get_csv_value(element,

  locals: 
     elements = ['36-1467']
     hint = None
     fields = [<wcs.backoffice.management.FakeField object at 0x7f18253aa550>, <wcs.backoffice.management.FakeField object at 0x7f18253aa510>, <wcs.backoffice.management.FakeField object at 0x7f18253aa490>, <wcs.backoffice.management.FakeField object at 0x7f18253aa1d0>, <wcs.fields.ItemField object at 0x7f182038ec50>, <wcs.fields.ItemField object at 0x654de10>, <wcs.backoffice.management.FakeField object at 0x7f18253aa5d0>]
     self = <wcs.backoffice.management.FormPage object at 0x63799d0>
     element = ''
     field = <wcs.backoffice.management.FakeField object at 0x7f18253aa510>
     data = <Inscription-Aux-Activites-Peri-Et-Extrascolaires-Rentree-20162017 id:1467>
     display_value = None

  File "/usr/lib/python2.7/dist-packages/wcs/backoffice/management.py", line 1351, in export
  1349 
  1350                 for i, filled in enumerate(items):
> 1351                     for j, elem in enumerate(self.formpage.csv_tuple(fields, filled)):  
  1352                         if elem and len(elem) > 32767:
  1353                             # xls cells have a limit of 32767 characters, cut


Fichiers

Révisions associées

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

backoffice: fix listing export of channel and anonymised columns (#12848)

Historique

#1

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 7 ans

Idée à valider, sans test pour l'instant.

#2

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

Sur la forme, une information importante c'était qu'il s'agissait de la colonne "Canal"; il est utile de lier les tickets techniques aux tickets client. Et aussi, pour w.c.s., les patchs déposés sans test ne méritent pas la case à cocher "Patch proposed".

Cela étant posé, la correction proposée par le patch est mauvaise, le résultat ce sera un export avec une colonne "Canal" vide, alors qu'on veut une info utile. Si ça n'avait pas été le cas, le patch était également mauvais dans la mesure où store_display_value aurait du être un attribut mis à None.

Patch correct, avec des tests, et considérant le même problème pour la colonne "Anonymised", attaché.

#4

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

  • Statut changé de Nouveau à En cours
#5

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

Ack

#6

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

  • Statut changé de En cours à Résolu (à déployer)
commit 6947f4b87137128966b11e3911a5b61ed93e29bd
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Aug 9 19:42:55 2016 +0200

    backoffice: fix listing export of channel and anonymised columns (#12848)
#7

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

(aussi posé en prod, sauvagement pour #12847)

#8

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

  • Version cible mis à v1.56
#9

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