Project

General

Profile

Development #46588

Liste de noms de FormDef interdits

Added by Lauréline Guerin 5 months ago. Updated 4 months ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
11 Sep 2020
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

et de CardDef aussi ? (à vérifier)

- créer un FormDef avec en titre "Form"
- => on ne peut pas éditer le titre, lancer une démarche, ... Car le code load la classe wcs.qommon.form.Form, qui ne convient pas

 [2020-09-11 15:24:12] exception caught
Exception:
 type = '<class 'AttributeError'>', value = 'type object 'Form' has no attribute '_formdef''

Stack trace (most recent call first):
 File "/home/lguerin/src/wcs/wcs/formdef.py", line 270, in data_class
  268             # only use existing data class if it has a reference to this actual
  269             # formdef
>  270             if data_class._formdef is self:
  271                 return data_class
  272         if (get_publisher().is_using_postgresql() and not mode == 'files') or mode == 'sql':

Plus globalement, préfixer les noms des classes générées dynamiquement. Attention au unpickle


Files

Associated revisions

Revision 8a098533 (diff)
Added by Lauréline Guérin 4 months ago

misc: prefix CardData & FormData class name to avoid conflict (#46588)

History

#1

Updated by Lauréline Guerin 4 months ago

  • Description updated (diff)
#2

Updated by Lauréline Guerin 4 months ago

(vraiment pas sûre de mon coup)

#3

Updated by Frédéric Péters 4 months ago

Sous tests/data/oldpickledata/ tu peux retirer les fichiers fichiers/répertoires cachés, ils ne doivent servir à rien.

Dans wcs/formdata.py tu as laissé un print() de debug.

Vu les tests ça fonctionne mais je trouve un peu osé d'inclure un caractère : dans le nom de la classe, je serais plus à l'aise avec wcs plutôt que "WCS:" comme préfixe.

#4

Updated by Lauréline Guerin 4 months ago

J'aimais assez le : dans le nom de la classe :)
Mais ok, nous n'avons pour le moment dans le code aucune classe préfixée par WCS.

#5

Updated by Frédéric Péters 4 months ago

Ah redmine a fait de mes underscores de l'italique, je voulais écrire _wcs_ comme préfixe.

#7

Updated by Frédéric Péters 4 months ago

  • Status changed from Solution proposée to Solution validée

Attendre quand même la fin du jenkins mais pour moi c'est ok.

#8

Updated by Lauréline Guerin 4 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 8a0985332ca480e57534b9cd75bc9390ee132d08
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Thu Sep 17 14:48:37 2020 +0200

    misc: prefix CardData & FormData class name to avoid conflict (#46588)
#9

Updated by Frédéric Péters 4 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF