Bug 46885

This bug was created due to the poi-user posting by Paul Dobson on Mar 20.

The following code will cause Excel to report the workbook as corrupted, and attempt a repair on it:

String xlsxFilePath = "empty.xlsx";
FileInputStream in = new FileInputStream(xlsxFilePath);
Workbook wb = WorkbookFactory.create(in);
Sheet sh = wb.getSheet("Sheet1");
Row row = sh.createRow(1);
Cell cell = row.createCell(1);
cell.setCellFormula("NA()");
cell.setCellValue("Help, I will not change!"); //corrupts workbook
in.close();
FileOutputStream out = new FileOutputStream("out.xlsx");
wb.write(out);
out.close();


With some investigation, I found that Excel is missing the type attribute from the following XML fragment:
<c r="B2">
  <f>NA()</f>
  <v>Help, I will not change!</v>
</c>

By adding 't="str"' (as follows) the workbook opens properly.
<c r="B2" t="str">
  <f>NA()</f>
  <v>Help, I will not change!</v>
</c>


Here is a patch that seems to have the desired effect.  Can someone review please?

$ svn diff src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFCell.java
Index: src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFCell.java
===================================================================
--- src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFCell.java  (revision 755692)
+++ src/ooxml/java/org/apache/poi/xssf/usermodel/XSSFCell.java  (working copy)
@@ -217,6 +217,7 @@
             case CELL_TYPE_ERROR:
             case CELL_TYPE_FORMULA:
                 cell.setV(String.valueOf(value));
+                cell.setT(STCellType.N);
                 break;
             default:
                 cell.setT(STCellType.N);
@@ -303,6 +304,7 @@
         switch(cellType){
             case Cell.CELL_TYPE_FORMULA:
                 cell.setV(str.getString());
+                cell.setT(STCellType.STR);
                 break;
             default:
                 if(cell.getT() == STCellType.INLINE_STR) {