Fix #19318: Admin's SimpleListFilter option not being displayed as selected if the lookup's first element is not a string #539

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants
@sebasmagri
Contributor

sebasmagri commented Nov 19, 2012

This pull request contains the fix for 1.4.x.

The lookup value is coerced to unicode by using force_unicode

sebasmagri added some commits Nov 19, 2012

Coerce lookup value to unicode in SimpleListFilter.choices
    This fixes #19318, where the filter option might not have set
    its selected class when active if using integers as option
    value
@jphalip

This comment has been minimized.

Show comment Hide comment
@jphalip

jphalip Dec 4, 2012

Member

Fixed in c721722. Thanks!

Member

jphalip commented Dec 4, 2012

Fixed in c721722. Thanks!

@jphalip jphalip closed this Dec 4, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment