[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 5422993..5c6f167 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
@@ -159,6 +159,26 @@
}
@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()) {
XSSFSheet sheet = workbook.createSheet("Sheet 1");
@@ -2015,4 +2035,5 @@
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 90b9556..5bd1b3d 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 @@
}
// 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 0000000..337f425
--- /dev/null
+++ b/test-data/spreadsheet/HeaderFooterComplexFormats.xlsx
Binary files differ