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

LoggedInUserAccountArgumentResolver needs to be adapted to work with Optional<UserAccount> only #37

Closed
odrotbohm opened this issue Oct 27, 2014 · 0 comments
Closed

LoggedInUserAccountArgumentResolver needs to be adapted to work with Optional<UserAccount> only #37

odrotbohm opened this issue Oct 27, 2014 · 0 comments
Assignees
@odrotbohm
Labels
type: bug Bugs
Milestone
6.0 RC1

Comments

@odrotbohm
Copy link
Member

No description provided.

@odrotbohm odrotbohm self-assigned this Oct 27, 2014
@odrotbohm odrotbohm added this to the 6.0 RC1 milestone Oct 27, 2014
odrotbohm added a commit that referenced this issue Oct 27, 2014
@odrotbohm
#37 - LoggedInUserAccountArgumentResolver now checks for Optional<Use…
eb22ee9 
…rAccount>.

LoggedInUserAccountArgumentResolver now explicitly checks the parameter type to be an Optional<UserAccount>. Added unit tests for argument resolution.

Polished Javadoc on @loggedin.
@odrotbohm odrotbohm added the type: bug Bugs label Oct 27, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@odrotbohm odrotbohm

Labels
type: bug Bugs
Projects
None yet
Milestone
6.0 RC1
Development

No branches or pull requests
1 participant
@odrotbohm