- Collector #2232: Can't call DTML templates from Page Templates

doc/CHANGES.txt

@@ -36,6 +36,8 @@ Zope Changes
 
     Bugs Fixed
 
+      - Collector #2232: Can't call DTML templates from Page Templates
+
       - Collector #2198: Zope 3.3 fix breaks Five 1.5 test_getNextUtility
 
       - Collector #2206: Set PYTHONPATH to include existing PYTHONPATH

lib/python/Products/PageTemplates/Expressions.py

@@ -89,7 +89,7 @@ def render(ob, ns):
         if callable(base):
             try:
                 if getattr(base, 'isDocTemp', 0):
-                    ob = call_with_ns(ob, ns, 2)
+                    ob = ZRPythonExpr.call_with_ns(ob, ns, 2)
                 else:
                     ob = ob()
             except AttributeError, n:

lib/python/Products/PageTemplates/tests/testExpressions.py

@@ -12,6 +12,12 @@ class Dummy:
     def __call__(self):
         return 'dummy'
 
+class DummyDocumentTemplate:
+    __allow_access_to_unprotected_subobjects__ = 1
+    isDocTemp = True
+    def __call__(self, client=None, REQUEST={}, RESPONSE=None, **kw):
+        return 'dummy'
+
 class ExpressionTests(zope.component.testing.PlacelessSetup, unittest.TestCase):
 
     def setUp(self):
@@ -23,7 +29,8 @@ class ExpressionTests(zope.component.testing.PlacelessSetup, unittest.TestCase):
             one = 1,
             d = {'one': 1, 'b': 'b', '': 'blank', '_': 'under'},
             blank = '',
-            dummy = Dummy()
+            dummy = Dummy(),
+            dummy2 = DummyDocumentTemplate()
             )
 
     def testCompile(self):
@@ -47,7 +54,12 @@ class ExpressionTests(zope.component.testing.PlacelessSetup, unittest.TestCase):
 
     def testRenderedEval(self):
         ec = self.ec
-        assert ec.evaluate('dummy') == 'dummy'
+        self.assertEquals(ec.evaluate('dummy'), 'dummy')
+
+        # http://www.zope.org/Collectors/Zope/2232
+        # DTML templates could not be called from a Page Template
+        # due to an ImportError
+        self.assertEquals(ec.evaluate('dummy2'), 'dummy')
 
     def testEval1(self):
         '''Test advanced expression evaluation 1'''