Development #1630: Vérifier la syntaxe des expressions de pré-remplissage - w.c.s. - Redmine Entr’ouvert

Development #1630

Vérifier la syntaxe des expressions de pré-remplissage

Ajouté par Benjamin Dauvergne il y a plus de 11 ans. Mis à jour il y a plus de 8 ans.

Statut:

Fermé

Priorité:

Normal

Assigné à:

Version cible:

Début:

07 septembre 2012

Echéance:

% réalisé:

Temps estimé:

Patch proposed:

Planning:

Description

diff --git a/wcs/fields.py b/wcs/fields.py
index 81a141a..34c072c 100644
--- a/wcs/fields.py
+++ b/wcs/fields.py
@@ -94,9 +94,12 @@ class PrefillSelectionWidget(CompositeWidget):
             value = self.get(name)
             if value:
                 values[name] = value
-        if values.get('data', '').startswith('------'):
-            values = None
         self.value = values or None
+        if values and values['type'] == 'formula' and values.get('value'):
+            try:
+                compile(values.get('value', ''), '<string>', 'eval')
+            except (SyntaxError, TypeError), e:
+                self.set_error(_('invalid expression: %s') % e)

Fichiers

0001-fields-removed-dead-code-in-PrefillSelectionWidget._.patch (764 octets) 0001-fields-removed-dead-code-in-PrefillSelectionWidget._.patch		Benjamin Dauvergne, 03 décembre 2012 15:22
0002-fields-added-validation-of-Python-expressions-in-Pre.patch (1,09 ko) 0002-fields-added-validation-of-Python-expressions-in-Pre.patch		Benjamin Dauvergne, 03 décembre 2012 15:22

Demandes liées

Historique

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

Pourquoi la suppression de ces lignes ?

-        if values.get('data', '').startswith('------'):
-            values = None

Mis à jour par Benjamin Dauvergne il y a plus de 11 ans

redmine@entrouvert.com écrivait:

La demande #1630 a été mise à jour par Frédéric Péters.

Pourquoi la suppression de ces lignes ?
> -        if values.get('data', '').startswith('------'):
> -            values = None
> 

Elle m'ont semblé ne plus servir, values ne pouvant jamais contenir la
clé 'data'. Je ferai 2 commits séparés pour l'expliquer.

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

Mais ce code est peut-être encore utile face à d'anciens formdefs, à défaut de pouvoir s'en assurer, je préférerais le conserver.

Mis à jour par Benjamin Dauvergne il y a plus de 11 ans

redmine@entrouvert.com écrivait:

La demande #1630 a été mise à jour par Frédéric Péters.

Mais ce code est peut-être encore utile face à d'anciens formdefs, à défaut de pouvoir s'en assurer, je préférerais le conserver.

Non ce n'est pas possible, "values" n'est pas extrait d'un formdef ou
d'un formdata il est créé à la volée dans la méthode _parse() et ne
contient que les clés "type" et "value"; lis le code si tu veux t'en
assurer [1].

De plus on parle du code de la méthode _parse() d'un widget je ne vois
pas ce que ça a avoir avec le fonctionnement d'un vieux formdef; je ne
vois pas dans quel cas d'utilisation on passerait dans ce code à part la
soumission d'une nouvelle valeur pour le champ "prefill".

[1]: pour t'éviter de le faire:

    def _parse(self, request):
        values = {}
        for name in ('type', 'value'):
            value = self.get(name)
            if value:
                values[name] = value
        self.value = values or None
        if values and values['type'] == 'formula' and values.get('value'):
            try:
                compile(values.get('value', ''), '<string>', 'eval')
            except (SyntaxError, TypeError), e:
                self.set_error(_('invalid expression: %s') % e)