Development #22102
Évaluer le contenu d'un champ "condition" au moment où il est écrit
Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
24 février 2018
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
Comme on a sur les ComputedExpressionWidget, en cours de frappe, marquer le champ en rouge s'il y a une erreur dans la syntaxe de la condition.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 6 ans
- Fichier 0001-forms-evaluate-conditions-during-typing-22102.patch 0001-forms-evaluate-conditions-during-typing-22102.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 environ 6 ans
- Lié à Development #21550: Stocker les conditions sous forme de dictionnaire ajouté
Mis à jour par Thomas Noël il y a environ 6 ans
Ack
(au passage, à relire le qommon.admin.js qui fait la validation sur les input[data-validation-url], juste au dessus, et sans chercher à factoriser pour l'instant, il y manque sans doute un "each" et un "$widget" devant "validation_timeout_id"… limite de mes compétences en jajascript dépassée)
Mis à jour par Frédéric Péters il y a environ 6 ans
- Fichier 0004-forms-evaluate-conditions-during-typing-22102.patch 0004-forms-evaluate-conditions-during-typing-22102.patch ajouté
Pas envoyé et retravaillé à l'occasion de #21545. (il vient désormais après).
Mis à jour par Thomas Noël il y a environ 6 ans
Dans la validation :
return getattr(self, 'validate_' + self.type, lambda x: None)()
je me dis que normalement jamais on ne devrait arriver dans le lambda, ou bien tu veux juste prévenir d'éventuelles "attaques" de l'api ? Si on le laisse, éventuellement ajouter un test un peu idiot du genre :
resp = get_app(pub).get('/api/validate-condition?type=foo&value_foo=bar') assert resp.json == {'klass': None, 'msg': ''} # just ignore bad type
Ce sont des détails sans grande importance : tu peux pousser tel quel si tu veux.
Mis à jour par Frédéric Péters il y a environ 6 ans
- Statut changé de En cours à Résolu (à déployer)
Ok, j'ai ajouté un "unknown condition type" pour la situation.
commit 62267edcfab8ed8209d68dd112446ca2b519a633 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Sat Mar 24 13:00:18 2018 +0100 forms: evaluate conditions during typing (#22102)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
forms: evaluate conditions during typing (#22102)