Project

General

Profile

Bug #90581

dataviz, trace sur filtre avec liste de choix vide

Added by Sentry Io 10 days ago. Updated 7 days ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
14 May 2024
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

Description

Depuis select2, on a plus de choix « Tous les formulaires », donc sur une instance sans formulaire la liste est désormais vide et ça donne cette trace.

https://sentry.entrouvert.org/entrouvert/publik/issues/126106/

IndexError: list index out of range
(33 additional frame(s) were not displayed)
...
  File "combo/apps/dataviz/forms.py", line 240, in __init__
    self.add_filter_fields()
  File "combo/apps/dataviz/forms.py", line 271, in add_filter_fields
    new_fields.update(self.get_filter_fields(self.instance))
  File "combo/apps/dataviz/forms.py", line 102, in get_filter_fields
    field = self.build_choice_field(cell, filter_, initial)
  File "combo/apps/dataviz/forms.py", line 159, in build_choice_field
    choices = self.get_filter_options(cell, filter_, initial)
  File "combo/apps/dataviz/forms.py", line 117, in get_filter_options
    if not isinstance(filter_options[0], list):

Associated revisions

Revision b072d47a (diff)
Added by Valentin Deniaud 10 days ago

dataviz: allow empty filter options (#90581)

History

#1

Updated by Valentin Deniaud 10 days ago

  • Project changed from Suivi des traces to Combo
  • Assignee set to Valentin Deniaud
#2

Updated by Robot Gitea 10 days ago

  • Status changed from Nouveau to Solution proposée

Valentin Deniaud (vdeniaud) a ouvert une pull request sur Gitea concernant cette demande :

#3

Updated by Robot Gitea 10 days ago

  • Status changed from Solution proposée to Solution validée

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

#4

Updated by Robot Gitea 10 days 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 :

#5

Updated by Transition automatique 7 days ago

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

Also available in: Atom PDF