From 0a43667eb3eaa3aec50474496b759aab10e85f7f Mon Sep 17 00:00:00 2001
From: Julien Millau <mxjulien@gmail.com>
Date: Tue, 13 Nov 2018 01:35:13 +0100
Subject: [PATCH] #146 - Avoid executing IN-query with empty parameter list.

These queries are illegal in Postgres and possible other databases.

Original pull request: #147.
---
 .../repository/support/EnversRevisionRepositoryImpl.java     | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/main/java/org/springframework/data/envers/repository/support/EnversRevisionRepositoryImpl.java b/src/main/java/org/springframework/data/envers/repository/support/EnversRevisionRepositoryImpl.java
index 6680a80..f65ba80 100755
--- a/src/main/java/org/springframework/data/envers/repository/support/EnversRevisionRepositoryImpl.java
+++ b/src/main/java/org/springframework/data/envers/repository/support/EnversRevisionRepositoryImpl.java
@@ -52,6 +52,7 @@ import org.springframework.util.Assert;
  * @author Philipp Huegelmeyer
  * @author Michael Igler
  * @author Jens Schauder
+ * @author Julien Millau
  */
 public class EnversRevisionRepositoryImpl<T, ID, N extends Number & Comparable<N>> extends SimpleJpaRepository<T, ID>
 		implements RevisionRepository<T, ID, N> {
@@ -151,8 +152,8 @@ public class EnversRevisionRepositoryImpl<T, ID, N extends Number & Comparable<N
 			Collections.reverse(revisionNumbers);
 		}
 
-		if (pageable.getOffset() > revisionNumbers.size()) {
-			return new PageImpl<Revision<N, T>>(Collections.<Revision<N, T>> emptyList(), pageable, 0);
+		if (revisionNumbers.isEmpty() || pageable.getOffset() > revisionNumbers.size()) {
+			return Page.empty(pageable);
 		}
 
 		long upperBound = pageable.getOffset() + pageable.getPageSize();