Collector #2213: Can't edit "old" ZopePageTemplate instances.

o Also, made docstrings in testcase methods into comments, per coding standard.

doc/CHANGES.txt

@@ -8,8 +8,10 @@ Zope Changes
 
     Bugs fixed
 
-      - Collector #2208: rewriting/setting the 'charset' part of the content-type
-        HTTP header will be done only for 'text/*'
+      - Collector #2213: Can't edit "old" ZopePageTemplate instances.
+
+      - Collector #2208: rewriting/setting the 'charset' part of the
+        content-type HTTP header will be done only for 'text/*'
 
       - Collector #2209: ZTUtils module could not be used inside ZPT
 
@@ -43,7 +45,8 @@ Zope Changes
       - Collector #2152: Fixed MailHost documentation; simple_send does not
         process or validate its arguments in any way.
 
-      - Collector #2175: ZTUtils.make_hidden_input did not escape double-quotes.
+      - Collector #2175: ZTUtils.make_hidden_input did not escape
+        double-quotes.
 
       - Collector #2176: Fixed bad logging call.
 

lib/python/Products/PageTemplates/ZopePageTemplate.py

@@ -79,6 +79,7 @@ class ZopePageTemplate(Script, PageTemplate, Historical, Cacheable,
 
     func_defaults = None
     func_code = FuncCode((), 0)
+    strict = False
 
     _default_bindings = {'name_subpath': 'traverse_subpath'}
     _default_content_fn = os.path.join(package_home(globals()),

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

@@ -40,7 +40,7 @@ class ZPTRegressions(unittest.TestCase):
         self.assertEqual(pt.document_src().strip(), default_text.strip())
 
     def testAddWithRequest(self):
-        """Test manage_add with file"""
+        # Test manage_add with file
         request = self.app.REQUEST
         request.form['file'] = DummyFileUpload(filename='some file',
                                                data=self.text,
@@ -51,13 +51,21 @@ class ZPTRegressions(unittest.TestCase):
         self.assertEqual(pt.document_src(), self.text)
 
     def testAddWithRequestButNoFile(self):
-        """Collector #596: manage_add with text but no file"""
+        # Collector #596: manage_add with text but no file
         request = self.app.REQUEST
         self._addPT('pt1', text=self.text, REQUEST=request)
         # no object is returned when REQUEST is passed.
         pt = self.app.pt1
         self.assertEqual(pt.document_src(), self.text)
 
+    def test_BBB_for_strict_attribute(self):
+        # Collector 2213:  old templates don't have 'strict' attribute.
+        from Products.PageTemplates.ZopePageTemplate import ZopePageTemplate
+        zpt = ZopePageTemplate('issue_2213')
+        del zpt.strict  # simulate old templates
+        self.assertEqual(zpt.strict, False)
+
+
 class ZPTMacros(zope.component.testing.PlacelessSetup, unittest.TestCase):
 
     def setUp(self):