preserve_filter doesn't work when running a site with a url prefix

[honyczek]

Function preserve_filter works only on development server at localhost. If you try ./manage.py runserver 192.168.1.1:8000 or if you try your project on production server (Apache HTTPD), it doesn't work. Filters in admin aren't preserved.

Related: #6903

[loic84]

@honyczek it works for me, both in production (uWSGI behind a Nginx reverse proxy) and with the dev server using various combinations of hosts and ports.

Would you be able to upload a minimal project that demonstrates the issue?

[honyczek]

There is one of my projects, where I enabled preserve_filters, which is available to public. ​http://confiback.sourceforge.net/

I'll try it at Nginx. But at Apache with mod_wsgi it doesn't work. Python 2.7.3/Django 1.6 or Python 2.7.5/Django 1.6

[Tim Graham]

@honyczek, it would be really helpful if you could do some more investigation yourself and provide some more details to reproduce the problem. Just saying it "doesn't work" is not a great bug report.

[honyczek]

If I enable preserve_filters parameter in my django.contrib.admin.ModelAdmin instance:

class DeviceAdmin(admin.ModelAdmin):
  list_display = ['name', 'ip', 'profileUrl', 'backupsUrl', 'linkUrl']
  list_filter = ['profile']
  inlines = [KeywordMacroInline]
  actions_on_bottom = True
  actions_on_top = False
  actions = ['manual_backup']
  preserve_filters = True
  
  def linkUrl(self, device):
    if device.url:
      return "<a href=\"%s\" target=\"_blank\">%s</a>" % (device.url, _("Click to view device custom URL"))
    else:
      return "<a>-</a>"
  linkUrl.allow_tags = True
  linkUrl.short_description = _("Custom URL")

  def profileUrl(self, device):
    return "<a href=\"%s\">%s</a>" % (urlresolvers.reverse('admin:frontend_profile_change', args=(device.profile.id,)),device.profile.name)
  profileUrl.allow_tags = True
  profileUrl.short_description = _("Profile")

  def backupsUrl(self, device):
    return "<a href=\"%s?device__id__exact=%s\">%s: %s</a> (<a href=\"%s\">%s</a>)" % (urlresolvers.reverse('admin:frontend_backup_changelist'), device.id, _("Total"), models.Backup.objects.filter(device=device).count(), urlresolvers.reverse('frontend:download_backup_as_file', kwargs={'backup_id': models.Backup.objects.filter(device=device).latest('date').id}), _("Download the latest"))
  backupsUrl.allow_tags = True
  backupsUrl.short_description = _("Backups")

  def manual_backup(modeladmin, request, queryset):
    from django.shortcuts import redirect
    
    _ret = "{}?ids={}".format(urlresolvers.reverse('frontend:run_backup'),"_".join(str(i.id) for i in queryset))
    return redirect(_ret)
  manual_backup.short_description = _("Make manual backup of selected devices")

then I select one of profiles on the right menu to filter list, then open one Device to edit details and then I click to Save button, so Admin returns to Device list with clean filter and displays devices from all profiles (profile filter is empty and All is selected).

What I expect:
After click to Save button, Admin returns to Device list and filter will be still enabled (previously selected profile will remain selected).

[Tim Graham]

Is _changelist_filters being dropped from the query string?

[honyczek]

No, on edit form I have URL: http://server/confiback/admin/frontend/device/89/?_changelist_filters=profile__id__exact%3D21. But after saving it returns to list with URL: http://server/confiback/admin/frontend/device/?e=1 and filter is not applied.

[Tim Graham]

It looks like the changelist_view function in contrib.admin.options is throwing an IncorrectLookupParameters exception. It would be great if you could look into the reason for that. I think the fact that you say it worked on localhost not on other servers is a red herring.

[honyczek]

Ok, I'd like to do it, but I haven't any knowledge how to debug this. Could you give me some recommendations what to use and where to look?

[Tim Graham]

Ping me (timograham) in #django-dev if you like, it will be easier than communicating via this ticket.

[Tim Graham]

We discussed this on #django-dev and determined that the problem is specific to running a site with a URL prefix. In particular, add_preserved_filters raises a Resolver404.

loic84 is looking into it further.

[loic84]

@honyczek could you check if ​https://github.com/loic/django/compare/ticket21795 fixes your issue?

I have a feeling writing a test for this patch is going to be pretty painful.

[loic84]

PR ​https://github.com/django/django/pull/2493.

[honyczek]

Great, it seems to be fixed. Thanks a lot.

[loic84]

@honyczek did you test the patch on the pull request? because it's not merged on master yet, hence why I'm reopening the ticket.

[Tim Graham <timograham@…>]

In 4339e9a92d98371a50a6fe54ab7ea0c602ecee28:

Fixed #21795 -- Made add_preserved_filters account for url prefixes.

Thanks to trac username honyczek for the report. Refs #6903.

[Tim Graham <timograham@…>]

In a5297c1ef4c4c010d1eb979177d4633beb5f9cad:

[1.6.x] Fixed #21795 -- Made add_preserved_filters account for url prefixes.

Thanks to trac username honyczek for the report. Refs #6903.

Backport of 4339e9a92d from master

[Tim Graham <timograham@…>]

In 5268d71f18d12c362d74010210309c1cec8e8a1a:

[1.7.x] Fixed #21795 -- Made add_preserved_filters account for url prefixes.

Thanks to trac username honyczek for the report. Refs #6903.

Backport of 4339e9a92d from master

[honyczek]

Replying to loic84:

@honyczek did you test the patch on the pull request? because it's not merged on master yet, hence why I'm reopening the ticket.

Yes, i did. Thanks.

[loic84]

Cool, thanks @honyczek.