Produits Entr'ouvert » w.c.s.

Je sais que la question a déjà été évoquée, je ne sais plus où on en était, mais j'imaginerais bien une fonction du genre:

def error_in_expression(expression):
    try:
        compile(expression, filename='<string>', mode='eval')
    except SyntaxError, e:
        return _('Erreur de syntaxe au caractère %s') % e.offset
    return None

qu'on pourrait appeler quand on affiche un form avec une expression, genre « if error_in_expression(condition): hint="il semble que cette expression comporte une erreur (%s)" % error_in_expression(condition) »

C'est une suite au ticket #1630 il me semble. À ce stade on pourrait même envisager un widget "Expression" qui centraliserait tout ça ? Pour l'instant je crois qu'on a simplement un widget 'Préremplissage' mais c'est un cas particulier du cas général "expression". L'autre source de warning qui serait utile ce serait de pouvoir signaler les variables qui nous semble ne pas exister, en analysant l'arbre de l'expression compilée par rapport aux variables de substitution disponibles. Il y aussi le fait de pouvoir filtrer les expressions dangereuses genre open('/etc/passwd').read() ou range(1000000000000), la seule bonne façon de faire semble être de faire une whitelist.

Je bouge ma note dans un nouveau ticket j'ai cru celui-ci plus large que ça.

Et ensuite évoluer pour que le EvalStringWidget ne renvoie pas juste une chaîne mais un objet d'un type PythonExpression (qui se pickle bien quand même), ayant une méthode eval() et qui implémenterait toutes les siouxeries qu'on pourrait inventer dans le futur.

Benjamin Dauvergne a écrit :

Je voyais plus un truc comme ça.

Yep, mieux.

compile() peu aussi faire du TypeError, on laisse passer ?

(et sinon, je suis tatasse, mais je trouve qu'utiliser ast.parse serait plus à propos ici)

Oui on peut remplacer par ast.parse.

Je pousserais dès maintenant l'idée vers un PythonConditionWidget (qui serait aussi à utiliser pour les conditions dans l'action de workflow "saut") et j'y ajouterais une lecture minimale du résultat de l'ast.parse, pour avertir en cas de _ast.Assign que c'est plutôt == que la personne veut (et ainsi totalement revenir à l'intitulé de ce ticket).

Frédéric Péters a écrit :

Je viens de mettre un a = 'b' dans un champ "condition" de saut et ça n'a pas râlé du tout. D'ailleurs le champ condition est un StringWidget.

J'ai confondu les expression de conditions avec celles pour compute.