[Zope-PAS] proposal: IUserAdder plugins

2007-05-04 Thread Miles
I seem to be generating a sizable proportion of the traffic on this 
list!  Apologies in advance for another email...


My use case is as follows:

I have a zope website, a php forum and blog that make up my site, each 
with their own user databases.  Whenever users register through the zope 
site (I have removed links so they cannot register elsewhere), I want to 
create corresponding user accounts for them in the database for the 
forum and the blog.


However, when a user is created, PAS runs through all the plugins 
implementing the IUserAdder interface, but once a plugin indicates that 
it created a user, it proceeds no further.


I was expecting it to pass the details onto all the plugins, to give 
each a chance to create any additional information the plugin needs, and 
to avoid having to produce a succession of plugins.  In the same way 
that the properties plugins each get to provide a separate property 
sheet, rather than just returning the first one.


I would like to change this so all IUserAdder plugins get a chance to 
fire - does anyone have an opinion on it?


Thanks,

Miles



___
Zope-PAS mailing list
Zope-PAS@zope.org
http://mail.zope.org/mailman/listinfo/zope-pas


[Zope-PAS] Re: missing method in interfaces

2007-05-04 Thread Miles

Jens Vagelpohl wrote:

On 4 May 2007, at 05:17, Miles wrote:




Can I provide a patch to add it into the definition, and to 
zodbrolemanager?


You're very welcome to provide any bug report or patch through the PAS 
collector:


http://www.zope.org/Collectors/PAS/

jens



Done

http://www.zope.org/Collectors/PAS/57

Thanks

___
Zope-PAS mailing list
Zope-PAS@zope.org
http://mail.zope.org/mailman/listinfo/zope-pas


Re: [Zope-PAS] missing method in interfaces

2007-05-04 Thread Jens Vagelpohl

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1


On 4 May 2007, at 05:17, Miles wrote:


Hi,

The interfaces definition from the plugins directory defines:

IRoleAssignerPlugin
def doAssignRoleToPrincipal( principal_id, role ):

However, it doesn't document any way to remove a role from a  
principal.  Should there be a doRemoveRoleFromPrincipal 
( principal_id ) method for this?


zodbrolemanager provides a removeRoleFromPrincipal method which  
does this, and could be wired up in the same way as  
doAssignRoleToPrincipal is currently.


Can I provide a patch to add it into the definition, and to  
zodbrolemanager?


You're very welcome to provide any bug report or patch through the  
PAS collector:


http://www.zope.org/Collectors/PAS/

jens


-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.7 (Darwin)

iD8DBQFGOyxJRAx5nvEhZLIRAtcLAJ9Lv/7nO2SpJpsurzS89OyUDYVAYwCdFlqO
OhG+IcKdv06ksix2AujjLac=
=qlO+
-END PGP SIGNATURE-
___
Zope-PAS mailing list
Zope-PAS@zope.org
http://mail.zope.org/mailman/listinfo/zope-pas


[Zope-PAS] missing method in interfaces

2007-05-04 Thread Miles

Hi,

The interfaces definition from the plugins directory defines:

IRoleAssignerPlugin
def doAssignRoleToPrincipal( principal_id, role ):

However, it doesn't document any way to remove a role from a principal. 
 Should there be a doRemoveRoleFromPrincipal( principal_id ) method for 
this?


zodbrolemanager provides a removeRoleFromPrincipal method which does 
this, and could be wired up in the same way as doAssignRoleToPrincipal 
is currently.


Can I provide a patch to add it into the definition, and to zodbrolemanager?

Thanks,

Miles

___
Zope-PAS mailing list
Zope-PAS@zope.org
http://mail.zope.org/mailman/listinfo/zope-pas