diff options
| author | Giannis Terzopoulos <terzo.giannis@gmail.com> | 2024-03-18 14:50:32 +0100 |
|---|---|---|
| committer | Mariusz Felisiak <felisiak.mariusz@gmail.com> | 2024-03-27 08:14:54 +0100 |
| commit | d658a3162fbeb68d148d1b2fcf4da4fe1437eddb (patch) | |
| tree | 2614270a87d4c178dfca714dc48b075a75c9676b /django/core | |
| parent | b98271a6e42107233311d17f5d7bc74fbb47f22c (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.py | 75 |
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 |
