summaryrefslogtreecommitdiff
path: root/sc
diff options
context:
space:
mode:
Diffstat (limited to 'sc')
-rw-r--r--sc/source/core/data/dpshttab.cxx28
-rw-r--r--sc/source/ui/view/cellsh1.cxx30
2 files changed, 6 insertions, 52 deletions
diff --git a/sc/source/core/data/dpshttab.cxx b/sc/source/core/data/dpshttab.cxx
index 0b0c49f9024a..17b7731d2f0d 100644
--- a/sc/source/core/data/dpshttab.cxx
+++ b/sc/source/core/data/dpshttab.cxx
@@ -232,39 +232,13 @@ const ScRange& ScSheetSourceDesc::GetSourceRange() const
{
// Obtain the source range from the range name first.
maSourceRange = ScRange();
-
- // Range names referring a sheet contain a .
- // See comment of ScCellShell::ExecuteDataPilotDialog
- // paragraph "Populate named ranges"
- sal_Int32 nAfterSheetName = ScGlobal::FindUnquoted( maRangeName, '.');
-
- // let's consider the range name is global to the doc by default
ScRangeName* pRangeName = mpDoc->GetRangeName();
- OUString searchRangeName(maRangeName);
-
- // the range name concerns a specificsheet
- if (nAfterSheetName != -1)
- {
- OUString sheetName = maRangeName.copy(0, nAfterSheetName);
- ScGlobal::EraseQuotes( sheetName, '\'', false);
- searchRangeName = maRangeName.copy(nAfterSheetName+1);
-
- SCTAB nTab = 0;
- if (!mpDoc->GetTable(sheetName, nTab))
- {
- // the sheetname should exist
- assert(false);
- return maSourceRange;
- }
- pRangeName = mpDoc->GetRangeName(nTab);
- }
-
do
{
if (!pRangeName)
break;
- OUString aUpper = ScGlobal::getCharClassPtr()->uppercase(searchRangeName);
+ OUString aUpper = ScGlobal::getCharClassPtr()->uppercase(maRangeName);
const ScRangeData* pData = pRangeName->findByUpperName(aUpper);
if (!pData)
break;
diff --git a/sc/source/ui/view/cellsh1.cxx b/sc/source/ui/view/cellsh1.cxx
index 12335f870ca6..4db57dc43f26 100644
--- a/sc/source/ui/view/cellsh1.cxx
+++ b/sc/source/ui/view/cellsh1.cxx
@@ -17,7 +17,6 @@
* 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>
@@ -2874,31 +2873,12 @@ void ScCellShell::ExecuteDataPilotDialog()
pTabViewShell->GetFrameWeld(), bEnableExt));
// Populate named ranges (if any).
- // 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* pRangeName = pDoc->GetRangeName();
+ if (pRangeName)
{
- 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());
- }
- }
- }
+ ScRangeName::const_iterator itr = pRangeName->begin(), itrEnd = pRangeName->end();
+ for (; itr != itrEnd; ++itr)
+ pTypeDlg->AppendNamedRange(itr->second->GetName());
}
if ( pTypeDlg->Execute() == RET_OK )