[PATCH libreoffice-4-0] range names need to be checked upper case, fdo#64031

Markus Mohrhard (via Code Review) gerrit at gerrit.libreoffice.org
Mon May 27 04:53:21 PDT 2013


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/4050

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/50/4050/1

range names need to be checked upper case, fdo#64031

Change-Id: I540821fd7058bf0da27d955c9d966a10dd21ec94
---
M sc/source/ui/docshell/arealink.cxx
1 file changed, 3 insertions(+), 2 deletions(-)



diff --git a/sc/source/ui/docshell/arealink.cxx b/sc/source/ui/docshell/arealink.cxx
index 84d30be..ee322d5 100644
--- a/sc/source/ui/docshell/arealink.cxx
+++ b/sc/source/ui/docshell/arealink.cxx
@@ -193,10 +193,11 @@
 bool ScAreaLink::FindExtRange( ScRange& rRange, ScDocument* pSrcDoc, const String& rAreaName )
 {
     bool bFound = false;
+    rtl::OUString aUpperName = ScGlobal::pCharClass->uppercase(rAreaName);
     ScRangeName* pNames = pSrcDoc->GetRangeName();
     if (pNames)         // benannte Bereiche
     {
-        const ScRangeData* p = pNames->findByUpperName(rAreaName);
+        const ScRangeData* p = pNames->findByUpperName(aUpperName);
         if (p && p->IsValidReference(rRange))
             bFound = true;
     }
@@ -205,7 +206,7 @@
         ScDBCollection* pDBColl = pSrcDoc->GetDBCollection();
         if (pDBColl)
         {
-            const ScDBData* pDB = pDBColl->getNamedDBs().findByUpperName(ScGlobal::pCharClass->uppercase(rAreaName));
+            const ScDBData* pDB = pDBColl->getNamedDBs().findByUpperName(aUpperName);
             if (pDB)
             {
                 SCTAB nTab;

-- 
To view, visit https://gerrit.libreoffice.org/4050
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I540821fd7058bf0da27d955c9d966a10dd21ec94
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: libreoffice-4-0
Gerrit-Owner: Markus Mohrhard <markus.mohrhard at googlemail.com>



More information about the LibreOffice mailing list