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
WW-4741: Do not create session #207
Conversation
test no session but with locale parameter
} | ||
|
||
return locale; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When shouldStore
field is not false (i.e. we should store), shouldn't we forcefully create a session and store the locale? if we should not, shouldn't we return null
here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we should forcefully create the session but I would leave that to a user, if there was no session just ignore (a user fault). And returning null
can break action processing as then the null will be set in ActionContext
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
test not create when no session
LGTM 👍 Nice catch! Thanks! |
Resolves WW-4741