Development #13650
Alerte sur l'utilisation de =[whatever] dans le champ d'expression calculée
0%
Description
On autorise à la fois du texte avec de l'ezt et des expressions Python et les gens mélangent et bim se trouvent à écrire =[form_var_hello].
On devrait ajouter au widget du js qui afficherait un message d'avertissement quand cette situation est rencontrée.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 7 ans
- Fichier 0001-admin-add-live-validation-hints-to-computed-expressi.patch 0001-admin-add-live-validation-hints-to-computed-expressi.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 7 ans
- Fichier 0001-admin-add-live-validation-hints-to-computed-expressi.patch 0001-admin-add-live-validation-hints-to-computed-expressi.patch ajouté
Modifié le js pour ne faire l'appel qu'après 250ms d'inaction, histoire de ne pas taper inutilement sur le serveur. (idéalement on devrait avoir un timeout par widget mais comme le délai est court je pense qu'il n'y aura pas de problème de personne navigant très vite de champ d'expression en champ d'expression).
Mis à jour par Thomas Noël il y a plus de 7 ans
Dans le « ComputedExpressionWidget.validate » il y a un « if expression.startswith('=') and len(expression) > 1: / else: » qui va faire que le else va gérer "=" (bon, c'est pas grave, mais bon, faut bien que je montre que je relis). Bref, je retirerais le "len(expression) > 1" pour remonter l'erreur de syntaxe "expression vide", qui est le comportement actuel.
À part ça, c'est un "ack".
Mis à jour par Frédéric Péters il y a plus de 7 ans
Oui, en fait c'est volontaire parce que eval('') c'est une erreur de syntaxe et je ne voulais pas l'afficher, histoire que bam, premier caractère tapé et déjà dans le rouge.
Mis à jour par Thomas Noël il y a plus de 7 ans
Oui mais là on va finalement laisser passer "=" comme contenu de champ valide alors que avant, non.
Mis à jour par Frédéric Péters il y a plus de 7 ans
- Statut changé de En cours à Résolu (à déployer)
Ok, len(expression) retirée.
commit ca9f9fac89f208c342f3fbb3114a5001aa260cd3 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue Oct 18 14:18:20 2016 +0200 admin: add live validation (+ hints) to computed expression widgets (#13650)
admin: add live validation (+ hints) to computed expression widgets (#13650)