Project

General

Profile

Development #38961

en évaluation paresseuse, comportement sur un formulaire contenant plusieurs champs avec le même nom de variable

Added by Frédéric Péters 3 months ago. Updated about 2 months ago.

Status:
Solution validée
Priority:
Normal
Target version:
-
Start date:
14 Jan 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

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.

0001-variables-use-first-field-with-a-value-in-case-of-du.patch View (2.52 KB) Frédéric Péters, 24 Jan 2020 01:45 PM


Related issues

Related to w.c.s. - Bug #12975: Empêcher d'utiliser deux fois le même varname dans un formulaire Nouveau 29 Aug 2016

History

#1 Updated by Frédéric Péters 3 months ago

  • Related to Bug #12975: Empêcher d'utiliser deux fois le même varname dans un formulaire added

#2 Updated by Frédéric Péters 3 months ago

(je pensais avoir déjà attaché ce patch...)

#3 Updated by Benjamin Dauvergne about 2 months ago

  • Status changed from Solution proposée to Solution validée

Ok.

#4 Updated by Thomas Noël about 2 months ago

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").

Also available in: Atom PDF