summaryrefslogtreecommitdiff
path: root/sc/source/ui/view
diff options
context:
space:
mode:
authorJulien Nabet <serval2412@yahoo.fr>2018-07-26 11:36:08 +0200
committerEike Rathke <erack@redhat.com>2018-09-03 14:39:15 +0200
commit73af06a6a7fcb2d92d36a45bbe54395bcba8dfb0 (patch)
tree30e3119c92e973135f40f402ca8963e4bc69a28b /sc/source/ui/view
parent6c21fb251ba6bb2455c947583709262deddf5e4a (diff)
tdf#37268: use also sheet local range in Pivot
There are 2 types of range names: - those global to the document - those specific to a sheet Before this patch, Pivot could only see global range names There are 2 parts on the patch: 1) ScCellShell::ExecuteDataPilotDialog() Retrieve all the range names and use: - for sheets range names: <scope>.<range name> - for global range names: <range name> 2) ScSheetSourceDesc::GetSourceRange() Search about the presence of . to know if it's a global or sheet name range Change-Id: I92ac321e1475516cce7ee42b6e6038c231d0514b Reviewed-on: https://gerrit.libreoffice.org/58070 Tested-by: Jenkins Reviewed-by: Eike Rathke <erack@redhat.com>
Diffstat (limited to 'sc/source/ui/view')
-rw-r--r--sc/source/ui/view/cellsh1.cxx30
1 files changed, 25 insertions, 5 deletions
diff --git a/sc/source/ui/view/cellsh1.cxx b/sc/source/ui/view/cellsh1.cxx
index 7722f9ca1c1b..45c47ef492b7 100644
--- a/sc/source/ui/view/cellsh1.cxx
+++ b/sc/source/ui/view/cellsh1.cxx
@@ -17,6 +17,7 @@
* the License at http://www.apache.org/licenses/LICENSE-2.0 .
*/
+#include <globalnames.hxx>
#include <config_features.h>
#include <com/sun/star/i18n/TextConversionOption.hpp>
@@ -2794,12 +2795,31 @@ void ScCellShell::ExecuteDataPilotDialog()
pTabViewShell->GetFrameWeld(), bEnableExt));
// Populate named ranges (if any).
- ScRangeName* pRangeName = pDoc->GetRangeName();
- if (pRangeName)
+ // We must take into account 2 types of scope : global doc and sheets
+ // for global doc: <name of the range>
+ // for sheets: <sheetname>.<name of the range>
+ std::map<OUString, ScRangeName*> aRangeMap;
+ pDoc->GetRangeNameMap(aRangeMap);
+ for (auto const& elemRangeMap : aRangeMap)
{
- ScRangeName::const_iterator itr = pRangeName->begin(), itrEnd = pRangeName->end();
- for (; itr != itrEnd; ++itr)
- pTypeDlg->AppendNamedRange(itr->second->GetName());
+ ScRangeName* pRangeName = elemRangeMap.second;
+ if (pRangeName)
+ {
+ if (elemRangeMap.first == STR_GLOBAL_RANGE_NAME)
+ {
+ for (auto const& elem : *pRangeName)
+ pTypeDlg->AppendNamedRange(elem.second->GetName());
+ }
+ else
+ {
+ OUString aScope(elemRangeMap.first);
+ ScGlobal::AddQuotes(aScope, '\'');
+ for (auto const& elem : *pRangeName)
+ {
+ pTypeDlg->AppendNamedRange(aScope + "." + elem.second->GetName());
+ }
+ }
+ }
}
if ( pTypeDlg->Execute() == RET_OK )