Projet

Général

Profil

Bug #23616

Conditions: tenter de détecter les confusions en Python (utilisation de {{ }})

Ajouté par Thomas Noël il y a presque 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
05 mai 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Exemple d'erreur vue sur une condition en Python : {{form_var_foo}} == 'Oui'

On devrait entourer l'input en orange dans ce cas pour signaler à la personne qu'on pense qu'elle utilise du template Django dans une condition, et que ça risque de ne pas le faire.

Idem sur détection des motifs [form_*], [if-any*]

Il faut faire une liste des motifs classiques d'erreur, et surtout trouver le message d'aide à afficher en face.


Fichiers

Révisions associées

Révision 7b6dc1dd (diff)
Ajouté par Thomas Noël il y a presque 6 ans

forbid Django {{ in Python expression or condition (#23616)

Historique

#1

Mis à jour par Thomas Noël il y a presque 6 ans

En première approche, et parce que c'est une erreur classique : interdire carrément l'utilisation de {{ ou {% dans une condition.
(l'improbable besoin de faire un dict de dict en Python restera possible avec { {)

#2

Mis à jour par Thomas Noël il y a presque 6 ans

#3

Mis à jour par Frédéric Péters il y a presque 6 ans

Plutôt que "do not use {{ in a Python condition/expression", je préférerais une forme genre "invalid usage, Python condition/expression cannot contain {{".

#5

Mis à jour par Frédéric Péters il y a presque 6 ans

  • Statut changé de En cours à Résolu (à déployer)
commit 7b6dc1dddfa2bb183b267eda07f644f5bdb54776
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Wed May 23 20:30:11 2018 +0200

    forbid Django {{ in Python expression or condition (#23616)
#6

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

Formats disponibles : Atom PDF