templating: Move template selection logic from the template loaders

            to the client.

Author: techtonik

roundup/cgi/client.py

@@ -1063,11 +1063,23 @@ def renderFrontPage(self, message):
         self.error_message.append(message)
         self.write_html(self.renderContext())
 
+    def selectTemplate(self):
+        """ Template selection logic """
+        loader = self.instance.templates
+
+        name = self.classname
+        view = self.template
+        
+        # if classname is not set, use "home" template
+        if name is None:
+            name = 'home'
+
+        return name, view
+
     def renderContext(self):
         """ Return a PageTemplate for the named page
         """
-        name = self.classname
-        view = self.template
+        name, view = self.selectTemplate()
 
         # catch errors so we can handle PT rendering errors more nicely
         args = {

roundup/cgi/engine_chameleon.py

@@ -13,10 +13,6 @@ def __init__(self, dir):
         self.loader = chameleon.PageTemplateLoader(dir)
 
     def load(self, name, view=None):
-        # default the name to "home"
-        if name is None:
-            name = 'home'
-
         src, filename = find_template(self.dir, name, view)
         return RoundupPageTemplate(self.loader.load(src))
 

roundup/cgi/engine_zopetal.py

@@ -22,10 +22,6 @@ def __init__(self, dir):
         self.dir = dir
 
     def load(self, name, view=None):
-        # default the name to "home"
-        if name is None:
-            name = 'home'
-
         # find the source
         src, filename = find_template(self.dir, name, view)