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

Spring data Jpa extension : sort with Pageable does not work #5406

Closed
MossabTN opened this issue Nov 12, 2019 · 1 comment · Fixed by #5415
Closed

Spring data Jpa extension : sort with Pageable does not work #5406

MossabTN opened this issue Nov 12, 2019 · 1 comment · Fixed by #5415
Assignees
Labels
area/spring Issues relating to the Spring integration kind/bug Something isn't working
Milestone

Comments

@MossabTN
Copy link

MossabTN commented Nov 12, 2019

When I use the extension of spring data Jpa, the sort with Pageable does not work.

This is my exp:

@GET
    @Produces("application/json")
    public List<Fruit> findAll() {
        return fruitRepository.findAll(PageRequest.of(0,3, Sort.by(Sort.Direction.DESC, "id"))).getContent();
    }

the method sort does not work as if it does not exist

@MossabTN MossabTN added the kind/bug Something isn't working label Nov 12, 2019
@MossabTN MossabTN changed the title Spring data Jpa extension : Sort.by() in PageRequest does not work Spring data Jpa extension : sort with Pageable does not work Nov 12, 2019
@geoand geoand added the area/spring Issues relating to the Spring integration label Nov 12, 2019
@geoand geoand self-assigned this Nov 12, 2019
geoand added a commit to geoand/quarkus that referenced this issue Nov 12, 2019
@geoand
Copy link
Contributor

geoand commented Nov 12, 2019

#5415 should take care of this

FroMage added a commit that referenced this issue Nov 13, 2019
Take Spring Data Pageable's Sort into account
@gsmet gsmet added this to the 1.0.0.Final milestone Nov 14, 2019
gsmet pushed a commit that referenced this issue Nov 14, 2019
ia3andy pushed a commit to dmlloyd/quarkus that referenced this issue Nov 19, 2019
mmusgrov pushed a commit to mmusgrov/quarkus that referenced this issue Dec 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/spring Issues relating to the Spring integration kind/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants