Développement #73562
statistiques, ne pas (trop) confondre les champs ayant le même identifiant dans le group_by
0%
Description
Si plusieurs champs ont le même identifiant, on va en prendre un « au pif », c'est comme ça.
Par contre si plusieurs champs ont le même identifiant, avant d'en prendre un au pif on devrait filtrer par affichage dans les statistiques, sinon on risque de prendre un champ dont le type n'est pas supporté et c'est un bug.
Ce ticket va inclure un revert de #73249 (dans ce ticket j'imaginais qu'on avait un bug lors de l'alimentation d'un champ liste avec un champ donnée calculée mais 1/ le champ liste reçoit la donnée finale, pas du json et 2/ c'est en fait une occurence du bug décrit plus haut).
Associated revisions
History
Updated by Frédéric Péters almost 2 years ago
Si plusieurs champs ont le même identifiant, on va en prendre un « au pif », c'est comme ça.
Pas d'accord, il faut prendre le premier qui a une valeur, dans l'ordre d'apparition des champs, c'est la règle qui a été posée dans #38961.
Updated by Valentin Deniaud almost 2 years ago
OK dans l'idée que si une demande aient form_var_foo¹ à 'bla' et une autre form_var_foo² à 'bla', le group_by retourne 2 pour 'bla' ?
Ça m'a l'air compliqué mais ce sera sûrement bien plus simple avec #73174, je propose donc de traiter ce problème dans un autre ticket (ce ticket ci étant vraiment là pour corriger le bug de #73249).
Updated by Frédéric Péters almost 2 years ago
OK dans l'idée que si une demande aient form_var_foo¹ à 'bla' et une autre form_var_foo² à 'bla', le group_by retourne 2 pour 'bla' ?
Ça m'a l'air compliqué mais ce sera sûrement bien plus simple avec #73174, je propose donc de traiter ce problème dans un autre ticket (ce ticket ci étant vraiment là pour corriger le bug de #73249).
Je ne suis pas sûr de comprendre ("group_by retourne 2") mais ok pour dire pars ici et que oui plus facile via #73174.
Updated by Robot Gitea almost 2 years ago
- Status changed from Nouveau to Solution proposée
Valentin Deniaud (vdeniaud) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://gitea.entrouvert.org/entrouvert/wcs/pulls/55
- Titre : statistiques, ne pas (trop) confondre les champs ayant le même identifiant dans le group_by (#73562)
- Modifications : https://gitea.entrouvert.org/entrouvert/wcs/pulls/55/files
Updated by Robot Gitea almost 2 years ago
- Status changed from Solution proposée to Solution validée
Frédéric Péters (fpeters) a approuvé une pull request sur Gitea concernant cette demande :
Updated by Robot Gitea almost 2 years ago
- Status changed from Solution validée to Résolu (à déployer)
Valentin Deniaud (vdeniaud) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://gitea.entrouvert.org/entrouvert/wcs/pulls/55
- Titre : statistiques, ne pas (trop) confondre les champs ayant le même identifiant dans le group_by (#73562)
- Modifications : https://gitea.entrouvert.org/entrouvert/wcs/pulls/55/files
Updated by Transition automatique almost 2 years ago
- Status changed from Résolu (à déployer) to Solution déployée
statistics: filter only on statistics fields even if same varname (#73562)