[Zope-CMF] getPortalRoles: should 'Owner' role be included?

[Tres Seaver]

Joseph Wayne Norton wrote:

> I would like to know if there is any specific reason why 'Owner' is
> not included (like 'Manager') in the list of roles returned by
> getPortalRoles.
> 
> thanks,
> 
> - joe
> 
> 
>     security.declareProtected(ManagePortal, 'getPortalRoles')
>     def getPortalRoles(self):
>         """
>         Return all local roles defined by the portal itself,
>         which means roles that are useful and understood 
>         by the portal object
>         """
>         parent = self.aq_inner.aq_parent
>         roles = list(parent.__ac_roles__)
> 
>         # This is *not* a local role in the portal but used by it
>         roles.append('Manager')
> 
>         return roles

I just checked in a fix to append 'Owner' to the list as well on

Friday.  Thanks for pointing this out!

 <http://cvs.zope.org/cgi-bin/viewcvs.cgi/CMF/CMFCore/MembershipTool.py.diff?r1=1.10&r2=1.11>


Tres.
-- 
===============================================================
Tres Seaver                                tseaver@zope.com
Zope Corporation      "Zope Dealers"       http://www.zope.com