Développement #13650
closedAlerte 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.
Files
Updated by Frédéric Péters over 9 years ago
- File 0001-admin-add-live-validation-hints-to-computed-expressi.patch 0001-admin-add-live-validation-hints-to-computed-expressi.patch added
- Status changed from Nouveau to En cours
- Patch proposed changed from No to Yes
Updated by Frédéric Péters over 9 years ago
- File 0001-admin-add-live-validation-hints-to-computed-expressi.patch 0001-admin-add-live-validation-hints-to-computed-expressi.patch added
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).
Updated by Thomas Noël over 9 years ago
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".
Updated by Frédéric Péters over 9 years ago
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.
Updated by Thomas Noël over 9 years ago
Oui mais là on va finalement laisser passer "=" comme contenu de champ valide alors que avant, non.
Updated by Frédéric Péters over 9 years ago
- Status changed from En cours to 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)
Updated by Frédéric Péters about 9 years ago
- Status changed from Résolu (à déployer) to Fermé