Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Too many groups break "Add new user" form #33

Closed
alecghica opened this issue Mar 25, 2015 · 1 comment
Closed

Too many groups break "Add new user" form #33

alecghica opened this issue Mar 25, 2015 · 1 comment

Comments

@alecghica
Copy link
Member

How to reproduce

  1. login as a Manager
  2. create many user groups
  3. check "Many groups?" under @@usergroup-controlpanel
  4. click "Add new user" under @@usergroup-userprefs

The issue

The above steps will make the "Add new user" form to take a huge amount of time to load. This become worst if you use/have many LDAP groups.

Expected behavior

When "Many groups?" is checked under @@usergroup-controlpanel it should not list anymore the groups to the "Add new user" form.

Suggested solution

Under plone.app.user.browser.register.py module hide the groups listing under from the "Add new user" form based on portal_properties.site_properties.many_groups value.

avoinea added a commit to eea/plone.app.users that referenced this issue May 18, 2015
avoinea added a commit to eea/plone.app.users that referenced this issue May 18, 2015
Conflicts:
	plone/app/users/browser/register.py
avoinea added a commit that referenced this issue May 18, 2015
Refs #33 - Fix "Add new user" form when there are too many groups.
avoinea added a commit that referenced this issue May 18, 2015
Refs #33 - Fix "Add new user" form when there are too many groups.
@avoinea
Copy link
Sponsor Member

avoinea commented May 18, 2015

Fixed.

@avoinea avoinea closed this as completed May 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants