Ticket #16721: 16721.diff

django/templatetags/i18n.py

@@ -110 +110 @@
         # the end of function
         context.update(tmp_context)
         singular, vars = self.render_token_list(self.singular)
+        # Escape all isolated '%'
+        singular = re.sub(u'%(?!\()', u'%%', singular)
         if self.plural and self.countervar and self.counter:
             count = self.counter.resolve(context)
             context[self.countervar] = count
             plural, plural_vars = self.render_token_list(self.plural)
+            plural = re.sub(u'%(?!\()', u'%%', plural)
             result = translation.ungettext(singular, plural, count)
             vars.extend(plural_vars)
         else:
             result = translation.ugettext(singular)
-        # Escape all isolated '%' before substituting in the context.
-        result = re.sub(u'%(?!\()', u'%%', result)
         data = dict([(v, _render_value_in_context(context.get(v, ''), context)) for v in vars])
         context.pop()
         return result % data

tests/regressiontests/i18n/other/locale/de/LC_MESSAGES/django.po

@@ -40 +40 @@
 msgid_plural "%d results"
 msgstr[0] "%d Resultat"
 msgstr[1] "%d Resultate"
+
+#: models.py:13
+#, python-format
+msgid "The result was %(percent)s%%"
+msgstr "Das Ergebnis war %(percent)s%%"
+
+#: models.py:13
+#, python-format
+msgid "%(percent)s%% represents %(num)s object"
+msgid_plural "%(percent)s%% represents %(num)s objects"
+msgstr[0] "%(percent)s%% darstellt %(num)s Objekt"
+msgstr[1] "%(percent)s%% darstellt %(num)s Objekte"

tests/regressiontests/i18n/tests.py

@@ -600 +600 @@
         r.META = {'HTTP_ACCEPT_LANGUAGE': 'de'}
         self.assertEqual(g(r), 'zh-cn')
 
+    def test_percent_in_translatable_block(self):
+        extended_locale_paths = settings.LOCALE_PATHS + (
+            os.path.join(os.path.dirname(os.path.abspath(__file__)), 'other', 'locale'),
+        )
+        with self.settings(LOCALE_PATHS=extended_locale_paths):
+            t_sing = Template("{% load i18n %}{% blocktrans %}The result was {{ percent }}%{% endblocktrans %}")
+            t_plur = Template("{% load i18n %}{% blocktrans count num as number %}{{ percent }}% represents {{ num }} object{% plural %}{{ percent }}% represents {{ num }} objects{% endblocktrans %}")
+            with translation.override('de'):
+                self.assertEqual(t_sing.render(Context({'percent': 42})), u'Das Ergebnis war 42%')
+                self.assertEqual(t_plur.render(Context({'percent': 42, 'num': 1})), u'42% darstellt 1 Objekt')
+                self.assertEqual(t_plur.render(Context({'percent': 42, 'num': 4})), u'42% darstellt 4 Objekte')
+
+
 class ResolutionOrderI18NTests(TestCase):
 
     def setUp(self):