Bug 58106: when a cell with a 'master' shared formula is removed, the next cell in the range becomes the master

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1819623 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Yegor Kozlov 2017-12-30 16:34:57 +00:00
parent 176b557f0d
commit 3d4acce3a7
4 changed files with 82 additions and 1 deletions

View File

@ -559,6 +559,7 @@ public final class XSSFCell implements Cell {
if (formula == null) {
wb.onDeleteFormula(this);
if (_cell.isSetF()) {
_row.getSheet().onDeleteFormula(this);
_cell.unsetF();
}
return;
@ -963,6 +964,9 @@ public final class XSSFCell implements Cell {
notifyArrayFormulaChanging();
}
if(prevType == CellType.FORMULA && cellType != CellType.FORMULA) {
if (_cell.isSetF()) {
_row.getSheet().onDeleteFormula(this);
}
getSheet().getWorkbook().onDeleteFormula(this);
}

View File

@ -4455,7 +4455,40 @@ public class XSSFSheet extends POIXMLDocumentPart implements Sheet {
removeRelation(part.getDocumentPart(), true);
}
}
/**
* when a cell with a 'master' shared formula is removed, the next cell in the range becomes the master
*/
protected void onDeleteFormula(XSSFCell cell){
CTCellFormula f = cell.getCTCell().getF();
if (f != null && f.getT() == STCellFormulaType.SHARED && f.isSetRef() && f.getStringValue() != null) {
CellRangeAddress ref = CellRangeAddress.valueOf(f.getRef());
if(ref.getNumberOfCells() > 1){
DONE:
for(int i = cell.getRowIndex(); i <= ref.getLastRow(); i++){
XSSFRow row = getRow(i);
if(row != null) for(int j = cell.getColumnIndex(); j <= ref.getLastColumn(); j++){
XSSFCell nextCell = row.getCell(j);
if(nextCell != null && nextCell != cell){
CTCellFormula nextF = nextCell.getCTCell().getF();
nextF.setStringValue(nextCell.getCellFormula());
CellRangeAddress nextRef = new CellRangeAddress(
nextCell.getRowIndex(), ref.getLastRow(),
nextCell.getColumnIndex(), ref.getLastColumn());
nextF.setRef(nextRef.formatAsString());
sharedFormulas.put((int)nextF.getSi(), nextF);
break DONE;
}
}
}
}
}
}
/**
* Determine the OleObject which links shapes with embedded resources
*

View File

@ -48,6 +48,7 @@ import org.apache.poi.xssf.XSSFTestDataSamples;
import org.apache.poi.xssf.model.SharedStringsTable;
import org.junit.Test;
import org.openxmlformats.schemas.spreadsheetml.x2006.main.CTCell;
import org.openxmlformats.schemas.spreadsheetml.x2006.main.CTCellFormula;
import org.openxmlformats.schemas.spreadsheetml.x2006.main.STCellFormulaType;
import org.openxmlformats.schemas.spreadsheetml.x2006.main.STCellType;
@ -701,4 +702,47 @@ public final class TestXSSFCell extends BaseTestXCell {
}
}
@Test
public void testBug58106RemoveSharedFormula() throws Exception {
try (XSSFWorkbook wb = XSSFTestDataSamples.openSampleWorkbook("58106.xlsx")) {
XSSFSheet sheet = wb.getSheetAt(0);
XSSFRow row = sheet.getRow(12);
XSSFCell cell = row.getCell(1);
CTCellFormula f = cell.getCTCell().getF();
assertEquals("B13:G13", f.getRef());
assertEquals("SUM(B1:B3)", f.getStringValue());
assertEquals(0, f.getSi());
assertEquals(STCellFormulaType.SHARED, f.getT());
for(char i = 'C'; i <= 'G'; i++){
XSSFCell sc =row.getCell(i-'A');
CTCellFormula sf = sc.getCTCell().getF();
assertFalse(sf.isSetRef());
assertEquals("", sf.getStringValue());
assertEquals(0, sf.getSi());
assertEquals(STCellFormulaType.SHARED, sf.getT());
}
assertEquals("B13:G13", sheet.getSharedFormula(0).getRef());
cell.setCellType(CellType.NUMERIC);
assertFalse(cell.getCTCell().isSetF());
XSSFCell nextFormulaMaster = row.getCell(2);
assertEquals("C13:G13", nextFormulaMaster.getCTCell().getF().getRef());
assertEquals("SUM(C1:C3)", nextFormulaMaster.getCTCell().getF().getStringValue());
assertEquals(0, nextFormulaMaster.getCTCell().getF().getSi());
for(char i = 'D'; i <= 'G'; i++){
XSSFCell sc =row.getCell(i-'A');
CTCellFormula sf = sc.getCTCell().getF();
assertFalse(sf.isSetRef());
assertEquals("", sf.getStringValue());
assertEquals(0, sf.getSi());
assertEquals(STCellFormulaType.SHARED, sf.getT());
}
assertEquals("C13:G13", sheet.getSharedFormula(0).getRef());
}
}
}

Binary file not shown.