Fix some issues in findClosestExistingPath

qgis · Jun 22, 2018 · 5cd251860bdb6d2e354347904c526264664af7fb · 5cd2518
1 parent b2eedd3
commit 5cd251860bdb6d2e354347904c526264664af7fb
Unified Split

Showing with 6 additions and 2 deletions.

+6 −2 src/core/qgsfileutils.cpp
diff --git a/src/core/qgsfileutils.cpp b/src/core/qgsfileutils.cpp
@@ -110,7 +110,7 @@ QString QgsFileUtils::findClosestExistingPath( const QString &path )
   QSet< QString > visited;
   while ( !currentPath.exists() )
   {
-    const QString parentPath = QDir::cleanPath( currentPath.path() + QStringLiteral( "/.." ) );
+    const QString parentPath = QDir::cleanPath( currentPath.absolutePath() + QStringLiteral( "/.." ) );
     if ( visited.contains( parentPath ) )
       return QString(); // break circular links
 
@@ -120,6 +120,10 @@ QString QgsFileUtils::findClosestExistingPath( const QString &path )
     visited << parentPath;
   }
 
-  const QString res = QDir::cleanPath( currentPath.path() );
+  const QString res = QDir::cleanPath( currentPath.absolutePath() );
+
+  if ( res == QDir::currentPath() )
+      return QString(); // avoid default to binary folder if a filename alone is specified
+
   return res == '.' ? QString() : res;
 }