Ticket #13881: 13881.tests_and_fix.patch

django/contrib/sitemaps/__init__.py

@@ -65 +65 @@
         urls = []
         for item in self.paginator.page(page).object_list:
             loc = "http://%s%s" % (current_site.domain, self.__get('location', item))
+            priority = self.__get('priority', item, None)
+            if priority is not None:
+                priority = u"%f" % priority
+                priority = priority.rstrip(u'0')
             url_info = {
                 'location':   loc,
                 'lastmod':    self.__get('lastmod', item, None),
                 'changefreq': self.__get('changefreq', item, None),
-                'priority':   self.__get('priority', item, None)
+                'priority':   priority
             }
             urls.append(url_info)
         return urls

tests/regressiontests/sitemaps/__init__.py

@@ +1 @@
+# empty.

tests/regressiontests/sitemaps/models.py

@@ +1 @@
+# empty.

tests/regressiontests/sitemaps/tests.py

@@ +1 @@
+# -*- coding: utf-8 -*-
+from datetime import datetime
+from django.conf import settings
+from django.utils import translation
+from django.test import TestCase
+from django.contrib.sitemaps import Sitemap
+
+
+class Page(object):
+    def get_absolute_url(self):
+        return '/'
+
+
+class PageSitemap(Sitemap):
+    changefreq = 'weekly'
+    priority = 0.5
+
+    def items(self):
+        return [Page(), Page(), Page()]
+
+    def lastmod(self, obj):
+        return datetime.now()
+
+
+class TestSitemap(TestCase):
+    urls = 'regressiontests.sitemaps.urls'
+
+    def test_localized_priority(self):
+        '''
+        Issue 13881: Priority's decimal separator is localized but shouldn't.
+        '''
+        org_l10n, settings.USE_L10N = settings.USE_L10N, True
+        org_lang = translation.get_language()
+        settings.INSTALLED_APPS += ('django.contrib.sitemaps',)
+        try:
+            translation.activate('ru')
+            sitemap = PageSitemap()
+            response = self.client.get('/sitemap-page.xml')
+            self.assertNotContains(response, '<priority>0,5</priority>')
+            self.assertContains(response, '<priority>0.5</priority>')
+        finally:
+            settings.L10N = org_l10n
+            settings.INSTALLED_APPS = settings.INSTALLED_APPS[:-1]
+            translation.activate(org_lang)

tests/regressiontests/sitemaps/urls.py

@@ +1 @@
+from django.conf.urls.defaults import handler404, handler500, patterns, url
+from regressiontests.sitemaps.tests import PageSitemap
+
+sitemaps = {
+    'page': PageSitemap(),
+}
+
+urlpatterns = patterns('django.contrib.sitemaps.views',
+    url(r'^sitemap-(?P<section>\w+).xml$', 'sitemap', {'sitemaps': sitemaps}),
+)