diff options
author | Markus Mohrhard <markus.mohrhard@googlemail.com> | 2016-10-01 01:31:09 +0200 |
---|---|---|
committer | Markus Mohrhard <markus.mohrhard@googlemail.com> | 2016-10-01 02:06:23 +0000 |
commit | 5f658dc87794036182aa1f8586d82c5ee997cd8b (patch) | |
tree | 163168c022de64266238818c339f9d3e96ceea57 /sc | |
parent | cf4ff92144726a91508fcaf4be21170eac5cb99a (diff) |
tdf#101910, mark ScRangeList updated when DeleteArea was used
Change-Id: I98390b28c5a1bbbd391ff054b2f574951c7c00c4
Reviewed-on: https://gerrit.libreoffice.org/29422
Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
Tested-by: Jenkins <ci@libreoffice.org>
Diffstat (limited to 'sc')
-rw-r--r-- | sc/inc/rangelst.hxx | 2 | ||||
-rw-r--r-- | sc/source/core/tool/rangelst.cxx | 22 |
2 files changed, 20 insertions, 4 deletions
diff --git a/sc/inc/rangelst.hxx b/sc/inc/rangelst.hxx index fb1df785f93c..18eb33da90a6 100644 --- a/sc/inc/rangelst.hxx +++ b/sc/inc/rangelst.hxx @@ -61,7 +61,7 @@ public: /** For now this method assumes that nTab1 == nTab2 * The algorithm will be much more complicated if nTab1 != nTab2 */ - void DeleteArea( SCCOL nCol1, SCROW nRow1, SCTAB nTab1, SCCOL nCol2, + bool DeleteArea( SCCOL nCol1, SCROW nRow1, SCTAB nTab1, SCCOL nCol2, SCROW nRow2, SCTAB nTab2 ); const ScRange* Find( const ScAddress& ) const; diff --git a/sc/source/core/tool/rangelst.cxx b/sc/source/core/tool/rangelst.cxx index 6d3f1dbbcdb0..208135232e31 100644 --- a/sc/source/core/tool/rangelst.cxx +++ b/sc/source/core/tool/rangelst.cxx @@ -397,11 +397,11 @@ bool ScRangeList::UpdateReference( { if(nDx < 0) { - DeleteArea(nCol1+nDx, nRow1, nTab1, nCol1-1, nRow2, nTab2); + bChanged = DeleteArea(nCol1+nDx, nRow1, nTab1, nCol1-1, nRow2, nTab2); } if(nDy < 0) { - DeleteArea(nCol1, nRow1+nDy, nTab1, nCol2, nRow1-1, nTab2); + bChanged = DeleteArea(nCol1, nRow1+nDy, nTab1, nCol2, nRow1-1, nTab2); } SAL_WARN_IF(nDx < 0 && nDy < 0, "sc", "nDx and nDy are negative, check why"); } @@ -939,9 +939,10 @@ bool handleFourRanges( const ScRange& rDelRange, ScRange* p, std::vector<ScRange } -void ScRangeList::DeleteArea( SCCOL nCol1, SCROW nRow1, SCTAB nTab1, +bool ScRangeList::DeleteArea( SCCOL nCol1, SCROW nRow1, SCTAB nTab1, SCCOL nCol2, SCROW nRow2, SCTAB nTab2 ) { + bool bChanged = false; ScRange aRange( nCol1, nRow1, nTab1, nCol2, nRow2, nTab2 ); for(size_t i = 0; i < maRanges.size();) { @@ -949,6 +950,7 @@ void ScRangeList::DeleteArea( SCCOL nCol1, SCROW nRow1, SCTAB nTab1, { ScRange* pRange = Remove(i); delete pRange; + bChanged = true; } else ++i; @@ -974,12 +976,18 @@ void ScRangeList::DeleteArea( SCCOL nCol1, SCROW nRow1, SCTAB nTab1, // r.aStart.X() <= p.aStart.X() && r.aEnd.X() >= p.aEnd.X() // && ( r.aStart.Y() <= p.aStart.Y() || r.aEnd.Y() >= r.aEnd.Y() ) if(handleOneRange( aRange, *itr )) + { + bChanged = true; continue; + } // getting two ranges // r.aStart.X() else if(handleTwoRanges( aRange, *itr, aNewRanges )) + { + bChanged = true; continue; + } // getting 3 ranges // r.aStart.X() > p.aStart.X() && r.aEnd.X() >= p.aEnd.X() @@ -988,16 +996,24 @@ void ScRangeList::DeleteArea( SCCOL nCol1, SCROW nRow1, SCTAB nTab1, // r.aStart.X() <= p.aStart.X() && r.aEnd.X() < p.aEnd.X() // && r.aStart.Y() > p.aStart.Y() && r.aEnd.Y() < p.aEnd.Y() else if(handleThreeRanges( aRange, *itr, aNewRanges )) + { + bChanged = true; continue; + } // getting 4 ranges // r.aStart.X() > p.aStart.X() && r.aEnd().X() < p.aEnd.X() // && r.aStart.Y() > p.aStart.Y() && r.aEnd().Y() < p.aEnd.Y() else if(handleFourRanges( aRange, *itr, aNewRanges )) + { + bChanged = true; continue; + } } for(vector<ScRange>::iterator itr = aNewRanges.begin(); itr != aNewRanges.end(); ++itr) Join( *itr); + + return bChanged; } const ScRange* ScRangeList::Find( const ScAddress& rAdr ) const |