Launchpad #198274: "empty" ZopePageTemplates could not be unpickled.

doc/CHANGES.txt

@@ -8,6 +8,9 @@ Zope Changes
 
     Bugs fixed
 
+      - Launchpad #198274:  "empty" ZopePageTemplates could not be
+        unpickled.
+
       - zope.security: switched to use standalone 3.3.3 version, which
         contains a backport of a huge performance bugfix from the 3.4 branch.
 

lib/python/Products/PageTemplates/ZopePageTemplate.py

@@ -413,7 +413,8 @@ class ZopePageTemplate(Script, PageTemplate, Historical, Cacheable,
         # Perform on-the-fly migration to unicode.
         # Perhaps it might be better to work with the 'generation' module 
         # here?
-        if not isinstance(state['_text'], unicode):
+        _text = state.get('_text')
+        if _text is not None and not isinstance(state['_text'], unicode):
             text, encoding = convertToUnicode(state['_text'], 
                                     state.get('content_type', 'text/html'), 
                                     preferred_encodings)

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

@@ -141,6 +141,17 @@ class ZPTUnicodeEncodingConflictResolution(ZopeTestCase):
         result = zpt.pt_render()
         self.assertEqual(result.startswith(unicode('<div></div>', 'iso-8859-15')), False)
 
+    def test_bug_198274(self):
+        # See https://bugs.launchpad.net/bugs/198274
+        # ZPT w/ '_text' not assigned can't be unpickled.
+        import cPickle
+        empty = ZopePageTemplate(id='empty', text=' ',
+                                 content_type='text/html',
+                                 output_encoding='ascii',
+                                )
+        state = cPickle.dumps(empty, protocol=1)
+        clone = cPickle.loads(state)
+
 class ZopePageTemplateFileTests(ZopeTestCase):
 
     def testPT_RenderWithAscii(self):