Ticket #500: optional_template_file_extensions.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'
 
 # List of callables that know how to import templates from various sources.

django/core/template/loaders/app_directories.py

@@ -28 +28 @@
 app_template_dirs = tuple(app_template_dirs)
 
 def get_template_sources(template_name, template_dirs=None):
+    if not os.path.splitext(template_name):
+        # Only append TEMPLATE_FILE_EXTENSION if there isn't an extension already.
+        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)
 
 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

@@ -7 +7 @@
 
 from django.core.template import TemplateDoesNotExist
 from django.conf.settings import INSTALLED_APPS, 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
+        if not os.path.splitext(template_name):
+            # Only append TEMPLATE_FILE_EXTENSION if there isn't an extension already.
+            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))

django/core/template/loaders/filesystem.py

@@ -7 +7 @@
 def get_template_sources(template_name, template_dirs=None):
     if not template_dirs:
         template_dirs = TEMPLATE_DIRS
+    if not os.path.splitext(template_name):
+        # Only append TEMPLATE_FILE_EXTENSION if there isn't an extension already.
+        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)
 
 def load_template_source(template_name, template_dirs=None):
     tried = []