Enable CFRule12 use #58130

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

diff --git a/src/java/org/apache/poi/hssf/dev/BiffViewer.java b/src/java/org/apache/poi/hssf/dev/BiffViewer.java
index 58753048c7ee3d5ad3bf2de155096f2aced2bbfb..3afd25a4ace44565f9c924ccefa524a10dd978e6 100644 (file)
--- a/src/java/org/apache/poi/hssf/dev/BiffViewer.java
+++ b/src/java/org/apache/poi/hssf/dev/BiffViewer.java
@@ -179,6 +179,8 @@ public final class BiffViewer {
             case CFHeaderRecord.sid:          return new CFHeaderRecord(in);
             case CFHeader12Record.sid:        return new CFHeader12Record(in);
             case CFRuleRecord.sid:            return new CFRuleRecord(in);
+            case CFRule12Record.sid:          return new CFRule12Record(in);
+            // TODO Add CF Ex, and remove from UnknownRecord 
             case CalcCountRecord.sid:         return new CalcCountRecord(in);
             case CalcModeRecord.sid:          return new CalcModeRecord(in);
             case CategorySeriesAxisRecord.sid:return new CategorySeriesAxisRecord(in);

diff --git a/src/java/org/apache/poi/hssf/record/CFRuleBase.java b/src/java/org/apache/poi/hssf/record/CFRuleBase.java
index b7db3a7a09c7e18f42d63faeeaa044cd3fd44927..449594414fab19b5e87b50568fd243b85882ef48 100644 (file)
--- a/src/java/org/apache/poi/hssf/record/CFRuleBase.java
+++ b/src/java/org/apache/poi/hssf/record/CFRuleBase.java
@@ -347,7 +347,7 @@ public abstract class CFRuleBase extends StandardRecord {
     }
     
     protected int getFormattingBlockSize() {
-        return
+        return 6 +
           (containsFontFormattingBlock()?_fontFormatting.getRawRecord().length:0)+
           (containsBorderFormattingBlock()?8:0)+
           (containsPatternFormattingBlock()?4:0);

diff --git a/src/java/org/apache/poi/hssf/record/CFRuleRecord.java b/src/java/org/apache/poi/hssf/record/CFRuleRecord.java
index a00fde73116e3dd231a2f429813e8a5a7818a5d9..52678c0fda930e6cd8ca41aec936bb4539e77eef 100644 (file)
--- a/src/java/org/apache/poi/hssf/record/CFRuleRecord.java
+++ b/src/java/org/apache/poi/hssf/record/CFRuleRecord.java
@@ -114,7 +114,7 @@ public final class CFRuleRecord extends CFRuleBase {
     }
 
     protected int getDataSize() {
-        return 12 + getFormattingBlockSize() +
+        return 6 + getFormattingBlockSize() +
                getFormulaSize(getFormula1())+
                getFormulaSize(getFormula2());
     }

diff --git a/src/java/org/apache/poi/hssf/record/RecordFactory.java b/src/java/org/apache/poi/hssf/record/RecordFactory.java
index 6163fc0009a5c414e689722395c5c7e287bfb6ca..c1a0f5707e08444d5c0ad55c83da0a9e136db014 100644 (file)
--- a/src/java/org/apache/poi/hssf/record/RecordFactory.java
+++ b/src/java/org/apache/poi/hssf/record/RecordFactory.java
@@ -151,6 +151,7 @@ public final class RecordFactory {
         CFHeaderRecord.class,
         CFHeader12Record.class,
         CFRuleRecord.class,
+        CFRule12Record.class,
         ChartRecord.class,
         ChartTitleFormatRecord.class,
         CodepageRecord.class,

diff --git a/src/testcases/org/apache/poi/hssf/record/TestCFRuleRecord.java b/src/testcases/org/apache/poi/hssf/record/TestCFRuleRecord.java
index e562f52f4fdace87198d8aa1ba417a47daffee01..3be13558b861f7ee1779873e57aec2d722e294cd 100644 (file)
--- a/src/testcases/org/apache/poi/hssf/record/TestCFRuleRecord.java
+++ b/src/testcases/org/apache/poi/hssf/record/TestCFRuleRecord.java
@@ -402,8 +402,7 @@ public final class TestCFRuleRecord extends TestCase {
         assertArrayEquals(serializedRecord, serializedClone);
     }
     
-    // TODO Fix this test!
-    public void IGNORED_testBug57231_rewrite() {
+    public void testBug57231_rewrite() {
         HSSFWorkbook wb = HSSFITestDataProvider.instance.openSampleWorkbook("57231_MixedGasReport.xls");
         assertEquals(7, wb.getNumberOfSheets());
         wb = HSSFITestDataProvider.instance.writeOutAndReadBack(wb);