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.

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

FetchableFluentQueryByPredicate.sortBy and FetachableFluentQueryByExample.sortBy use wrong object reference #2438

Closed
jonyschak opened this issue Feb 13, 2022 · 0 comments
Assignees
Labels
type: bug A general bug

Comments

@jonyschak
Copy link
Contributor

The sortBy method in FetchableFluentQueryByPredicate and FetachableFluentQueryByExample calls sort.and(sort) rather than this.sort.and(sort).

This causes the sort to be duplicated and any previous sort to be lost.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Feb 13, 2022
jonyschak added a commit to jonyschak/spring-data-jpa that referenced this issue Feb 13, 2022
jonyschak added a commit to jonyschak/spring-data-jpa that referenced this issue Feb 13, 2022
@mp911de mp911de linked a pull request Feb 14, 2022 that will close this issue
4 tasks
@mp911de mp911de added type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Feb 14, 2022
schauder added a commit that referenced this issue Feb 15, 2022
See #2438
Original pull request #2439
schauder pushed a commit that referenced this issue Feb 15, 2022
…tBy and FetchableFluentQueryByPredicate.sortBy

Closes #2438
Original pull request #2439
schauder added a commit that referenced this issue Feb 15, 2022
See #2438
Original pull request #2439
schauder pushed a commit that referenced this issue Feb 15, 2022
…tBy and FetchableFluentQueryByPredicate.sortBy

Closes #2438
Original pull request #2439
schauder added a commit that referenced this issue Feb 15, 2022
See #2438
Original pull request #2439
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants