Skip to content

Commit 5acc9ab

Browse files
committed
not using the 'render' shortcut for easier hooking of HTML postprocessing later on
1 parent c981bbf commit 5acc9ab

File tree

1 file changed

+9
-5
lines changed

1 file changed

+9
-5
lines changed

src/py/main/shortcuts.py

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,13 @@
1+
from re import sub
2+
13
from django.conf import settings
2-
from django.http import JsonResponse
3-
from django.shortcuts import render
4+
from django.http import JsonResponse, HttpResponse
45
from django.template import loader
56
from django.utils.translation import gettext_lazy, override as translation_override
67

78
from .models import BaiBrother
89

10+
911
def is_request_for_json(request):
1012
return request.path.startswith('/api/')
1113

@@ -38,16 +40,18 @@ def render_template(request, template, context, language):
3840
context['root_template'] = determine_root_template(request)
3941
context['language_urls'] = list(get_language_links(context, settings.LANGUAGES))
4042

43+
template = loader.get_template(template)
44+
html = template.render(context, request)
45+
4146
if is_request_for_json(request):
42-
template = loader.get_template(template)
4347
response = JsonResponse({
44-
'content': template.render(context, request),
48+
'content': html,
4549
'description': context.get('page', {}).get('description', ''),
4650
'title': '%s | Dimmu Saurus' % context.get('page', {}).get('title', ''),
4751
'urls': context['language_urls']
4852
})
4953
else:
50-
response = render(request, template, context)
54+
response = HttpResponse(html)
5155

5256
response.set_cookie(settings.LANGUAGE_COOKIE_NAME, language)
5357
return response

0 commit comments

Comments
 (0)