Development #38961
en évaluation paresseuse, comportement sur un formulaire contenant plusieurs champs avec le même nom de variable
0%
Description
De #38960#note-5,
en évaluation paresseuse on pourrait facilement, en cas de variable au nom dupliqué, ignorer celles sans valeur; ça aiderait pas mal pour ce genre de formulaires où un champ ou un autre est affiché selon des conditions antérieures, et où ce serait pratique de leur donner le même nom de variable.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 4 ans
- Lié à Bug #12975: Empêcher d'utiliser deux fois le même varname dans un formulaire ajouté
Mis à jour par Frédéric Péters il y a environ 4 ans
- Fichier 0001-variables-use-first-field-with-a-value-in-case-of-du.patch 0001-variables-use-first-field-with-a-value-in-case-of-du.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
(je pensais avoir déjà attaché ce patch...)
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Ok.
Mis à jour par Thomas Noël il y a environ 4 ans
Discussions sur le salon :
si ça permet de donner le même nom de variable sur des champs dont on est sûr qu'n seul sera rempli oui c'est bonnard
oui ça permettrait ça, sans vérification que les champs avec le même identifiant sont bien exclusifs
ce que ça définit surtout c'est le comportement quand il y a plusieurs champs
alors qu'aujourd'hui c'est indéfini et surprises.
Pour ma part j'ai l'impression que ça ajoute juste une autre magie, que l'on voit dans le test : c'est la première valeur, mais c'est la deuxième "s'il n'y pas de première valeur". Je trouve que ça serait plus simple que ça doit toujours la première valeur (if field.varname in self._varnames: continue
) et évite aux gens d'avoir des solutions magiques incompréhensibles à maintenir ("j'utilise form_var_truc parce que ça peut être ça ou ça ou ça dans le formulaire ou ses données de traitement, parce que j'ai mis les conditions qu'il faut").
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 8faf46cfc3194b1e2e58da9b373af32738650ac4 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue Jan 14 16:49:43 2020 +0100 variables: use first field with a value in case of duplicated varnames (#38961)
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
variables: use first field with a value in case of duplicated varnames (#38961)