diff options
author | Jochen Nitschke <j.nitschke+logerrit@ok.de> | 2017-02-15 16:19:00 +0100 |
---|---|---|
committer | Jochen Nitschke <j.nitschke+logerrit@ok.de> | 2017-02-15 16:50:35 +0000 |
commit | deff2acd61de53e29a8cf1eed7f8db458244dab0 (patch) | |
tree | b233063ff7f53f768a72f2e200d848aeade773e1 /sc/source/filter/oox | |
parent | 2af4221bd1f91b4cef22d6b1867a250e3780f6e3 (diff) |
tdf#105909 auto filter state lost on xlsx import
in commit 77dc03c29501234bb07170004a7b8204987bad0c
the new variant of FormulaProcessorBase::extractCellRange(..)
replaced a CellRangeAddress& ApiCellRangeList::front() with
ScAddress& ApiCellRangeList::getBaseAddress() which returned only
the start address of the front range. This went unnoticed because
ScRange has a non explicit constructor from ScAddress.
later ApiCellRangeList::getBaseAddress() got replaced
by ScRangeList::GetTopLeftCorner().
use ScRangeList::front() to fix this regression.
Change-Id: I338e5d3054b62d5dd191492a38451d0adc85ae0d
Reviewed-on: https://gerrit.libreoffice.org/34304
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Jochen Nitschke <j.nitschke+logerrit@ok.de>
Diffstat (limited to 'sc/source/filter/oox')
-rw-r--r-- | sc/source/filter/oox/formulabase.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sc/source/filter/oox/formulabase.cxx b/sc/source/filter/oox/formulabase.cxx index d0e73d5d54dc..fa096ca7392c 100644 --- a/sc/source/filter/oox/formulabase.cxx +++ b/sc/source/filter/oox/formulabase.cxx @@ -1603,7 +1603,7 @@ bool FormulaProcessorBase::extractCellRange( ScRange& orRange, lclProcessRef( aRanges, extractReference( rTokens ), bAllowRelative, -1 ); if( !aRanges.empty() ) { - orRange = aRanges.GetTopLeftCorner(); + orRange = *aRanges.front(); return true; } return false; |