Project

General

Profile

Development #42429

Affichage d'un message d'erreur quand deux champs ont le même varname

Added by Pierre Cros 26 days ago. Updated 26 days ago.

Status:
Solution proposée
Priority:
Normal
Target version:
-
Start date:
04 May 2020
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

C'était bien avant mais maintenant c'est autorisé d'avoir 2 champs avec le même varname.

varname-bijoe.png View (45.3 KB) Pierre Cros, 04 May 2020 10:05 AM

0001-misc-accept-duplicate-fields-with-the-same-type-4242.patch View (8.03 KB) Benjamin Dauvergne, 04 May 2020 09:42 PM

44208

Related issues

Related to w.c.s. - Bug #42468: API : la règle « premier varname arrivé, premier servi » n'est pas respectée Nouveau 04 May 2020

History

#1 Updated by Pierre Cros 26 days ago

  • File deleted (varname-bijoe.png)

#2 Updated by Pierre Cros 26 days ago

  • File varname-bijoe.png added

#3 Updated by Pierre Cros 26 days ago

  • File deleted (varname-bijoe.png)

#4 Updated by Pierre Cros 26 days ago

44208

#5 Updated by Benjamin Dauvergne 26 days ago

Ce n'est pas vraiment autorisé, en vrai, il y a juste une sorte de contournement pour que de temps en temps ça fonctionne comme les gens espèremt que ça va fonctionner. Et ça ne marchera toujours pas dans wcs-olap sauf si les APIs change pour n'afficher qu'un champ; je reçois une valeur pour le champ "varname=x" qui existe deux fois en type bool et item, je le met dans quelle colonne de ma table des statistiques ?

#6 Updated by Frédéric Péters 26 days ago

Ce n'est pas vraiment autorisé,

C'est totalement autorisé et le comportement est désormais défini : en cas de plusieurs champs portant la même variable, c'est le premier rempli, par ordre d'apparition dans le formulaire, qui vaut.

Alors oui il reste que ça n'a sans doute que peu de sens d'utiliser le même identifiant pour des champs sans rapport / de types différents, et très bien que ces cas soient toujours exposés en erreur, mais ce n'est pas le cas exposé ici.

#7 Updated by Benjamin Dauvergne 26 days ago

  • Project changed from BiJoe to OLAP / Businesse Intelligence pour Publik

#8 Updated by Benjamin Dauvergne 26 days ago

Frédéric Péters a écrit :

Ce n'est pas vraiment autorisé,

C'est totalement autorisé et le comportement est désormais défini : en cas de plusieurs champs portant la même variable, c'est le premier rempli, par ordre d'apparition dans le formulaire, qui vaut.

Oui mais l'API ne fonctionne pas comme cela, wcs.formdata.get_json_dict() ne garde que la dernière valeur.

#9 Updated by Benjamin Dauvergne 26 days ago

  • Related to Bug #42468: API : la règle « premier varname arrivé, premier servi » n'est pas respectée added

#10 Updated by Benjamin Dauvergne 26 days ago

  • Assignee set to Benjamin Dauvergne

#11 Updated by Benjamin Dauvergne 26 days ago

Voilà, les tests devraient passer quand w.c.s. renverra les bonnes valeurs; ce serait bien si w.c.s. respectait la même règle au niveau du type (i.e. qu'un champ ne prenne la place d'un autre que s'il vient après, que le premier n'a pas de valeur et qu'ils sont du même type).

Also available in: Atom PDF