[ZCM] [ZC] 2074/ 2 Comment "misbehaviour of OFS.Owned.changeOwnership"

[Collector: Zope Bugs, Features, and Patches ...]

Issue #2074 Update (Comment) "misbehaviour of OFS.Owned.changeOwnership"
 Status Pending, Zope/bug+solution medium
To followup, visit:
  http://www.zope.org/Collectors/Zope/2074

==============================================================
= Comment - Entry #2 by sbunse on Apr 21, 2006 11:43 am

please close, this is a double post (hit cancel then reload in browser)
________________________________________
= Request - Entry #1 by sbunse on Apr 21, 2006 11:02 am

when i request a recursive changeOwnership on a folder i expext itself and all children to be owned by the given user. but if the folder is already owned by the target user the method returns before processing it's children.

proposed solution: wrap lines 150 & 151 into if clause like this:
>        if not recursive:
>            if old==new: return
>            if old is UnownableOwner: return

sry it's not a patch but i think it would be overkill for this simple change ;)
==============================================================