BUG 27496: get/setPageBreak and get/getColumnBreak now work correctly if a template excel file is loaded which does not contain PaneRecords is loaded.

git-svn-id: https://svn.apache.org/repos/asf/jakarta/poi/trunk@441652 13f79535-47bb-0310-9956-ffa450edef68

1 files changed, 21 insertions, 13 deletions

diff --git a/src/java/org/apache/poi/hssf/usermodel/HSSFSheet.java b/src/java/org/apache/poi/hssf/usermodel/HSSFSheet.java
index 99a2b5fef3..006d096d05 100644
--- a/src/java/org/apache/poi/hssf/usermodel/HSSFSheet.java
+++ b/src/java/org/apache/poi/hssf/usermodel/HSSFSheet.java
@@ -1204,34 +1204,42 @@ public class HSSFSheet
     
     /**
      * Retrieves all the horizontal page breaks
-     * @return all the horizontal page breaks
+     * @return all the horizontal page breaks, or null if there are no row page breaks
      */
     public int[] getRowBreaks(){
-    	//we can probably cache this information, but this should be a sparsely used function 
-    	int[] returnValue = new int[sheet.getNumRowBreaks()];
-    	Iterator iterator = sheet.getRowBreaks();
-    	int i = 0;
-    	while (iterator.hasNext()) {
+    	//we can probably cache this information, but this should be a sparsely used function
+    	int count = sheet.getNumRowBreaks();
+    	if (count > 0) {
+    	  int[] returnValue = new int[count];
+    	  Iterator iterator = sheet.getRowBreaks();
+    	  int i = 0;
+    	  while (iterator.hasNext()) {
     		PageBreakRecord.Break breakItem = (PageBreakRecord.Break)iterator.next();
     		returnValue[i++] = (int)breakItem.main;
+    	  }
+    	  return returnValue;
     	}
-    	return returnValue;
+    	return null;
     }
 
     /**
      * Retrieves all the vertical page breaks
-     * @return all the vertical page breaks
+     * @return all the vertical page breaks, or null if there are no column page breaks
      */
     public short[] getColumnBreaks(){
     	//we can probably cache this information, but this should be a sparsely used function 
-    	short[] returnValue = new short[sheet.getNumColumnBreaks()];
-    	Iterator iterator = sheet.getColumnBreaks();
-    	int i = 0;
-    	while (iterator.hasNext()) {
+    	int count = sheet.getNumColumnBreaks();
+    	if (count > 0) {
+    	  short[] returnValue = new short[count];
+    	  Iterator iterator = sheet.getColumnBreaks();
+    	  int i = 0;
+    	  while (iterator.hasNext()) {
     		PageBreakRecord.Break breakItem = (PageBreakRecord.Break)iterator.next();
     		returnValue[i++] = breakItem.main;
+    	  }
+    	  return returnValue;
     	}
-    	return returnValue;
+    	return null;
     }