summaryrefslogtreecommitdiff
path: root/django/core
diff options
context:
space:
mode:
authorGiannis Terzopoulos <terzo.giannis@gmail.com>2024-03-18 14:50:32 +0100
committerMariusz Felisiak <felisiak.mariusz@gmail.com>2024-03-27 08:14:54 +0100
commitd658a3162fbeb68d148d1b2fcf4da4fe1437eddb (patch)
tree2614270a87d4c178dfca714dc48b075a75c9676b /django/core
parentb98271a6e42107233311d17f5d7bc74fbb47f22c (diff)
Fixed #35233 -- Moved template engine system checks to backend methods.
Thanks Adam Johnson for reviews.
Diffstat (limited to 'django/core')
-rw-r--r--django/core/checks/templates.py75
1 files changed, 6 insertions, 69 deletions
diff --git a/django/core/checks/templates.py b/django/core/checks/templates.py
index 681aa1f317..72a3212e56 100644
--- a/django/core/checks/templates.py
+++ b/django/core/checks/templates.py
@@ -1,75 +1,12 @@
-import copy
-from collections import defaultdict
-
-from django.conf import settings
-from django.template.backends.django import get_template_tag_modules
-
-from . import Error, Tags, Warning, register
-
-E001 = Error(
- "You have 'APP_DIRS': True in your TEMPLATES but also specify 'loaders' "
- "in OPTIONS. Either remove APP_DIRS or remove the 'loaders' option.",
- id="templates.E001",
-)
-E002 = Error(
- "'string_if_invalid' in TEMPLATES OPTIONS must be a string but got: {} ({}).",
- id="templates.E002",
-)
-W003 = Warning(
- "{} is used for multiple template tag modules: {}",
- id="templates.E003",
-)
+from . import Tags, register
@register(Tags.templates)
-def check_setting_app_dirs_loaders(app_configs, **kwargs):
- return (
- [E001]
- if any(
- conf.get("APP_DIRS") and "loaders" in conf.get("OPTIONS", {})
- for conf in settings.TEMPLATES
- )
- else []
- )
-
-
-@register(Tags.templates)
-def check_string_if_invalid_is_string(app_configs, **kwargs):
- errors = []
- for conf in settings.TEMPLATES:
- string_if_invalid = conf.get("OPTIONS", {}).get("string_if_invalid", "")
- if not isinstance(string_if_invalid, str):
- error = copy.copy(E002)
- error.msg = error.msg.format(
- string_if_invalid, type(string_if_invalid).__name__
- )
- errors.append(error)
- return errors
-
+def check_templates(app_configs, **kwargs):
+ """Check all registered template engines."""
+ from django.template import engines
-@register(Tags.templates)
-def check_for_template_tags_with_the_same_name(app_configs, **kwargs):
errors = []
- libraries = defaultdict(set)
-
- for conf in settings.TEMPLATES:
- custom_libraries = conf.get("OPTIONS", {}).get("libraries", {})
- for module_name, module_path in custom_libraries.items():
- libraries[module_name].add(module_path)
-
- for module_name, module_path in get_template_tag_modules():
- libraries[module_name].add(module_path)
-
- for library_name, items in libraries.items():
- if len(items) > 1:
- errors.append(
- Warning(
- W003.msg.format(
- repr(library_name),
- ", ".join(repr(item) for item in sorted(items)),
- ),
- id=W003.id,
- )
- )
-
+ for engine in engines.all():
+ errors.extend(engine.check())
return errors