/**
* Creates a new Table, and associates it with this Sheet.
+ * <p>
+ * The table is assigned a default display name (since 4.1.1) which can be overridden
+ * by calling {@code setDisplayName}. The default display name is guaranteed to not conflict
+ * with the names of any {@code XSSFName} or {@code XSSFTable} in the workbook.
*
* @param tableArea
* the area that the table should cover, should not be null
table.setArea(tableArea);
}
- // Bug 62906: Must set a display name; can be overridden using setDisplayName
- final String displayName = "Table" + tableNumber;
- table.setDisplayName(displayName);
+ // Set the default name of the table. This must not conflict with any defined names.
+ while(tableNumber<Integer.MAX_VALUE) {
+ final String displayName="Table"+tableNumber;
+ if(getWorkbook().getTable(displayName) == null &&
+ getWorkbook().getName(displayName) == null) {
+ table.setDisplayName(displayName);
+ table.setName(displayName);
+ break;
+ }
+ ++tableNumber;
+ }
return table;
}
package org.apache.poi.xssf.usermodel;
+import org.apache.poi.ss.SpreadsheetVersion;
import org.apache.poi.ss.usermodel.Cell;
import org.apache.poi.ss.util.AreaReference;
import org.apache.poi.ss.util.CellReference;
import java.io.IOException;
import java.util.ArrayList;
import java.util.List;
+import java.util.Locale;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertFalse;
table1.setDisplayName("");
}
}
+
+ /**
+ * Delete table2, and create a named range in sheet0; it should automatically be assigned the name "Table4"
+ */
+ @Test
+ public void testBug63401And62906() throws IOException {
+ try (XSSFWorkbook workbook = new XSSFWorkbook()) {
+ XSSFSheet sheet0 = workbook.createSheet();
+ XSSFTable table = addTable(sheet0, 3, 0, 2, 2);
+
+ final String procName = "testXSSFTableGetName";
+ final String name = table.getName();
+ System.out.println(String.format(Locale.ROOT, "%s: table.getName=%s", procName, name));
+ }
+ }
+
+ private static XSSFTable addTable(XSSFSheet sheet,int nRow, int nCol, int nNumRows, int nNumCols) {
+ for (int i = 0; i < nNumRows; i++) {
+ XSSFRow row = sheet.createRow(i + nRow);
+ for (int j = 0; j < nNumCols; j++) {
+ XSSFCell localXSSFCell = row.createCell(j + nCol);
+ if (i == 0) {
+ localXSSFCell.setCellValue(String.format(Locale.ROOT, "Col%d", j + 1));
+ } else {
+ localXSSFCell.setCellValue(String.format(Locale.ROOT, "(%d,%d)", i + 1, j + 1));
+ }
+ }
+ }
+ final CellReference upperLeft = new CellReference(nRow, nCol);
+ final CellReference lowerRight = new CellReference(nNumRows - 1, nNumCols - 1);
+ final AreaReference area = new AreaReference(upperLeft, lowerRight, SpreadsheetVersion.EXCEL2007);
+ return sheet.createTable(area);
+ }
}