Fixed #16516 -- Relaxed the blocktrans rendering a little by falling back to the default language if resolving one of the arguments fails, raising a KeyError. Thanks, Claude Paroz and Aymeric Augustin.

git-svn-id: http://code.djangoproject.com/svn/django/trunk@16723 bcc190cf-cafb-0310-a4f2-bffc1f526a37

3 files changed, 41 insertions, 2 deletions

diff --git a/tests/regressiontests/i18n/other/locale/fr/LC_MESSAGES/django.mo b/tests/regressiontests/i18n/other/locale/fr/LC_MESSAGES/django.mo
new file mode 100644
index 0000000000..f0a2179767
--- /dev/null
+++ b/tests/regressiontests/i18n/other/locale/fr/LC_MESSAGES/django.mo
diff --git a/tests/regressiontests/i18n/other/locale/fr/LC_MESSAGES/django.po b/tests/regressiontests/i18n/other/locale/fr/LC_MESSAGES/django.po
new file mode 100644
index 0000000000..8e9b9e1034
--- /dev/null
+++ b/tests/regressiontests/i18n/other/locale/fr/LC_MESSAGES/django.po
@@ -0,0 +1,21 @@
+# SOME DESCRIPTIVE TITLE.
+# Copyright (C) YEAR THE PACKAGE'S COPYRIGHT HOLDER
+# This file is distributed under the same license as the PACKAGE package.
+# FIRST AUTHOR <EMAIL@ADDRESS>, YEAR.
+#
+msgid ""
+msgstr ""
+"Project-Id-Version: django tests\n"
+"Report-Msgid-Bugs-To: \n"
+"POT-Creation-Date: 2010-02-14 17:33+0100\n"
+"PO-Revision-Date: 2011-01-21 21:37-0300\n"
+"Last-Translator: Claude\n"
+"Language-Team: fr <fr@li.org>\n"
+"MIME-Version: 1.0\n"
+"Content-Type: text/plain; charset=UTF-8\n"
+"Content-Transfer-Encoding: 8bit\n"
+"Plural-Forms: nplurals=2; plural=(n != 1)\n"
+
+#: template.html:3
+msgid "My name is %(person)s."
+msgstr "Mon nom est %(personne)s."
diff --git a/tests/regressiontests/i18n/tests.py b/tests/regressiontests/i18n/tests.py
index 0b955ad3da..f230e59556 100644
--- a/tests/regressiontests/i18n/tests.py
+++ b/tests/regressiontests/i18n/tests.py
@@ -9,6 +9,7 @@ from threading import local
 from django.conf import settings
 from django.template import Template, Context
 from django.test import TestCase, RequestFactory
+from django.test.utils import override_settings
 from django.utils.formats import (get_format, date_format, time_format,
     localize, localize_input, iter_format_modules, get_format_modules)
 from django.utils.importlib import import_module
@@ -27,6 +28,8 @@ from commands.tests import *
 from patterns.tests import *
 from test_warnings import DeprecationWarningTests
 
+here = os.path.dirname(os.path.abspath(__file__))
+
 class TranslationTests(TestCase):
 
     def test_override(self):
@@ -34,6 +37,9 @@ class TranslationTests(TestCase):
         with translation.override('pl'):
             self.assertEqual(get_language(), 'pl')
         self.assertEqual(get_language(), 'de')
+        with translation.override(None):
+            self.assertEqual(get_language(), settings.LANGUAGE_CODE)
+        self.assertEqual(get_language(), 'de')
         deactivate()
 
     def test_lazy_objects(self):
@@ -67,7 +73,7 @@ class TranslationTests(TestCase):
     def test_pgettext(self):
         # Reset translation catalog to include other/locale/de
         extended_locale_paths = settings.LOCALE_PATHS + (
-            os.path.join(os.path.dirname(os.path.abspath(__file__)), 'other', 'locale'),
+            os.path.join(here, 'other', 'locale'),
         )
         with self.settings(LOCALE_PATHS=extended_locale_paths):
             from django.utils.translation import trans_real
@@ -129,6 +135,18 @@ class TranslationTests(TestCase):
         self.assertEqual(to_language('en_US'), 'en-us')
         self.assertEqual(to_language('sr_Lat'), 'sr-lat')
 
+    @override_settings(LOCALE_PATHS=(os.path.join(here, 'other', 'locale'),))
+    def test_bad_placeholder(self):
+        """
+        Error in translation file should not crash template rendering
+        (%(person)s is translated as %(personne)s in fr.po)
+        """
+        from django.template import Template, Context
+        with translation.override('fr'):
+            t = Template('{% load i18n %}{% blocktrans %}My name is {{ person }}.{% endblocktrans %}')
+            rendered = t.render(Context({'person': 'James'}))
+            self.assertEqual(rendered, 'My name is James.')
+
 
 class FormattingTests(TestCase):
 
@@ -636,7 +654,7 @@ class LocalePathsResolutionOrderI18NTests(ResolutionOrderI18NTests):
 
     def setUp(self):
         self.old_locale_paths = settings.LOCALE_PATHS
-        settings.LOCALE_PATHS += (os.path.join(os.path.dirname(os.path.abspath(__file__)), 'other', 'locale'),)
+        settings.LOCALE_PATHS += (os.path.join(here, 'other', 'locale'),)
         super(LocalePathsResolutionOrderI18NTests, self).setUp()
 
     def tearDown(self):