[Libreoffice-commits] core.git: dbaccess/source

[Julien Nabet (via logerrit)]

 dbaccess/source/core/misc/dsntypes.cxx |    9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

New commits:
commit d34ed0008f12cfe47dd239672f1941f3c54d97fd
Author:     Julien Nabet <serval2412 at yahoo.fr>
AuthorDate: Sun May 19 21:08:52 2019 +0200
Commit:     Julien Nabet <serval2412 at yahoo.fr>
CommitDate: Sat Jun 8 16:43:46 2019 +0200

    tdf#125381: Manage URLs beginning with an "~" in cutPrefix (dbaccess/dsntypes)
    
    On Windows or with other renderings than gtk3, the urls may begin with an ~
    In this case, cutPrefix returns only empty string
    When trying to access Index management for odb files related to dBase
    indexes don't display because cutPrefix returns empty url
    in dbaccess/source/ui/dlg/detailpages.cxx line 200
    
    198         ::dbaccess::ODsnTypeCollection* pTypeCollection = pTypesItem ? pTypesItem->getCollection() : nullptr;
    199         if (pTypeCollection && pUrlItem && pUrlItem->GetValue().getLength())
    200             m_sDsn = pTypeCollection->cutPrefix(pUrlItem->GetValue());
    
    Change-Id: I47360e205ddea7b060a31da697bbe9857ad3f4c0
    Reviewed-on: https://gerrit.libreoffice.org/72568
    Tested-by: Jenkins
    Reviewed-by: Julien Nabet <serval2412 at yahoo.fr>

diff --git a/dbaccess/source/core/misc/dsntypes.cxx b/dbaccess/source/core/misc/dsntypes.cxx
index 05e64ba5a001..7332ccffa196 100644
--- a/dbaccess/source/core/misc/dsntypes.cxx
+++ b/dbaccess/source/core/misc/dsntypes.cxx
@@ -83,17 +83,20 @@ OUString ODsnTypeCollection::cutPrefix(const OUString& _sURL) const
     OUString sRet;
     OUString sOldPattern;
 
+    // on Windows or with gen rendering, the urls may begin with an ~
+    const OUString& sCleanURL = comphelper::string::stripStart(_sURL, '~');
+
     for (auto const& dsnPrefix : m_aDsnPrefixes)
     {
         WildCard aWildCard(dsnPrefix);
-        if ( sOldPattern.getLength() < dsnPrefix.getLength() && aWildCard.Matches(_sURL) )
+        if ( sOldPattern.getLength() < dsnPrefix.getLength() && aWildCard.Matches(sCleanURL) )
         {
             // This relies on the fact that all patterns are of the form
             //   foo*
             // that is, the very concept of "prefix" applies.
             OUString prefix(comphelper::string::stripEnd(dsnPrefix, '*'));
-            OSL_ENSURE(prefix.getLength() <= _sURL.getLength(), "How can A match B when A shorter than B?");
-            sRet = _sURL.copy(prefix.getLength());
+            OSL_ENSURE(prefix.getLength() <= sCleanURL.getLength(), "How can A match B when A shorter than B?");
+            sRet = sCleanURL.copy(prefix.getLength());
             sOldPattern = dsnPrefix;
         }
     }