fix: Correct array copy length in setPageRanges method to include all ranges

manuelsblanco · manuelsblanco · commit eee327c85ce8 · 2025-10-27T22:09:11.000+01:00
diff --git a/java/src/org/openqa/selenium/print/PrintOptions.java b/java/src/org/openqa/selenium/print/PrintOptions.java
@@ -70,7 +70,7 @@ public void setPageRanges(String firstRange, String... ranges) {
 
     this.pageRanges[0] = firstRange;
 
-    if (ranges.length > 0) System.arraycopy(ranges, 0, this.pageRanges, 1, ranges.length - 1);
+    if (ranges.length > 0) System.arraycopy(ranges, 0, this.pageRanges, 1, ranges.length);
   }
 
   public void setPageRanges(List<String> ranges) {

Original file line number	Diff line number	Diff line change
`@@ -70,7 +70,7 @@ public void setPageRanges(String firstRange, String... ranges) {`
`70`	`70`
`71`	`71`	`this.pageRanges[0] = firstRange;`
`72`	`72`
`73`		`- if (ranges.length > 0) System.arraycopy(ranges, 0, this.pageRanges, 1, ranges.length - 1);`
	`73`	`+ if (ranges.length > 0) System.arraycopy(ranges, 0, this.pageRanges, 1, ranges.length);`
`74`	`74`	`}`
`75`	`75`
`76`	`76`	`public void setPageRanges(List<String> ranges) {`