From: PJ Fanning <fanningpj@apache.org>
Date: Fri, 24 Sep 2021 15:05:10 +0000 (+0000)
Subject: [bug-65595] Strip colour formatting in headers and footers
X-Git-Tag: REL_5_2_0~495
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=abe22ef8f75df2213b41b986c4201835bdd55fe3;p=poi.git

[bug-65595] Strip colour formatting in headers and footers

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

diff --git a/poi-ooxml/src/test/java/org/apache/poi/xssf/usermodel/TestXSSFSheet.java b/poi-ooxml/src/test/java/org/apache/poi/xssf/usermodel/TestXSSFSheet.java
index 54229938e4..5c6f1670f1 100644
--- a/poi-ooxml/src/test/java/org/apache/poi/xssf/usermodel/TestXSSFSheet.java
+++ b/poi-ooxml/src/test/java/org/apache/poi/xssf/usermodel/TestXSSFSheet.java
@@ -158,6 +158,26 @@ public final class TestXSSFSheet extends BaseTestXSheet {
         }
     }
 
+    @Test
+    void headerFooterWithFormat() throws IOException {
+        try (XSSFWorkbook wb1 = XSSFTestDataSamples.openSampleWorkbook("HeaderFooterComplexFormats.xlsx")) {
+            XSSFOddHeader hdr;
+            XSSFOddFooter ftr;
+
+            XSSFSheet s1 = wb1.getSheetAt(0);
+            assertNotNull(s1.getHeader());
+            assertNotNull(s1.getFooter());
+            hdr = (XSSFOddHeader) s1.getHeader();
+            ftr = (XSSFOddFooter) s1.getFooter();
+
+            // The following is how the header/footer appear in Excel (with formatting removed)
+            hdr.setAreFieldsStripped(true);
+            assertEquals("Header Bold RedUnderlined Bolditalics", hdr.getCenter());
+            ftr.setAreFieldsStripped(true);
+            assertEquals("Footer ArialBlue TahomaBoldGreen", ftr.getCenter());
+        }
+    }
+
     @Test
     void getAllHeadersFooters() throws IOException {
         try (XSSFWorkbook workbook = new XSSFWorkbook()) {
@@ -2015,4 +2035,5 @@ public final class TestXSSFSheet extends BaseTestXSheet {
             assertEquals("/xl/drawings/vmlDrawing2.vml", vml2.getPackagePart().getPartName().getName());
         }
     }
+
 }
diff --git a/poi/src/main/java/org/apache/poi/hssf/usermodel/HeaderFooter.java b/poi/src/main/java/org/apache/poi/hssf/usermodel/HeaderFooter.java
index 90b9556923..5bd1b3d0be 100644
--- a/poi/src/main/java/org/apache/poi/hssf/usermodel/HeaderFooter.java
+++ b/poi/src/main/java/org/apache/poi/hssf/usermodel/HeaderFooter.java
@@ -295,9 +295,11 @@ public abstract class HeaderFooter implements org.apache.poi.ss.usermodel.Header
         }
 
         // Now do the tricky, dynamic ones
-        // These are things like font sizes and font names
+        // These are things like font sizes, font names and colours
         text = text.replaceAll("\\&\\d+", "");
         text = text.replaceAll("\\&\".*?,.*?\"", "");
+        text = text.replaceAll("\\&K[\\dA-F]{6}", "");
+        text = text.replaceAll("\\&K[\\d]{2}[+][\\d]{3}", "");
 
         // All done
         return text;
diff --git a/test-data/spreadsheet/HeaderFooterComplexFormats.xlsx b/test-data/spreadsheet/HeaderFooterComplexFormats.xlsx
new file mode 100644
index 0000000000..337f425440
Binary files /dev/null and b/test-data/spreadsheet/HeaderFooterComplexFormats.xlsx differ