Author: jezdez
Date: 2010-02-16 06:13:23 -0600 (Tue, 16 Feb 2010)
New Revision: 12441
Added:
django/trunk/tests/regressiontests/makemessages/
django/trunk/tests/regressiontests/makemessages/__init__.py
django/trunk/tests/regressiontests/makemessages/javascript.js
django/trunk/tests/regressiontests/makemessages/locale/
django/trunk/tests/regressiontests/makemessages/locale/dummy
django/trunk/tests/regressiontests/makemessages/models.py
django/trunk/tests/regressiontests/makemessages/tests.py
Modified:
django/trunk/django/core/management/commands/makemessages.py
Log:
Fixed #4695 - Worked around a problem of xgettext ignoring some translation
strings in JavaScript files. Thanks, Ramiro Morales.
Modified: django/trunk/django/core/management/commands/makemessages.py
===================================================================
--- django/trunk/django/core/management/commands/makemessages.py
2010-02-16 12:12:53 UTC (rev 12440)
+++ django/trunk/django/core/management/commands/makemessages.py
2010-02-16 12:13:23 UTC (rev 12441)
@@ -9,7 +9,7 @@
from django.core.management.base import CommandError, BaseCommand
from django.utils.text import get_text_list
-pythonize_re = re.compile(r'\n\s*//')
+pythonize_re = re.compile(r'(?:^|\n)\s*//')
def handle_extensions(extensions=('html',)):
"""
Added: django/trunk/tests/regressiontests/makemessages/__init__.py
===================================================================
Added: django/trunk/tests/regressiontests/makemessages/javascript.js
===================================================================
--- django/trunk/tests/regressiontests/makemessages/javascript.js
(rev 0)
+++ django/trunk/tests/regressiontests/makemessages/javascript.js
2010-02-16 12:13:23 UTC (rev 12441)
@@ -0,0 +1,4 @@
+// '
+gettext('This literal should be included.')
+// '
+gettext('This one as well.')
Added: django/trunk/tests/regressiontests/makemessages/locale/dummy
===================================================================
Added: django/trunk/tests/regressiontests/makemessages/models.py
===================================================================
Added: django/trunk/tests/regressiontests/makemessages/tests.py
===================================================================
--- django/trunk/tests/regressiontests/makemessages/tests.py
(rev 0)
+++ django/trunk/tests/regressiontests/makemessages/tests.py 2010-02-16
12:13:23 UTC (rev 12441)
@@ -0,0 +1,42 @@
+import os
+import re
+import shutil
+from django.test import TestCase
+from django.core import management
+
+LOCALE='de'
+
+class ExtractorTests(TestCase):
+
+ def setUp(self):
+ self._cwd = os.getcwd()
+ self.test_dir = os.path.abspath(os.path.dirname(__file__))
+
+ def _rmrf(self, dname):
+ if os.path.commonprefix([self.test_dir, os.path.abspath(dname)]) !=
self.test_dir:
+ return
+ shutil.rmtree(dname)
+
+ def tearDown(self):
+ os.chdir(self.test_dir)
+ try:
+ self._rmrf('locale/%s' % LOCALE)
+ except OSError:
+ pass
+ os.chdir(self._cwd)
+
+ def assertMsgId(self, msgid, s):
+ return self.assert_(re.search('^msgid "%s"' % msgid, s, re.MULTILINE))
+
+
+class JavascriptExtractorTests(ExtractorTests):
+
+ PO_FILE='locale/%s/LC_MESSAGES/djangojs.po' % LOCALE
+
+ def test_javascript_literals(self):
+ os.chdir(self.test_dir)
+ management.call_command('makemessages', domain='djangojs',
locale=LOCALE, verbosity=0)
+ self.assert_(os.path.exists(self.PO_FILE))
+ po_contents = open(self.PO_FILE, 'r').read()
+ self.assertMsgId('This literal should be included.', po_contents)
+ self.assertMsgId('This one as well.', po_contents)
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en.
