diff options
-rw-r--r-- | sc/qa/unit/ucalc.hxx | 2 | ||||
-rw-r--r-- | sc/qa/unit/ucalc_sharedformula.cxx | 53 | ||||
-rw-r--r-- | sc/source/core/data/formulacell.cxx | 5 |
3 files changed, 59 insertions, 1 deletions
diff --git a/sc/qa/unit/ucalc.hxx b/sc/qa/unit/ucalc.hxx index 9ad69b663701..e97bb8df24e7 100644 --- a/sc/qa/unit/ucalc.hxx +++ b/sc/qa/unit/ucalc.hxx @@ -220,6 +220,7 @@ public: void testSearchCells(); void testSharedFormulas(); void testSharedFormulasRefUpdate(); + void testSharedFormulasRefUpdateRange(); void testSharedFormulasCopyPaste(); void testFormulaPosition(); @@ -350,6 +351,7 @@ public: CPPUNIT_TEST(testSearchCells); CPPUNIT_TEST(testSharedFormulas); CPPUNIT_TEST(testSharedFormulasRefUpdate); + CPPUNIT_TEST(testSharedFormulasRefUpdateRange); CPPUNIT_TEST(testSharedFormulasCopyPaste); CPPUNIT_TEST(testFormulaPosition); CPPUNIT_TEST(testJumpToPrecedentsDependents); diff --git a/sc/qa/unit/ucalc_sharedformula.cxx b/sc/qa/unit/ucalc_sharedformula.cxx index 1ecff32fb098..00378ddb2155 100644 --- a/sc/qa/unit/ucalc_sharedformula.cxx +++ b/sc/qa/unit/ucalc_sharedformula.cxx @@ -391,6 +391,59 @@ void Test::testSharedFormulasRefUpdate() m_pDoc->DeleteTab(0); } +void Test::testSharedFormulasRefUpdateRange() +{ + m_pDoc->InsertTab(0, "Test"); + + // Insert values to A3:A5. + m_pDoc->SetValue(ScAddress(0,2,0), 1); + m_pDoc->SetValue(ScAddress(0,3,0), 2); + m_pDoc->SetValue(ScAddress(0,4,0), 3); + + // Insert formulas to B3:B5. + m_pDoc->SetString(ScAddress(1,2,0), "=SUM($A$3:$A$5)"); + m_pDoc->SetString(ScAddress(1,3,0), "=SUM($A$3:$A$5)"); + m_pDoc->SetString(ScAddress(1,4,0), "=SUM($A$3:$A$5)"); + + if (!checkFormula(*m_pDoc, ScAddress(1,2,0), "SUM($A$3:$A$5)")) + CPPUNIT_FAIL("Wrong formula"); + if (!checkFormula(*m_pDoc, ScAddress(1,3,0), "SUM($A$3:$A$5)")) + CPPUNIT_FAIL("Wrong formula"); + if (!checkFormula(*m_pDoc, ScAddress(1,4,0), "SUM($A$3:$A$5)")) + CPPUNIT_FAIL("Wrong formula"); + + // B3:B5 should be shared. + const ScFormulaCell* pFC = m_pDoc->GetFormulaCell(ScAddress(1,2,0)); + CPPUNIT_ASSERT_MESSAGE("B3 should be shared.", pFC && pFC->IsShared()); + pFC = m_pDoc->GetFormulaCell(ScAddress(1,3,0)); + CPPUNIT_ASSERT_MESSAGE("B4 should be shared.", pFC && pFC->IsShared()); + pFC = m_pDoc->GetFormulaCell(ScAddress(1,4,0)); + CPPUNIT_ASSERT_MESSAGE("B3 should be shared.", pFC && pFC->IsShared()); + + // Insert 2 rows at row 1. + m_pDoc->InsertRow(ScRange(0,0,0,MAXCOL,1,0)); + + // B5:B7 should be shared. + pFC = m_pDoc->GetFormulaCell(ScAddress(1,4,0)); + CPPUNIT_ASSERT_MESSAGE("B5 should be shared.", pFC && pFC->IsShared()); + pFC = m_pDoc->GetFormulaCell(ScAddress(1,5,0)); + CPPUNIT_ASSERT_MESSAGE("B6 should be shared.", pFC && pFC->IsShared()); + pFC = m_pDoc->GetFormulaCell(ScAddress(1,6,0)); + CPPUNIT_ASSERT_MESSAGE("B7 should be shared.", pFC && pFC->IsShared()); + + CPPUNIT_ASSERT_EQUAL(static_cast<SCROW>(4), pFC->GetSharedTopRow()); + CPPUNIT_ASSERT_EQUAL(static_cast<SCROW>(3), pFC->GetSharedLength()); + + if (!checkFormula(*m_pDoc, ScAddress(1,4,0), "SUM($A$5:$A$7)")) + CPPUNIT_FAIL("Wrong formula"); + if (!checkFormula(*m_pDoc, ScAddress(1,5,0), "SUM($A$5:$A$7)")) + CPPUNIT_FAIL("Wrong formula"); + if (!checkFormula(*m_pDoc, ScAddress(1,6,0), "SUM($A$5:$A$7)")) + CPPUNIT_FAIL("Wrong formula"); + + m_pDoc->DeleteTab(0); +} + void Test::testSharedFormulasCopyPaste() { m_pDoc->InsertTab(0, "Test"); diff --git a/sc/source/core/data/formulacell.cxx b/sc/source/core/data/formulacell.cxx index 143351e8326e..0f248a26705f 100644 --- a/sc/source/core/data/formulacell.cxx +++ b/sc/source/core/data/formulacell.cxx @@ -2186,7 +2186,10 @@ bool ScFormulaCell::UpdatePosOnShift( const sc::RefUpdateContext& rCxt ) // This formula cell itself is being shifted during cell range // insertion or deletion. Update its position. - if (mxGroup && mxGroup->mnStart == aPos.Row()) + + if (mxGroup && (mxGroup->mnStart+mxGroup->mnLength-1) == aPos.Row()) + // For a shared formula cell, update its group start position only + // when it's the last cell of the group. mxGroup->mnStart += rCxt.mnRowDelta; aPos.Move(rCxt.mnColDelta, rCxt.mnRowDelta, rCxt.mnTabDelta); |