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

Unable to save @@personal-preferences when using email as login #56

Closed
caryan opened this issue Apr 29, 2016 · 2 comments
Closed

Unable to save @@personal-preferences when using email as login #56

caryan opened this issue Apr 29, 2016 · 2 comments

Comments

@caryan
Copy link

caryan commented Apr 29, 2016

I've got a an issue where if the site uses "email address as login name" users cannot save their personal preferences. This could be related to this closed but unsolved issue.

Steps to reproduce:

  1. Fire up new Plone container: docker run --rm -p 8080:8080 plone/plone. This is 5.0.4 right now.
  2. Goto localhost:8080 and Create a new site
  3. Login as admin:admin and under "Site Setup -> Security: or @@security-controlpanel enable "Use email address as login name"
  4. Create a new user.
  5. Log out from admin
  6. Login as new user
  7. Go to "Preferences" or @@personal-preferences and change "Wysiwyg editor" to "TinyMCE.
  8. Try to "Save"

Throws an error with the traceback in the console:

2016-04-29T01:51:55 ERROR Zope.SiteErrorLog 1461894715.450.410967712394 http://localhost:8080/Plone/@@personal-preferences
Traceback (innermost last):
  Module ZPublisher.Publish, line 138, in publish
  Module ZPublisher.mapply, line 77, in mapply
  Module ZPublisher.Publish, line 48, in call_object
  Module plone.app.users.browser.personalpreferences, line 104, in __call__
  Module z3c.form.form, line 233, in __call__
  Module plone.z3cform.fieldsets.extensible, line 59, in update
  Module plone.z3cform.patch, line 30, in GroupForm_update
  Module z3c.form.group, line 145, in update
  Module plone.app.z3cform.csrf, line 21, in execute
  Module z3c.form.action, line 98, in execute
  Module z3c.form.button, line 315, in __call__
  Module z3c.form.button, line 170, in __call__
  Module plone.app.users.browser.account, line 210, in handleSave
  Module plone.app.users.browser.account, line 193, in validate_email
KeyError: 'email'
mauritsvanrees added a commit that referenced this issue Apr 29, 2016
This could hebben when email is used as login name.

Fixes #56
mauritsvanrees added a commit that referenced this issue Apr 29, 2016
This could happen when email is used as login name.

Fixes #56
and plone/Products.CMFPlone#1146
@mauritsvanrees
Copy link
Member

I hope to have fixed this with the pull request linked above. Waiting for what the test server says.

jensens pushed a commit that referenced this issue Apr 30, 2016
This could happen when email is used as login name.

Fixes #56
and plone/Products.CMFPlone#1146
mister-roboto pushed a commit to plone/buildout.coredev that referenced this issue Apr 30, 2016
Branch: refs/heads/master
Date: 2016-04-30T23:01:26+02:00
Author: Maurits van Rees (mauritsvanrees) <maurits@vanrees.org>
Commit: plone/plone.app.users@caa6a2f

Fixed KeyError email on personal preferences form. (#58)

This could happen when email is used as login name.

Fixes plone/plone.app.users#56
and plone/Products.CMFPlone#1146

Files changed:
M CHANGES.rst
M plone/app/users/browser/account.py
mister-roboto pushed a commit to plone/buildout.coredev that referenced this issue Apr 30, 2016
Branch: refs/heads/master
Date: 2016-04-30T23:01:26+02:00
Author: Maurits van Rees (mauritsvanrees) <maurits@vanrees.org>
Commit: plone/plone.app.users@caa6a2f

Fixed KeyError email on personal preferences form. (#58)

This could happen when email is used as login name.

Fixes plone/plone.app.users#56
and plone/Products.CMFPlone#1146

Files changed:
M CHANGES.rst
M plone/app/users/browser/account.py
@caryan
Copy link
Author

caryan commented May 2, 2016

Looks good. Thanks for the quick patch.

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