Project

General

Profile

Bug #11000

la classe PicklableUpload peut exister sous deux namespaces différents

Added by Frédéric Péters (de retour le 10/10) over 6 years ago. Updated over 6 years ago.

Status:
Fermé
Priority:
Normal
Target version:
Start date:
20 May 2016
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:

Description

(wcs.qommon.form.PicklableUpload et qommon.form.PicklableUpload).


Files


Related issues

Related to w.c.s. - Bug #10769: Lorsque workflow_data contient des PickableUpload (formulaire backoffice) l'export JSON planteFermé29 April 2016

Actions

Associated revisions

Revision 2adbdc17 (diff)
Added by Frédéric Péters (de retour le 10/10) over 6 years ago

misc: check for serializable file using hasattr() (#11000)

PicklableUpload may exist in different namespaces, and then there's also an
independant UploadedFile class; just check if will duck like a file.

Revision 7e91ff8e (diff)
Added by Thomas Noël almost 6 years ago

workflows: check file type with hasattr() for backoffice fields (#13777)

PicklableUpload may exist in different namespaces, and then there's also
an independant UploadedFile class; just check if will duck like a file.
Similar to #11000 (commit 2adbdc1720)

History

#1

Updated by Frédéric Péters (de retour le 10/10) over 6 years ago

Je suis frileux à tenter de passer partout et ça ne corrigerait pas les données déjà sauvegardées (mais on pourrait jouer sur le setstate de PicklableUpload).

Bref, code pas terrible mais qui contourne le problème.

#3

Updated by Frédéric Péters (de retour le 10/10) over 6 years ago

  • Related to Bug #10769: Lorsque workflow_data contient des PickableUpload (formulaire backoffice) l'export JSON plante added
#4

Updated by Benjamin Dauvergne over 6 years ago

Pour le futur je vois deux pistes, qui commence toutes les deux par supprimer le sys.path.append(wcs_path):
  • déplacer qommon à la racine et mettre des fichiers de remplacement dans wcs/qommon/xxxx.py qui font juste from qommon.xxx import *
  • l'inverse: créer à la racine un qommon avec des fichiers proxy..

On peut aussi jouer dans sys.modules mais bon...

#5

Updated by Frédéric Péters (de retour le 10/10) over 6 years ago

En important les deux de manière absolues, pour être sûr d'avoir deux différents…

#6

Updated by Frédéric Péters (de retour le 10/10) over 6 years ago

Alternative que je préfère, si ça sonne comme un fichier, on le prend comme un fichier.

#7

Updated by Thomas Noël over 6 years ago

Ack

Note pour plus tard, UploadedFile dispose aussi d'un attribut base_filename mais pas d'un get_content. Cette gestion de UploadedFile ne semble cependant pas utile pour l'instant, on laisse ce nettoyage plus propre pour plus tard (sans doute avec une utilisation de hasattr(obj, 'get_json_export_dict')

#8

Updated by Frédéric Péters (de retour le 10/10) over 6 years ago

  • Status changed from En cours to Résolu (à déployer)
commit 2adbdc1720f9f1a60b8c690ac53eeda71c051d20
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri May 20 09:40:11 2016 +0200

    misc: check for serializable file using hasattr() (#11000)

    PicklableUpload may exist in different namespaces, and then there's also an
    independant UploadedFile class; just check if will duck like a file.
#9

Updated by Thomas Noël over 6 years ago

  • Target version set to v1.43
#10

Updated by Frédéric Péters (de retour le 10/10) over 6 years ago

  • Status changed from Résolu (à déployer) to Fermé

Also available in: Atom PDF