Ticket #500: optional_template_file_extensions.3.diff

django/conf/global_settings.py

@@ -91 +91 @@
 # List of locations of the template source files, in search order.
 TEMPLATE_DIRS = ()
 
-# Extension on all templates.
+# Default extension for templates if none is given.
 TEMPLATE_FILE_EXTENSION = '.html'
 
+# File extension for admin templates.
+ADMIN_TEMPLATE_FILE_EXTENSION = '.html'
+
 # List of callables that know how to import templates from various sources.
 # See the comments in django/core/template/loader.py for interface
 # documentation.

django/core/template/loaders/app_directories.py

@@ -1 +1 @@
 # Wrapper for loading templates from "template" directories in installed app packages.
 
-from django.conf.settings import INSTALLED_APPS, TEMPLATE_FILE_EXTENSION
+from django.conf.settings import INSTALLED_APPS, TEMPLATE_FILE_EXTENSION, ADMIN_TEMPLATE_FILE_EXTENSION
 from django.core.exceptions import ImproperlyConfigured
 from django.core.template import TemplateDoesNotExist
 import os
@@ -28 +28 @@
 app_template_dirs = tuple(app_template_dirs)
 
 def get_template_sources(template_name, template_dirs=None):
+    admin_template_name = None
+    # Only append file extension if there isn't an extension already.
+    if not os.path.splitext(template_name)[-1]:
+        if TEMPLATE_FILE_EXTENSION != ADMIN_TEMPLATE_FILE_EXTENSION:
+            admin_template_name = template_name + TEMPLATE_FILE_EXTENSION
+        template_name = template_name + TEMPLATE_FILE_EXTENSION
     for template_dir in app_template_dirs:
-        yield os.path.join(template_dir, template_name) + TEMPLATE_FILE_EXTENSION
+        yield os.path.join(template_dir, template_name)
+        if admin_template_name:
+            yield os.path.join(template_dir, admin_template_name)
 
 def load_template_source(template_name, template_dirs=None):
     for filepath in get_template_sources(template_name, template_dirs):

django/core/template/loaders/eggs.py

@@ -6 +6 @@
     resource_string = None
 
 from django.core.template import TemplateDoesNotExist
-from django.conf.settings import INSTALLED_APPS, TEMPLATE_FILE_EXTENSION
+from django.conf.settings import INSTALLED_APPS, TEMPLATE_FILE_EXTENSION, ADMIN_TEMPLATE_FILE_EXTENSION
+import os
 
 def load_template_source(template_name, template_dirs=None):
     """
@@ -15 +16 @@
     For every installed app, it tries to get the resource (app, template_name).
     """
     if resource_string is not None:
-        pkg_name = 'templates/' + template_name + TEMPLATE_FILE_EXTENSION
+        admin_pkg_name = None
+        # Only append TEMPLATE_FILE_EXTENSION if there isn't an extension already.
+        if not os.path.splitext(template_name)[-1]:
+            if TEMPLATE_FILE_EXTENSION != ADMIN_TEMPLATE_FILE_EXTENSION:
+                admin_pkg_name = 'templates/' + template_name + TEMPLATE_FILE_EXTENSION
+            template_name = template_name + TEMPLATE_FILE_EXTENSION
+        pkg_name = 'templates/' + template_name
         for app in INSTALLED_APPS:
             try:
                 return (resource_string(app, pkg_name), 'egg:%s:%s ' % (app, pkg_name))
             except:
                 pass
+            if admin_pkg_name:
+                try:
+                    return (resource_string(app, admin_pkg_name), 'egg:%s:%s ' % (app, admin_pkg_name))
+                except:
+                    pass
     raise TemplateDoesNotExist, template_name
 load_template_source.is_usable = resource_string is not None

django/core/template/loaders/filesystem.py

@@ -1 +1 @@
 # Wrapper for loading templates from the filesystem.
 
-from django.conf.settings import TEMPLATE_DIRS, TEMPLATE_FILE_EXTENSION
+from django.conf.settings import TEMPLATE_DIRS, TEMPLATE_FILE_EXTENSION, ADMIN_TEMPLATE_FILE_EXTENSION
 from django.core.template import TemplateDoesNotExist
 import os
 
 def get_template_sources(template_name, template_dirs=None):
     if not template_dirs:
         template_dirs = TEMPLATE_DIRS
+    admin_template_name = None
+    # Only append TEMPLATE_FILE_EXTENSION if there isn't an extension already.
+    if not os.path.splitext(template_name)[-1]:
+        if TEMPLATE_FILE_EXTENSION != ADMIN_TEMPLATE_FILE_EXTENSION:
+            admin_template_name = template_name + TEMPLATE_FILE_EXTENSION
+        template_name = template_name + TEMPLATE_FILE_EXTENSION
     for template_dir in template_dirs:
-        yield os.path.join(template_dir, template_name) + TEMPLATE_FILE_EXTENSION
+        yield os.path.join(template_dir, template_name)
+        if admin_template_name:
+            yield os.path.join(template_dir, admin_template_name)
 
 def load_template_source(template_name, template_dirs=None):
     tried = []