Fix default values for Permission's title and description in Py2issue48

And test this. This applies to both the class and the ZCML directive.

Also adjust IPermission to use NativeStringLine for ``id``, since
that's what ZCML has always been creating (it uses an Id, which
subclasses NativeStringLine), and what's typically written in source
code. Test that the permissions created from ZCML are valid with this
change.

1 files changed, 8 insertions, 3 deletions

diff --git a/src/zope/security/interfaces.py b/src/zope/security/interfaces.py
index 4c62bdc..1d646ee 100644
--- a/src/zope/security/interfaces.py
+++ b/src/zope/security/interfaces.py
@@ -52,7 +52,7 @@ means the name of the permission, or the special object
 
 from zope.interface import Interface, Attribute, implementer
 from zope.interface.common.interfaces import IException, IAttributeError
-from zope.schema import Text, TextLine
+from zope.schema import Text, TextLine, NativeStringLine
 from zope.security.i18n import ZopeMessageFactory as _
 
 #: The name (id) of the registered :class:`IPermission` utility that signifies
@@ -438,9 +438,14 @@ class IMemberAwareGroup(IMemberGetterGroup):
         """
 
 class IPermission(Interface):
-    """A permission object."""
+    """A permission object.
 
-    id = TextLine(
+    Note that the ZCML ``<permission>`` directive restricts the ``id`` to
+    be an identifier (a dotted name or a URI), but this interface allows
+    any native string.
+    """
+
+    id = NativeStringLine(
         title=_("Id"),
         description=_("Id as which this permission will be known and used."),
         readonly=True,