[Checkins] SVN: Products.CMFCore/trunk/Products/CMFCore/CMFCatalogAware.py - fixed last checkin: don't set 'Owner' to None

Yvo Schubbe y.2007- at wcm-solutions.de
Wed Dec 26 09:47:42 EST 2007


Log message for revision 82451:
  - fixed last checkin: don't set 'Owner' to None

Changed:
  U   Products.CMFCore/trunk/Products/CMFCore/CMFCatalogAware.py

-=-
Modified: Products.CMFCore/trunk/Products/CMFCore/CMFCatalogAware.py
===================================================================
--- Products.CMFCore/trunk/Products/CMFCore/CMFCatalogAware.py	2007-12-26 14:30:45 UTC (rev 82450)
+++ Products.CMFCore/trunk/Products/CMFCore/CMFCatalogAware.py	2007-12-26 14:47:42 UTC (rev 82451)
@@ -249,10 +249,14 @@
     # Make sure owner local role is set after pasting
     # The standard Zope mechanisms take care of executable ownership
     current_user = _getAuthenticatedUser(ob)
-    if current_user is not None:
+    if current_user is None:
+        return
+
+    current_user_id = current_user.getId()
+    if current_user_id is not None:
         local_role_holders = [ x[0] for x in ob.get_local_roles() ]
         ob.manage_delLocalRoles(local_role_holders)
-        ob.manage_setLocalRoles(current_user.getId(), ['Owner'])
+        ob.manage_setLocalRoles(current_user_id, ['Owner'])
 
 def dispatchToOpaqueItems(ob, event):
     """Dispatch an event to opaque sub-items of a given object.



More information about the Checkins mailing list