Consider sort order for unpaged Pageable in Querydsl.

We now apply sorting using Querydsl for Pageable that is sorted but not paged.

Closes #3761

Author: mp911de

spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/support/Querydsl.java

@@ -124,12 +124,11 @@ public <T> JPQLQuery<T> applyPagination(Pageable pageable, JPQLQuery<T> query) {
 		Assert.notNull(pageable, "Pageable must not be null");
 		Assert.notNull(query, "JPQLQuery must not be null");
 
-		if (pageable.isUnpaged()) {
-			return query;
-		}
+		if (pageable.isPaged()) {
 
-		query.offset(pageable.getOffset());
-		query.limit(pageable.getPageSize());
+			query.offset(pageable.getOffset());
+			query.limit(pageable.getPageSize());
+		}
 
 		return applySorting(pageable.getSort(), query);
 	}

spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/support/SimpleJpaRepository.java

@@ -738,7 +738,6 @@ protected <S extends T> Page<S> readPage(TypedQuery<S> query, Class<S> domainCla
 	 * @param pageable must not be {@literal null}.
 	 */
 	protected TypedQuery<T> getQuery(Specification<T> spec, Pageable pageable) {
-
 		return getQuery(spec, getDomainClass(), pageable.getSort());
 	}
 
@@ -750,7 +749,6 @@ protected TypedQuery<T> getQuery(Specification<T> spec, Pageable pageable) {
 	 * @param pageable must not be {@literal null}.
 	 */
 	protected <S extends T> TypedQuery<S> getQuery(Specification<S> spec, Class<S> domainClass, Pageable pageable) {
-
 		return getQuery(spec, domainClass, pageable.getSort());
 	}
 

spring-data-jpa/src/test/java/org/springframework/data/jpa/repository/support/QuerydslJpaPredicateExecutorUnitTests.java

@@ -282,9 +282,15 @@ void shouldSupportFindAllWithPredicateAndSort() {
 		assertThat(users).contains(carter, dave, oliver);
 	}
 
-	@Test // DATAJPA-585
+	@Test // DATAJPA-585, 3761
 	void worksWithUnpagedPageable() {
+
 		assertThat(predicateExecutor.findAll(user.dateOfBirth.isNull(), Pageable.unpaged()).getContent()).hasSize(3);
+
+		Page<User> users = predicateExecutor.findAll(user.dateOfBirth.isNull(),
+				Pageable.unpaged(Sort.by(Direction.ASC, "firstname")));
+
+		assertThat(users).containsExactly(carter, dave, oliver);
 	}
 
 	@Test // DATAJPA-912