Welcome to the Treehouse Community

Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.

Looking to learn something new?

Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.

Start your free trial

Python Customizing the Django Admin Customizing the List View Building Custom Filters

Chris Komaroff
PLUS
Chris Komaroff
Courses Plus Student 14,198 Points
Posted by Chris Komaroff
Chris Komaroff
Courses Plus Student 14,198 Points

Test: Custom Filter - step 3 of 4

Hi all, I just figured out how to pass 3 of 4. The following code for queryset() method passed. I think this is weird. My testing shows self.value() case must be identical to the lookup tuple value case, i.e. 'Python' != 'python', so this code would not work in real world? If someone can explain why this code should work let me know, but it crashes for me (but it passes the code challenge 3 of 4).

class TopicListFilter(admin.SimpleListFilter):
    title = 'topic'
    parameter_name = 'topic'

    def lookups(self, request, model_admin):
        return (
            ('Python', 'Python'),
            ('Ruby', 'Ruby'),
            ('Java', 'Java'),
        )

    def queryset(self, request, queryset):
        if self.value() == 'python':
            return queryset.filter(title__contains='python')
        if self.value() == 'ruby':
            return queryset.filter(title__contains='ruby')
        if self.value() == 'java':
            return queryset.filter(title__contains='java')
David Lin
David Lin
35,864 Points
David Lin
David Lin
35,864 Points

My hunch is that there's typos in the answer for this question, that the values in queryset really should match the lookups as you said.

Thanks for posting your solution ... it saved me a lot of aggravation!

3 Answers

Chris Komaroff
PLUS
Chris Komaroff
Courses Plus Student 14,198 Points
Chris Komaroff
Chris Komaroff
Courses Plus Student 14,198 Points

My question above contains my answer, but if I am mistaken let me know.

Ian Salmon
PLUS
Ian Salmon
Courses Plus Student 10,687 Points
Ian Salmon
Ian Salmon
Courses Plus Student 10,687 Points

ARGGGG... I try to work on these challenges for at least an hour before looking up answers. This has me thinking twice about that strategy..

Thank you for posting this.

SHAME on you TreeHouse for still not correcting it 2 years later! I don't mind looking up the differences b/w Django 2 and the video's outdated version, but this is not cool.