Spring/spring-data-jpa
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

DATAJPA-94 - Repository queries consider Sort instance in a Pageable again.

Browse Source
This commit is contained in:
Phillip Webb
2011-08-27 20:31:06 +01:00
committed by Oliver Gierke
parent 5b9e959b3e
commit aa6a7c1983
3 changed files with 17 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/main/java/org/springframework/data/jpa/repository/query/PartTreeJpaQuery.java
View File

@@ -167,7 +167,7 @@ public class PartTreeJpaQuery extends AbstractJpaQuery {
private Sort getDynamicSort(Object[] values) {
return parameters.hasSortParameter() ? new ParametersParameterAccessor(
return parameters.potentiallySortsDynamically() ? new ParametersParameterAccessor(
parameters, values).getSort() : null;
}
}

13
src/test/java/org/springframework/data/jpa/repository/UserRepositoryFinderTests.java
View File

@@ -27,6 +27,8 @@ import org.junit.runner.RunWith;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.data.domain.Page;
import org.springframework.data.domain.PageRequest;
import org.springframework.data.domain.Sort;
import org.springframework.data.domain.Sort.Direction;
import org.springframework.data.jpa.domain.sample.User;
import org.springframework.data.jpa.repository.sample.UserRepository;
import org.springframework.data.repository.query.QueryLookupStrategy;
@@ -177,4 +179,15 @@ public class UserRepositoryFinderTests {
assertThat(result.get(0), is(dave));
}
@Test
public void respectsPageableOrderOnQueryGenerateFromMethodName() throws Exception {
Page<User> ascending = userRepository.findByLastnameIgnoringCase(new PageRequest(0, 10, new Sort(Direction.ASC, "firstname")),"Matthews");
Page<User> descending = userRepository.findByLastnameIgnoringCase(new PageRequest(0, 10, new Sort(Direction.DESC, "firstname")),"Matthews");
assertThat(ascending.getTotalElements(), is(2L));
assertThat(descending.getTotalElements(), is(2L));
assertThat(ascending.getContent().get(0).getFirstname(), is(not(equalTo(descending.getContent().get(0).getFirstname()))));
assertThat(ascending.getContent().get(0).getFirstname(), is(equalTo(descending.getContent().get(1).getFirstname())));
assertThat(ascending.getContent().get(1).getFirstname(), is(equalTo(descending.getContent().get(0).getFirstname())));
}
}

3
src/test/java/org/springframework/data/jpa/repository/sample/UserRepository.java
View File

@@ -220,6 +220,9 @@ public interface UserRepository extends JpaRepository<User, Integer>,
List<User> findByLastnameIgnoringCase(String lastname);
Page<User> findByLastnameIgnoringCase(Pageable pageable, String lastname);
List<User> findByLastnameIgnoringCaseLike(String lastname);