Development #56105
site-options: interpréter les valeurs True/False comme des booléens
0%
Description
Pour que les variables soient accessibles dans le contexte des templates en tant que booléens.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 2 ans
Tu ne peux pas juste en faire des booléens ça va amener des bugs aux endroits qui existent et les considèrent comme des chaines; c'est ma parenthèse dans l'autre ticket : (obtenir une compatibilité avec l'existant demanderait dans ce cas sans doute une classe spéciale, avec eq etc.).
Aussi, ast.literal_eval(v.capitalize()) est une manière totalement pétée pour obtenir un booléen.
Mis à jour par Frédéric Péters il y a plus de 2 ans
Aussi, #37989 a amené
- d.update(dict(self.site_options.items('variables')))
+ d.update(dict(self.site_options.items('variables', raw=True)))
tu ne peux pas le défaire.
Mis à jour par Frédéric Péters il y a plus de 2 ans
ça va amener des bugs aux endroits qui existent et les considèrent comme des chaines
Ceci n'est pas juste une crainte farfelue, d'un grep dans publik-base-theme,
{% if activate_plus1_action == "True" %}
et s'il y en a un aussi facilement trouvable, il doit y en avoir d'autres cachés, variables posées depuis hobo et utilisées dans des workflows, etc.
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Nouveau à En cours
- Assigné à mis à Frédéric Péters
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Fichier 0001-misc-add-custom-behaviour-for-booleans-in-variables-.patch 0001-misc-add-custom-behaviour-for-booleans-in-variables-.patch ajouté
- Statut changé de En cours à Solution proposée
- Patch proposed changé de Non à Oui
Voilà ce que j'essayais de suggérer.
Mis à jour par Thomas Noël il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 9e6bd493a6737d1ffa6a47fb681b4c5ed83a9d43
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Tue Aug 10 22:11:27 2021 +0200
misc: add custom behaviour for booleans in [variables] (#56105)
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: add custom behaviour for booleans in [variables] (#56105)