Dont forget to set the palette pos of the new palette record

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

diff --git a/src/java/org/apache/poi/hssf/model/Workbook.java b/src/java/org/apache/poi/hssf/model/Workbook.java
index e41867996b43765faaefd296e067fad777d295ba..2ef73c2f92c85811e18854dcfdd07010609a69d2 100644 (file)
--- a/src/java/org/apache/poi/hssf/model/Workbook.java
+++ b/src/java/org/apache/poi/hssf/model/Workbook.java
@@ -2086,6 +2086,7 @@ public class Workbook implements Model {
             palette = createPalette();
           //Add the palette record after the bof which is always the first record
           records.add(1, palette);
+          records.setPalettepos(1);
         }
         return palette;
     }

diff --git a/src/java/org/apache/poi/hssf/model/WorkbookRecordList.java b/src/java/org/apache/poi/hssf/model/WorkbookRecordList.java
index 90eefaefa0f0a42deaffe411309b8ca28de0e0c4..654ea88782d042d6f5d37263a31744051d71533b 100644 (file)
--- a/src/java/org/apache/poi/hssf/model/WorkbookRecordList.java
+++ b/src/java/org/apache/poi/hssf/model/WorkbookRecordList.java
@@ -103,7 +103,7 @@ public class WorkbookRecordList
         if (getBackuppos() >= pos) setBackuppos( backuppos + 1 );
         if (getNamepos() >= pos) setNamepos(namepos+1);
         if (getSupbookpos() >= pos) setSupbookpos(supbookpos+1);
-        if (getPalettepos() >= pos) setPalettepos( palettepos + 1 );
+        if ((getPalettepos() != -1) && (getPalettepos() >= pos)) setPalettepos( palettepos + 1 );
         if (getExternsheetPos() >= pos) setExternsheetPos(getExternsheetPos() + 1);
     }
 
@@ -128,7 +128,7 @@ public class WorkbookRecordList
         if (getBackuppos() >= pos) setBackuppos( backuppos - 1 );
         if (getNamepos() >= pos) setNamepos(getNamepos()-1);
         if (getSupbookpos() >= pos) setSupbookpos(getSupbookpos()-1);
-        if (getPalettepos() >= pos) setPalettepos( palettepos - 1 );
+        if ((getPalettepos() != -1) && (getPalettepos() >= pos)) setPalettepos( palettepos - 1 );
         if (getExternsheetPos() >= pos) setExternsheetPos( getExternsheetPos() -1);
     }