Context Navigation

Ticket #19488: 19488-1.diff

File 19488-1.diff, 2.7 KB (added by Claude Paroz, 12 years ago)

diff --git a/django/middleware/locale.py b/django/middleware/locale.py
index 628782b..f693e4d 100644

                class LocaleMiddleware(object):
                     and self.is_language_prefix_patterns_used()):
             urlconf = getattr(request, 'urlconf', None)
             language_path = '/%s%s' % (language, request.path_info)
+            if settings.APPEND_SLASH and not language_path.endswith('/'):
+                language_path = language_path + '/'
+            path_valid = is_valid_path(language_path, urlconf)
+            if (not path_valid and settings.APPEND_SLASH
+                    and not language_path.endswith('/')):
+                path_valid = is_valid_path("%s/" % language_path, urlconf)
             if is_valid_path(language_path, urlconf):
+            if path_valid:
                 language_url = "%s://%s/%s%s" % (
                     request.is_secure() and 'https' or 'http',
                     request.get_host(), language, request.get_full_path())

diff --git a/tests/regressiontests/i18n/patterns/tests.py b/tests/regressiontests/i18n/patterns/tests.py
index 358cdf6..639e03f 100644

                class URLTranslationTests(URLTestCaseBase):
         with translation.override('nl'):
             self.assertEqual(reverse('users'), '/nl/gebruikers/')
+            self.assertEqual(reverse('prefixed_xml'), '/nl/prefixed.xml')
         with translation.override('pt-br'):
             self.assertEqual(reverse('users'), '/pt-br/usuarios/')
-…
+               class URLRedirectWithoutTrailingSlashTests(URLTestCaseBase):
         self.assertIn(('http://testserver/en/account/register/', 301), response.redirect_chain)
         self.assertRedirects(response, '/en/account/register/', 302)
+        response = self.client.get('/prefixed.xml', HTTP_ACCEPT_LANGUAGE='en', follow=True)
+        self.assertRedirects(response, '/en/prefixed.xml', 302)
 class URLRedirectWithoutTrailingSlashSettingTests(URLTestCaseBase):
     """

diff --git a/tests/regressiontests/i18n/patterns/urls/default.py b/tests/regressiontests/i18n/patterns/urls/default.py
index f117502..00b90b1 100644

                urlpatterns = patterns('',
 urlpatterns += i18n_patterns('',
     url(r'^prefixed/$', view, name='prefixed'),
+    url(r'^prefixed\.xml$', view, name='prefixed_xml'),
     url(_(r'^users/$'), view, name='users'),
     url(_(r'^account/'), include('regressiontests.i18n.patterns.urls.namespace', namespace='account')),
+)