[Checkins] SVN: z3c.securitytool/trunk/src/z3c/securitytool/securitytool.py Misc comment changes

Daniel Blackburn blackburnd at gmail.com
Sat May 24 10:12:25 EDT 2008


Log message for revision 86927:
  Misc comment changes

Changed:
  U   z3c.securitytool/trunk/src/z3c/securitytool/securitytool.py

-=-
Modified: z3c.securitytool/trunk/src/z3c/securitytool/securitytool.py
===================================================================
--- z3c.securitytool/trunk/src/z3c/securitytool/securitytool.py	2008-05-24 11:39:08 UTC (rev 86926)
+++ z3c.securitytool/trunk/src/z3c/securitytool/securitytool.py	2008-05-24 14:12:22 UTC (rev 86927)
@@ -109,6 +109,7 @@
                     permSetting = (role,'Allow')
 
                 elif role['setting'] == Deny:
+                    #If the role has a setting and it is Deny.
                     try:
                         # Here we see if we have added a security setting with
                         # this role before, if it is now denied we remove it.
@@ -118,6 +119,8 @@
                         pass
 
                 else:
+                    # The role has a setting and is Allow so we add it to the
+                    # matrix.
                     permSetting =  principalRoleProvidesPermission(
                                    principalRoles, rolePermMap,
                                    principal, read_perm,
@@ -162,7 +165,7 @@
 
             if matrix.setdefault(principal,{self.name:setting}) == \
                                                  {self.name:setting}:
-                #If the prin is not in the matrix add it
+                #If the principal is not in the matrix add it
                 continue
 
             elif  matrix[principal].setdefault(



More information about the Checkins mailing list