Bug 62758

createColumn corrupts an XSSFTable by assigning the new column an ID equal to the maximum ID of any existing column.  Unit test code is presented below. If bApplyBugFix is false, Excel will report a corrupted file when the file is open.  Otherwise, the file opens correctly.

It is possible the the bug fix for 62740 will fix this.

- David Gauntt

  public static void doUnitTest(File file) {
    final XSSFWorkbook workbook = new XSSFWorkbook();
    final XSSFSheet sheet = workbook.createSheet();

    createColumnUnitTest(sheet);

    try (OutputStream fileOut = new FileOutputStream(file)) {
      workbook.write(fileOut);
    } catch (Exception e) {
      System.err.println(e.getMessage());
    } finally {
      try {
        workbook.close();
      } catch (IOException e) {
        System.err.println(e.getMessage());
      }
    }
    System.out.println("doUnitTest: done");
    final Desktop desktop = Desktop.getDesktop();
    try {
      desktop.open(file);
    } catch (Exception e) {
      System.err.println(e.getMessage());
    }

  }

  private static void createColumnUnitTest(XSSFSheet sheet) {
    final String procName = "createColumnUnitTest";
    final boolean bApplyBugFix = true;
    final int NUMCOLS = 3, NUMROWS = 4;

    System.out.println(String.format("\r\n%s: bFixBug=%s", procName, bApplyBugFix));

    /* Fill a range with data */
    for (int i = 0; i < NUMROWS; i++) {
      XSSFRow row = sheet.createRow(i);
      for (int j = 0; j < NUMCOLS; j++) {
        XSSFCell localXSSFCell = row.createCell(j);
        if (i == 0) {
          localXSSFCell.setCellValue(String.format("Col%d", j + 1));
        } else {
          localXSSFCell.setCellValue(String.format("(%d,%d)", i + 1, j + 1));
        }
      }
    }

    /* Define a single column data range including headers */
    final AreaReference my_data_range = new AreaReference(new CellReference(0, 0),
        new CellReference(NUMROWS - 1, 1), SpreadsheetVersion.EXCEL2007);

    /* Create an object of type XSSFTable */
    final XSSFTable my_table = sheet.createTable(my_data_range);
    my_table.setDisplayName("testCreateColumn");

    /* Add a new column */
    System.out.println(my_table.getCTTable().toString());
    my_table.createColumn(null);
    my_table.updateHeaders();
    System.out.println(my_table.getCTTable().toString());

    if (bApplyBugFix) {
      final CTTable ctTable = my_table.getCTTable();
      final List<CTTableColumn> ctTableColumns = ctTable.getTableColumns().getTableColumnList();
      final long numCols = ctTableColumns.size();
      for (int n = 0; n < numCols; ++n) {
        ctTableColumns.get(n).setId(n + 1);
      }
      System.out.println("After applying bug fix");
      System.out.println(my_table.getCTTable().toString());
    }

  }