[Zope-CMF] Re: Old code...

[Yuppie]

Hi!


Florent Guillaume wrote:
> CMFCore/MembershipTool's addMember has since v1.1 this code:
> 
>  else:
>      # The acl_users folder is a LoginManager.  Search for a UserSource
>      # with the needed support.
>      for source in acl_users.UserSourcesGroup.objectValues():
>          if hasattr(source, 'addUser'):
>              source.__of__(self).addUser(id, password, roles, domains)
>      raise "Can't add Member", "No supported UserSources"
> 
> I think I'll remove it, seeing that it cannot ever have worked in the 
> first place...

I don't see it. Could you give me a hint?

Anyway. I don't think there is a need to support the LoginManager 
product. But if you remove this code, why not removing all the 
LoginManager related stuff and deprecating __getPUS?

+1 for removing LoginManager cruft


Cheers,
	Yuppie