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.

Sign up for GitHub

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

Jump to bottom

IUser::getEMailAddress() returns null #2235

Closed
georgehrke opened this issue Nov 22, 2016 · 2 comments · Fixed by #2276
Closed

IUser::getEMailAddress() returns null #2235

georgehrke opened this issue Nov 22, 2016 · 2 comments · Fixed by #2276
Assignees
@schiessle
Labels
bug high regression
Milestone
Nextcloud 11.0

Comments

@georgehrke
Copy link
Member

Steps to reproduce

  1. use IUser in an app
  2. call getEMailAddress() to get user's email address

Expected behaviour

getEMailAddress() returns e-mail entered in personal settings

Actual behaviour

getEMailAddress() returns null

Reproducible on master
Regression
Most certainly introduced with AccountsManager

@schiessle
@georgehrke georgehrke added this to the Nextcloud 11.0 milestone Nov 22, 2016
@schiessle
Copy link
Member

I will have a look at it

@schiessle schiessle self-assigned this Nov 23, 2016
@schiessle schiessle mentioned this issue Nov 23, 2016
Merged
@schiessle
Copy link
Member

fix is here: #2276

Problem was not the getEMailAddress(), but that we didn't updated the users setting correctly after the user changed his email address... Please review.

@nextcloud-bot nextcloud-bot mentioned this issue Sep 25, 2018
Closed
@nextcloud-bot nextcloud-bot mentioned this issue Oct 23, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@schiessle schiessle

Labels
bug high regression
Projects
None yet
Milestone
Nextcloud 11.0
Development

Successfully merging a pull request may close this issue.

Update email address nextcloud/server
3 participants
@nickvergessen @georgehrke @schiessle