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

Fix issues where some user settings cannot be loaded when the user id… #1142

Merged
merged 1 commit into from
Aug 29, 2016

Conversation

LukasReschke
Copy link
Member

… differs in case sensitivity - fixes #25684 (#25686)

From owncloud/core#25686

@LukasReschke LukasReschke added 3. to review Waiting for reviews downstream labels Aug 29, 2016
@LukasReschke LukasReschke added this to the Nextcloud 11.0 milestone Aug 29, 2016
@mention-bot
Copy link

@LukasReschke, thanks for your PR! By analyzing the annotation information on this pull request, we identified @icewind1991, @DeepDiver1975 and @blizzz to be potential reviewers

@icewind1991
Copy link
Member

👍

@nickvergessen
Copy link
Member

This will be a performance slow down for operations that iterate over a list of users.
But never the less, necessary. 👍

@nickvergessen nickvergessen added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Aug 29, 2016
@rullzer rullzer merged commit 7c7f862 into master Aug 29, 2016
@rullzer rullzer deleted the fix-case-sensitive-settings branch August 29, 2016 17:56
@MorrisJobke
Copy link
Member

@nickvergessen should we also backport this to stable10?

@nickvergessen
Copy link
Member

See #1156

@blizzz
Copy link
Member

blizzz commented May 22, 2017

And this introduced #3365 🎆 🌊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants