swev-id: django__django-16612 — Preserve query string in AdminSite.catch_all_view redirect

[casey-brooks]

Fixes #507

Reproduction

1. .venv/bin/python tests/runtests.py admin_views.test_adminsite.SiteAppendSlashRedirectTests.test_catch_all_preserves_querystring_on_slash_redirect

Observed failure:

FAIL: test_catch_all_preserves_querystring_on_slash_redirect (admin_views.test_adminsite.SiteAppendSlashRedirectTests.test_catch_all_preserves_querystring_on_slash_redirect)
----------------------------------------------------------------------
Traceback (most recent call last):
  File \"/workspace/django/tests/admin_views/test_adminsite.py\", line 127, in test_catch_all_preserves_querystring_on_slash_redirect
    self.assertEqual(resp[\"Location\"], \"/test_admin/admin/auth/?id=123\")
AssertionError: '/test_admin/admin/auth/' != '/test_admin/admin/auth/?id=123'

Fix

• Redirect using request.get_full_path(force_append_slash=True) so the querystring is preserved.
• Add a regression test enforcing the behavior when APPEND_SLASH is True.
• Regression was introduced in ba31b01.

Testing

• .venv/bin/python tests/runtests.py admin_views.test_adminsite.SiteAppendSlashRedirectTests.test_catch_all_preserves_querystring_on_slash_redirect

[casey-brooks]

Test & Lint Summary

• .venv/bin/python tests/runtests.py admin_views.test_adminsite.SiteAppendSlashRedirectTests.test_catch_all_preserves_querystring_on_slash_redirect: 1 passed, 0 failed, 0 skipped
• .venv/bin/flake8 django/contrib/admin/sites.py tests/admin_views/test_adminsite.py: no lint issues

[noa-lucent]

Looks good. The redirect now uses request.get_full_path(force_append_slash=True), which preserves query strings while still normalizing the trailing slash, and the regression test covers the failure case from the issue.

[noa-lucent]

Looks good