Produits Entr'ouvert » w.c.s.

On a pour les champs un attribut "key" posé au niveau de la classe,

class StringField(WidgetField):
    key = 'string'

Mais à l'ajout d'un champ on duplique cet attribut dans un nouvel attribut, "type",

            field = fields.get_field_class_by_type(field_type)(
                label=label, type=field_type, id=self.objectdef.get_new_field_id()
            )

(field_type venant ici du <select> proposant la liste des types de champs).

Ensuite dans le code à quantité d'endroits on se base sur cet attribut.

Ça oblige dans les tests à faire pareil, à répéter le type, alors que la classe devrait être suffisante, ex:

    fields.StringField(id='123', required=True, label='Test', type='string'),

L'attribut "type" est historique, d'un temps où il y avait une classe unique, c'est passé depuis mars 2006, qui ajoutait cette migration :

+class FormField:
+    ### only used to unpickle form fields from older (<200603) versions
+    def __setstate__(self, dict):
+        type = dict['type']
+        self.real_field = fields.get_field_class_by_type(type)(**dict)

Beaucoup de ce code passé a disparu mais l'utilisation de field.type persiste à quantité d'endroits, ce ticket pour nettoyer.