Projet

Général

Profil

Development #38961

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

Ajouté par Frédéric Péters il y a environ 4 ans. Mis à jour il y a presque 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
14 janvier 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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

Lié à w.c.s. - Bug #12975: Empêcher d'utiliser deux fois le même varname dans un formulaireRejeté29 août 2016

Actions

Révisions associées

Révision 8faf46cf (diff)
Ajouté par Frédéric Péters il y a presque 4 ans

variables: use first field with a value in case of duplicated varnames (#38961)

Historique

#1

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é
#2

Mis à jour par Frédéric Péters il y a environ 4 ans

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

#3

Mis à jour par Benjamin Dauvergne il y a environ 4 ans

  • Statut changé de Solution proposée à Solution validée

Ok.

#4

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

#5

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)
#6

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

Formats disponibles : Atom PDF