django-grappelli icon indicating copy to clipboard operation
django-grappelli copied to clipboard

`change_list_filter_confirm` is using `admin_list.admin_list_filter` instead of `grp_tags.admin_list_filter`

Open pricco opened this issue 3 years ago • 3 comments

Packages: Django==3.2.2 django-grappelli==2.15.1

The current change_list_filter_confirm contains: {% load i18n grp_tags admin_urls static admin_list %}

Django is using the admin_list_filter template tag from Django's admin_list instead of Grapellis grp_tags`.

Looks like changing the order in the load tag fix the issue: {% load i18n admin_urls static admin_list grp_tags %}

pricco avatar May 19 '21 06:05 pricco

@pricco I'm sorry, but I don't understand what the issue is exactly. What's the unwanted behaviour and how am I able to reproduce it?

sehmaschine avatar Jul 12 '21 15:07 sehmaschine

closing this since no further details have been given.

sehmaschine avatar Oct 07 '21 12:10 sehmaschine

Hi @sehmaschine,

change_list_filter_confirm currently loads grp_tags before admin_list, which result in the template to use admin_list_filter method of admin_list instead of grp_tags. As a result, the rendered filters always have empty data-field-name attribute causing the following claim to be not true:

This template supports use cases where multiple filters have to be selected and applied at the same time.

sutris avatar Apr 05 '22 06:04 sutris

solved with 7eeb284

sehmaschine avatar Mar 20 '23 13:03 sehmaschine