Projet

Général

Profil

Development #46588

Liste de noms de FormDef interdits

Ajouté par Lauréline Guérin il y a plus de 3 ans. Mis à jour il y a plus de 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
11 septembre 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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


Fichiers

Révisions associées

Révision 8a098533 (diff)
Ajouté par Lauréline Guérin il y a plus de 3 ans

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

Historique

#1

Mis à jour par Lauréline Guérin il y a plus de 3 ans

  • Description mis à jour (diff)
#2

Mis à jour par Lauréline Guérin il y a plus de 3 ans

(vraiment pas sûre de mon coup)

#3

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

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

Mis à jour par Lauréline Guérin il y a plus de 3 ans

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

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

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

#7

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

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

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

#8

Mis à jour par Lauréline Guérin il y a plus de 3 ans

  • Statut changé de Solution validée à 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

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

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

Formats disponibles : Atom PDF