Merge branch 'karlrl-support-new-python-and-django'

Author: mjumbewu

.travis.yml

@@ -1,10 +1,11 @@
 language: python
 python:
-  - "2.7"
-  - "3.4"
+  - "3.7"
+  - "3.8"
+  - "3.9"
+  - "3.10"
 
 env:
-  - DJANGO="django<1.9"  # 1.8.x
   - DJANGO="django<3"  # 2.2.x
   - DJANGO="django<4"  # 3.2.x
   - DJANGO="git+git://github.com/django/django.git@master#egg=django"

CHANGES.rst

@@ -1,6 +1,14 @@
 CHANGES
 =======
 
+2.0.0
+-----
+
+Thank you to @karlrl for the following changes:
+- Support Python 3.7-3.10
+- Support Django 2.2-4.1
+- Drop support for Python<3.7 and Django<3.2
+
 1.3.9
 -----
 

jstemplate/__init__.py

@@ -1 +1 @@
-__version__ = "1.3.9"
+__version__ = "2.0.0"

jstemplate/finders.py

@@ -1,15 +1,13 @@
-from __future__ import unicode_literals
-
+import glob
+import os
+import re
+import sys
 import warnings
-import glob, os, sys, re
-import six
-
 from importlib import import_module
 
 from .conf import conf
 
 
-
 class BaseFinder(object):
     def find(self, name):
         raise NotImplementedError()
@@ -115,8 +113,6 @@ def _get_app_template_dirs():
         for dirname in conf.JSTEMPLATE_APP_DIRNAMES:
             template_dir = os.path.join(app_dir, dirname)
             if os.path.isdir(template_dir):
-                if not six.PY3:
-                    template_dir = template_dir.decode(fs_encoding)
                 ret.append(template_dir)
     return ret
 

jstemplate/loading.py

@@ -1,13 +1,9 @@
-from __future__ import unicode_literals
-
-import six
 from django.core.exceptions import ImproperlyConfigured
 from importlib import import_module
 
 from .conf import conf
 
 
-
 def find(name):
     all_matches = {}
 
@@ -16,7 +12,7 @@ def find(name):
 
         # <finder>.find may return a single string.  The name of the template
         # will then be the name given to 'find'
-        if isinstance(matches, six.text_type):
+        if isinstance(matches, str):
             filepath = matches
             if name not in all_matches:
                 all_matches[name] = filepath

jstemplate/preprocessors.py

@@ -1,9 +1,8 @@
-from __future__ import unicode_literals
-
 import re
-from django.utils.translation import ugettext
+from django.utils.translation import gettext
 from .conf import conf
 
+
 class I18nPreprocessor(object):
     @property
     def tagnames(self):
@@ -13,7 +12,7 @@ def tagnames(self):
     def short_trans_re(self):
         # Should match strings like: {{ _ "Hello, world! }}
         tagnames = '|'.join(['(?:{0})'.format(t) for t in self.tagnames])
-        
+
         left_side = r'''(?P<left>\{\{\s*(?P<tag>(?:''' + tagnames + r''')\s+)(?P<quote>['"]))'''
         right_side = r'''(?P<right>(?P=quote)\s*\}\})'''
 
@@ -23,7 +22,7 @@ def short_trans_re(self):
     def long_trans_re(self):
         # Should match strings like: {{# _ }}Hello, {{ name }}.{{/ _ }}
         tagnames = '|'.join(['(?:{0})'.format(t) for t in self.tagnames])
-        
+
         start_tag = r'\{\{#\s*(?P<tag>' + tagnames + r')\s*\}\}'
         end_tag = r'\{\{\/\s*(?P=tag)\s*\}\}'
 
@@ -33,14 +32,14 @@ def translate_short_form(self, match):
         """Translate a result of matching the compiled trans_re pattern."""
         tag = match.group('tag')
         msg = match.group('msg')
-        msg = ugettext(msg) if len(msg) > 0 else ''
+        msg = gettext(msg) if len(msg) > 0 else ''
         string = match.group('left').replace(tag, '', 1) + msg + match.group('right')
         return string
 
     def translate_long_form(self, match):
         """Translate a result of matching the compiled trans_re pattern."""
         msg = match.group('msg')
-        string = ugettext(msg) if len(msg) > 0 else ''
+        string = gettext(msg) if len(msg) > 0 else ''
         return string
 
     def process(self, content):

jstemplate/templatetags/base.py

@@ -1,6 +1,3 @@
-from __future__ import unicode_literals
-
-import six
 from django import template
 from ..conf import conf
 from ..loading import find, preprocess, JSTemplateNotFound
@@ -49,13 +46,8 @@ def read_template_file_contents(self, filepath):
         # Django 3.1, UTF-8 is always expected.
         encoding = getattr(conf, 'FILE_CHARSET', 'utf-8')
 
-        if six.PY3:
-            with open(filepath, "r", encoding=encoding) as fp:
-                template_text = fp.read()
-        else:
-            with open(filepath, "r") as fp:
-                template_text = fp.read().decode(encoding)
-
+        with open(filepath, "r", encoding=encoding) as fp:
+            template_text = fp.read()
         template_text = self.preprocess(template_text)
         return template_text
 
@@ -75,4 +67,4 @@ def jstemplate_tag_helper(tagname, TagNodeClass, parser, token):
             "'%s' tag takes at least one argument: the name/id of "
             "the template, or a pattern matching a set of templates. "
             % tagname)
-    return TagNodeClass(*bits[1:])
+    return TagNodeClass(*bits[1:])

jstemplate/templatetags/handlebarsjs.py

@@ -1,6 +1,4 @@
-from __future__ import unicode_literals
 from django import template
-from ..conf import conf
 from .base import BaseJSTemplateNode, jstemplate_tag_helper
 
 
@@ -49,7 +47,7 @@ def handlebarsjs(parser, token):
     Finds the Handlebars template for the given name and renders it surrounded
     by the requisite Handlebars <script> tags.
 
-    We don't use the jstemplate_tag_helper here, since we can take an 
+    We don't use the jstemplate_tag_helper here, since we can take an
     additional parameter denoting whether to register partials inline.
 
     """

jstemplate/tests/project/project/urls.py

@@ -1,17 +1 @@
-from django.conf.urls import *
-
-# Uncomment the next two lines to enable the admin:
-# from django.contrib import admin
-# admin.autodiscover()
-
-urlpatterns = [
-    # Examples:
-    # url(r'^$', 'project.views.home', name='home'),
-    # url(r'^project/', include('project.foo.urls')),
-
-    # Uncomment the admin/doc line below to enable admin documentation:
-    # url(r'^admin/doc/', include('django.contrib.admindocs.urls')),
-
-    # Uncomment the next line to enable the admin:
-    # url(r'^admin/', include(admin.site.urls)),
-]
+urlpatterns = []

jstemplate/tests/project/project/views.py

@@ -1,4 +1,4 @@
-from django.utils.translation import ugettext as _
+from django.utils.translation import gettext as _
 
 def f():
     return _('View message')