Project

General

Profile

Développement #62264

API stats, le regroupement sur un champ liste se fait selon les ids

Added by Valentin Deniaud almost 3 years ago. Updated almost 3 years ago.

Status:
Fermé
Priority:
Normal
Target version:
-
Start date:
28 February 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Ce n'est pas grave pour une liste simple, mais dans le cas d'une source de donnée c'est souvent complètement illisible.

On pourrait vouloir taper direct le GROUP BY sur la colonne _display mais ça ne fonctionnera pas pour les listes à choix multiples (où le comportement est de séparer les choix, cf #62263).

Donc plutôt repasser sur les résultats de la requête pour traduire l'id en display, à partir de la liste des choix.


Files

Associated revisions

Revision dae6a823 (diff)
Added by Valentin Deniaud almost 3 years ago

statistics: use display value on group by (#62264)

History

#1

Updated by Valentin Deniaud almost 3 years ago

#2

Updated by Frédéric Péters almost 3 years ago

  • Status changed from Solution proposée to Solution validée
        if not 'group_by' in totals_kwargs:

curieux il n'y a pas pylint pour suggérer d'écrire if 'group_by' not in total_kwargs ?

#3

Updated by Valentin Deniaud almost 3 years ago

  • Status changed from Solution validée to Résolu (à déployer)
commit dae6a823d7686f886bcd006ac7d3abec5591db7a
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Mon Feb 28 18:17:53 2022 +0100

    statistics: use display value on group by (#62264)
#5

Updated by Transition automatique almost 3 years ago

  • Status changed from Résolu (à déployer) to Solution déployée
#6

Updated by Transition automatique over 2 years ago

Automatic expiration

Also available in: Atom PDF