minor updates/corrections

Author: d-w-moore

irods/access.py

@@ -127,7 +127,7 @@ def copy(self, decanonicalize=False, implied_zone=''):
             }.get(self.access_name)
             other.access_name = replacement_string if replacement_string is not None else self.access_name
 
-        # Useful if we wish to force an explicitly specified local zone to null length for equality testing:
+        # Useful if we wish to force an explicitly specified local zone to an implicit zone spec in the copy, for equality testing:
         if '' != implied_zone == other.user_zone:
             other.user_zone = ''
 
@@ -173,9 +173,7 @@ def __lt__(self, other):
         )
 
     def __repr__(self):
-        return f"<ACLOperation: access_name={self.access_name!r} "\
-               f"user_name={self.user_name!r} "\
-               f"user_zone={self.user_zone!r}>"
+        return f"<ACLOperation {self.access_name} {self.user_name} {self.user_zone}>"
 
 class _iRODSAccess_pre_4_3_0(iRODSAccess):
     codes = collections.OrderedDict(

irods/manager/access_manager.py

@@ -38,7 +38,8 @@ def users_by_ids(session, ids=()):
 
 class AccessManager(Manager):
 
-    def _ACL_operation(self, op_input: iRODSAccess):
+    @staticmethod
+    def _to_acl_operation_json(op_input: iRODSAccess):
         return {
             "acl": op_input.access_name,
             "entity_name": op_input.user_name,
@@ -52,7 +53,7 @@ def apply_atomic_acl_operations(self, logical_path : str, *operations, admin=Fal
         request_text = {
             "logical_path": logical_path,
             "admin_mode": admin,
-            "operations": [self._ACL_operation(op) for op in operations]
+            "operations": [self._to_acl_operation_json(op) for op in operations]
         }
 
         with self.sess.pool.get_connection() as conn: