move permissions() directive to grokcore.security where it belongs

CHANGES.txt

@@ -5,8 +5,7 @@ Changes
 1.2 (unreleased)
 ================
 
-- Nothing changed yet.
-
+- Move permissions directive to grokcore.security where it belongs.
 
 1.1 (2011-01-20)
 ================

setup.py

@@ -44,10 +44,11 @@ def read(*rnames):
     zip_safe=False,
     install_requires=[
         'setuptools',
+        'martian',
         'grokcore.component',
         'grokcore.security',
         'zope.securitypolicy',
-        'martian',
+
         'zope.component',
         'zope.interface',
         'zope.publisher',

src/grokcore/permission/__init__.py

@@ -13,4 +13,3 @@
 ##############################################################################
 
 from grokcore.permission.components import Role
-from grokcore.permission.directive import permissions

src/grokcore/permission/directive.py

@@ -20,41 +20,3 @@
 from martian.error import GrokImportError
 from grokcore.security import Permission
 
-
-class permissions(martian.Directive):
-    """The `grokcore.permission.permissions()` directive.
-
-    This directive is used inside of a `grok.Role` subclass to list the
-    permissions which each member of the role should always possess.
-    Note that permissions should be passed as strings, and that several
-    permissions they can simply be supplied as multiple arguments; there
-    is no need to place them inside of a tuple or list::
-
-        class MyRole(grokcore.permission.Role):
-            grokcore.permission.permissions('page.CreatePage', 'page.EditPage')
-            ...
-
-    """
-    scope = martian.CLASS
-    store = martian.ONCE
-    default = []
-
-    def validate(self, *values):
-        for value in values:
-            if martian.util.check_subclass(value, Permission):
-                continue
-            if martian.util.not_unicode_or_ascii(value):
-                raise GrokImportError(
-                    "You can only pass unicode values, ASCII values, or "
-                    "subclasses of grokcore.security.Permission to the '%s'"
-                    " directive."
-                    % self.name)
-
-    def factory(self, *values):
-        permission_ids = []
-        for value in values:
-            if martian.util.check_subclass(value, Permission):
-                permission_ids.append(grokcore.component.name.bind().get(value))
-            else:
-                permission_ids.append(value)
-        return permission_ids

src/grokcore/permission/ftests/permission/roles.py

@@ -64,7 +64,7 @@ class ApprovePermission(grokcore.security.Permission):
 class PaintingOwner(grokcore.permission.Role):
     grok.name('paint.PaintingOwner')
     grok.title('Painting Owner')
-    grokcore.permission.permissions(
+    grokcore.security.permissions(
         'paint.ViewPainting', 'paint.EditPainting', 'paint.ErasePainting')
 
 class CavePainting(grokcore.view.View):

src/grokcore/permission/meta.py

@@ -33,7 +33,7 @@
 from grokcore.security.meta import PermissionGrokker
 from zope.securitypolicy.rolepermission import rolePermissionManager
 from grokcore.permission.components import Role
-from grokcore.permission.directive import permissions
+from grokcore.security.directive import permissions
 from zope.securitypolicy.interfaces import IRole
 
 

src/grokcore/permission/tests/permission/permissions.py

@@ -10,8 +10,8 @@
 
 import grokcore.component as grok
 import grokcore.permission.testing
-from grokcore.permission import permissions, Role
-from grokcore.security import Permission
+from grokcore.permission import Role
+from grokcore.security import Permission, permissions
 import zope.interface
 
 class FirstPermission(Permission):