Skip to content

[Test case DATACMNS-753] parsing of Sort fails to fallback properly #138

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

Closed
wants to merge 1 commit into from

Conversation

Shredder121
Copy link
Contributor

The test cases of DATACMNS-408 included a input of null, but a request with ?sort= means that the parameter is "".
Discussion https://jira.spring.io/browse/DATACMNS-753.

@Shredder121 Shredder121 changed the title [Test case DATACMNS-753] parsing of Sort fails with empty request parameter [Test case DATACMNS-753] parsing of Sort fails to fallback properly Aug 14, 2015
odrotbohm pushed a commit that referenced this pull request Sep 9, 2015
…ing defaults for empty parameter.

Related ticket: DATACMNS-408.
Original pull request: #138.
odrotbohm added a commit that referenced this pull request Sep 9, 2015
We now correctly inspect the sort parameter values and also fall back to the default if a single, empty parameter value is given.

Related ticket: DATACMNS-408.
Related pull request: #138.
odrotbohm pushed a commit that referenced this pull request Sep 9, 2015
…ing defaults for empty parameter.

Related ticket: DATACMNS-408.
Original pull request: #138.
odrotbohm added a commit that referenced this pull request Sep 9, 2015
We now correctly inspect the sort parameter values and also fall back to the default if a single, empty parameter value is given.

Related ticket: DATACMNS-408.
Related pull request: #138.
odrotbohm pushed a commit that referenced this pull request Sep 9, 2015
…ing defaults for empty parameter.

Related ticket: DATACMNS-408.
Original pull request: #138.
odrotbohm added a commit that referenced this pull request Sep 9, 2015
We now correctly inspect the sort parameter values and also fall back to the default if a single, empty parameter value is given.

Related ticket: DATACMNS-408.
Related pull request: #138.
@odrotbohm
Copy link
Member

That's merged and a fix applied, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants