search

plone / plone.app.users

User registration and profile forms for the Plone CMS
https://pypi.python.org/pypi/plone.app.users
14 stars 22 forks source link

Fix for #12297: Plone Administrator unable to edit User Data #1

Closed vmaksymiv closed 12 years ago

vmaksymiv commented 12 years ago

There is bug with a fix at https://dev.plone.org/ticket/12297, the fix is proposed in this pull request. More details in the ticket.

myroslav commented 12 years ago

What is the circumstance for reg_tool.isMemberIdAllowed() to throw Unauthorized exception? Are there any 3rd party solutions that have their isMemberAllowed() doing so?

myroslav commented 12 years ago

We've updated the code to improve user experience in case reg_tool.isMemberIdAllowed() throws Unauthorized just in case there is some 3rd party solutions that have their isMemberAllowed() throwing it?

myroslav commented 12 years ago

@esteele, @mauritsvanrees, @huub_bouma, @khink, @kcleong, @mrsavage - let me know if solution provided in this pull request is acceptable one.

mauritsvanrees commented 12 years ago

Just a quick note that the ticket and this pull request were on my radar already and that I do intend to look at it. I am not going to promise that it will be this weekend though. I did some fixes recently in the collective.emaillogin code for Plone 3 and want to have a look if I need to port any of those fixes to core Plone 4.