Skip to content

Commit 735a319

Browse files
authored
fix(jinja): remove redundant Jinja environment definitions and use unified setup (#1125)
1 parent 8bbcbd5 commit 735a319

File tree

3 files changed

+4
-41
lines changed

3 files changed

+4
-41
lines changed

app/eventyay/config/settings.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -298,7 +298,7 @@ def instance_name(request):
298298
BASE_DIR / 'jinja-templates',
299299
],
300300
'OPTIONS': {
301-
'environment': 'eventyay.helpers.jinja.environment',
301+
'environment': 'eventyay.jinja.environment',
302302
'extensions': (
303303
'jinja2.ext.i18n',
304304
'jinja2.ext.do',

app/eventyay/helpers/jinja.py

Lines changed: 1 addition & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,6 @@
11
from django.urls import reverse
2-
from django.utils.translation import gettext as django_gettext, ngettext
3-
from django.utils.formats import date_format
4-
from jinja2 import Environment, pass_context
2+
from jinja2 import pass_context
53
from jinja2.runtime import Context
6-
from eventyay.helpers.templatetags.thumb import thumb
74

85

96
@pass_context
@@ -20,39 +17,3 @@ def url_for(context: Context, name: str, *args, query_string=None, **kwargs):
2017
base_url = reverse(name, args=args, kwargs=kwargs, current_app=current_app)
2118
return f"{base_url}?{query_string}" if query_string else base_url
2219

23-
24-
def jinja_gettext(message, **kwargs):
25-
text = django_gettext(message)
26-
if kwargs:
27-
try:
28-
text = text % kwargs
29-
except Exception:
30-
pass
31-
return text
32-
33-
34-
def format_date(value, format_str="SHORT_DATE_FORMAT"):
35-
if not value:
36-
return ""
37-
try:
38-
return date_format(value, format_str)
39-
except Exception:
40-
return str(value)
41-
42-
43-
def environment(**options):
44-
env = Environment(**options, autoescape=True)
45-
46-
env.globals.update({
47-
'url_for': url_for,
48-
'_': jinja_gettext,
49-
'gettext': jinja_gettext,
50-
'ngettext': ngettext,
51-
})
52-
53-
env.filters.update({
54-
'thumb': thumb,
55-
'format_date': format_date,
56-
})
57-
58-
return env

app/eventyay/jinja.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,13 +4,15 @@
44
from jinja2 import Environment
55

66
from .helpers.jinja import url_for
7+
from eventyay.helpers.templatetags.thumb import thumb
78

89
jj_globals = {
910
'url_for': url_for,
1011
'site_url': settings.SITE_URL,
1112
}
1213

1314
jj_filters = {
15+
'thumb': thumb,
1416
'format_date': date,
1517
}
1618

0 commit comments

Comments
 (0)