Projet

Général

Profil

Development #74320

Optimiser la génération de criteria._label dans le cas du filtrage d'un champ item sur une valeur

Ajouté par Benjamin Dauvergne il y a environ un an. Mis à jour il y a environ un an.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
08 février 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Le code actuel récupère toutes les valeurs possibles via ItemField.get_options() au lieu d'utiliser ItemField.get_display_value() qui est déjà optimisé pour ça.

             elif filter_field.type in ('item', 'items', 'bool', 'string', 'email', 'date'):
                 criterias.append(criteria('f%s' % filter_field.id, filter_field_value, field=filter_field))
                 if filter_field.type in ('item', 'items'):
-                    field_options = filter_field.get_options()
-                    if field_options and type(field_options[0]) in (list, tuple):
-                        for option in field_options:
-                            if filter_field_value in (option[0], option[-1]):
-                                filter_field_value = option[1]
-                                break
-                    criterias[-1]._label = '%s: %s' % (filter_field.label, filter_field_value)
+                    criterias[-1]._label = '%s: %s' % (filter_field.label, filter_field.get_display_value(filter_field_value))

Révisions associées

Révision e1f8d168 (diff)
Ajouté par Benjamin Dauvergne il y a environ un an

api: do not fully load the related table when filtering on an item(s) field (#74320)

Historique

#2

Mis à jour par Benjamin Dauvergne il y a environ un an

  • Assigné à mis à Benjamin Dauvergne
#3

Mis à jour par Robot Gitea il y a environ un an

  • Statut changé de Nouveau à Solution proposée

Benjamin Dauvergne (bdauvergne) a ouvert une pull request sur Gitea concernant cette demande :

#4

Mis à jour par Robot Gitea il y a environ un an

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

Lauréline Guérin (lguerin) a approuvé une pull request sur Gitea concernant cette demande :

#5

Mis à jour par Robot Gitea il y a environ un an

  • Statut changé de Solution validée à Résolu (à déployer)

Benjamin Dauvergne (bdauvergne) a mergé une pull request sur Gitea concernant cette demande :

#6

Mis à jour par Transition automatique il y a environ un an

  • Statut changé de Résolu (à déployer) à Solution déployée
#7

Mis à jour par Transition automatique il y a environ un an

Automatic expiration

Formats disponibles : Atom PDF