Fix a bug in SimpleConnection where if a 'last' argument is used that was larger than the data set an error was thrown.

misterzirillo · dminkovsky · commit 7b87d5487bb7 · 2016-10-25T12:53:49.000-04:00
diff --git a/src/main/java/graphql/relay/SimpleListConnection.java b/src/main/java/graphql/relay/SimpleListConnection.java
@@ -55,7 +55,7 @@ public Object get(DataFetchingEnvironment environment) {
             edges = edges.subList(0, first <= edges.size() ? first : edges.size());
         }
         if (last != null) {
-            edges = edges.subList(edges.size() - last, edges.size());
+            edges = edges.subList( last > edges.size() ? 0 : edges.size() - last, edges.size());
         }
 
         if (edges.size() == 0) {

Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,7 @@ public Object get(DataFetchingEnvironment environment) {`
`55`	`55`	`edges = edges.subList(0, first <= edges.size() ? first : edges.size());`
`56`	`56`	`}`
`57`	`57`	`if (last != null) {`
`58`		`- edges = edges.subList(edges.size() - last, edges.size());`
	`58`	`+ edges = edges.subList( last > edges.size() ? 0 : edges.size() - last, edges.size());`
`59`	`59`	`}`
`60`	`60`
`61`	`61`	`if (edges.size() == 0) {`