Development #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).
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ un an
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.
Mis à jour par Valentin Deniaud il y a environ un an
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).
Mis à jour par Frédéric Péters il y a environ un an
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.
Mis à jour par Robot Gitea il y a environ un an
- Statut changé de Nouveau à 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
Mis à jour par Robot Gitea il y a environ un an
- Statut changé de Solution proposée à Solution validée
Frédéric Péters (fpeters) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a environ un an
- Statut changé de Solution validée à 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
Mis à jour par Transition automatique il y a environ un an
- Statut changé de Résolu (à déployer) à Solution déployée
statistics: filter only on statistics fields even if same varname (#73562)