-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Handle case where the user is not known #23777
Conversation
By analyzing the blame information on this pull request, we identified @blizzz, @nickvergessen and @LukasReschke to be potential reviewers |
👍 |
I guess there is the same problem in apps/dav/appinfo/application.php at 222 ($migration->syncUser($user->getUID());). It fails with the similar error, only different method. |
No, because callForAllUsers makes sure that user is set. |
nickvergessen, please look at #23759 comments. |
Interesting not until this PR - #23794 🙈 |
👍 code looks good |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
fixes #23759