Apply sort from unpaged Pageable to query.

This commit makes sure to pass on a given Sort from an unpaged Pageable to the actual query.

Closes: #3476
Original Pull Request: #3517

Author: birariro

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

@@ -426,11 +426,6 @@ public List<T> findAll(Sort sort) {
 
 	@Override
 	public Page<T> findAll(Pageable pageable) {
-
-		if (pageable.isUnpaged()) {
-			return new PageImpl<>(findAll());
-		}
-
 		return findAll((Specification<T>) null, pageable);
 	}
 
@@ -717,8 +712,7 @@ protected <S extends T> Page<S> readPage(TypedQuery<S> query, final Class<S> dom
 	 */
 	protected TypedQuery<T> getQuery(@Nullable Specification<T> spec, Pageable pageable) {
 
-		Sort sort = pageable.isPaged() ? pageable.getSort() : Sort.unsorted();
-		return getQuery(spec, getDomainClass(), sort);
+		return getQuery(spec, getDomainClass(), pageable.getSort());
 	}
 
 	/**
@@ -731,8 +725,7 @@ protected TypedQuery<T> getQuery(@Nullable Specification<T> spec, Pageable pagea
 	protected <S extends T> TypedQuery<S> getQuery(@Nullable Specification<S> spec, Class<S> domainClass,
 			Pageable pageable) {
 
-		Sort sort = pageable.isPaged() ? pageable.getSort() : Sort.unsorted();
-		return getQuery(spec, domainClass, sort);
+		return getQuery(spec, domainClass, pageable.getSort());
 	}
 
 	/**

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

@@ -2887,6 +2887,18 @@ void existsByExampleNegative() {
 		assertThat(exists).isFalse();
 	}
 
+	@Test // GH-3476
+	void unPagedSortedQuery() {
+
+		flushTestUsers();
+
+		Sort sort = Sort.by(DESC, "firstname");
+		Page<User> firstPage = repository.findAll(PageRequest.of(0, 10, sort));
+		Page<User> secondPage = repository.findAll(Pageable.unpaged(sort));
+		assertThat(firstPage.getContent()).isEqualTo(secondPage.getContent());
+	}
+
+
 	@Test // DATAJPA-905
 	void executesPagedSpecificationSettingAnOrder() {