Use str __name__ for generated simple view classes.

This fixes LP #1129030 M
zopefoundation · Feb 18, 2013 · 5d3a189 · 5d3a189
1 parent a955562
commit 5d3a189
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/doc/CHANGES.rst b/doc/CHANGES.rst
@@ -11,6 +11,9 @@ http://docs.zope.org/zope2/releases/.
 Bugs Fixed
 ++++++++++
 
+- Make sure the generated classes for simple browser pages (SimpleViewClasses)
+  have a str __name__. See LP #1129030.
+
 - In ``PageTemplate.pt_errors`` accept the ``check_macro_expansion`` argument.
   This is added for compatibility with ``zope.pagetemplate`` 4.0.0.
   The argument is ignored (LP #732972).

diff --git a/src/Products/Five/browser/metaconfigure.py b/src/Products/Five/browser/metaconfigure.py
@@ -92,6 +92,7 @@ def page(_context, name, permission, for_=Interface,
          allowed_interface=None, allowed_attributes=None,
          attribute='__call__', menu=None, title=None, 
          ):
+    name = str(name)  # De-unicode
     _handle_menu(_context, menu, title, [for_], name, permission, layer)
     required = {}
 
@@ -452,7 +453,7 @@ def __getitem__(self, name):
 
 
 # Original version: zope.browserpage.simpleviewclass.SimpleViewClass
-def SimpleViewClass(src, offering=None, used_for=None, bases=(), name=u''):
+def SimpleViewClass(src, offering=None, used_for=None, bases=(), name=''):
     if offering is None:
         offering = sys._getframe(1).f_globals