-
Notifications
You must be signed in to change notification settings - Fork 28
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
LPS-47130 Create an inner class even if I only found one cause currently #18999
Conversation
…orkflow status is encountered
…. Use PortalException and provide a more explanatory message instead. Another option would be to create an specific exception for this case.
…or duplicate field values
… refers to the existing user
…t be met (and it hasn't)
I've left this as a separate commit because I'm unsure about whether this is the best solution. Another alternative could be to rename this exceptions to make them more specific and pass the more specific parameters to their constructors.
Hey guys, I'm not sure what the status of this LPS is but as I was cherry picking to build a support driven hotfix for it, I noticed on line 5156 of UserLocalServiceImpl that it should probably be 'existingUser' instead of 'user', so just wanted to let you know. |
Thanks Bryan, I'll keep that into account. |
Brian, since this pull has been stuck for several days, I'll close it and will send it in smaller pieces. |
No description provided.