Use 'FindMimeFromData' from Win API instead of QMimeDatabase() functi…

…ons to get the mimetype. Prevents freeze from VFS placeholders.

Signed-off-by: alex-z <[email protected]>

src/common/utility.h

@@ -28,6 +28,7 @@
 #include <QElapsedTimer>
 #include <QLoggingCategory>
 #include <QMap>
+#include <QMimeDatabase>
 #include <QUrl>
 #include <QUrlQuery>
 #include <functional>
@@ -183,6 +184,8 @@ namespace Utility {
      */
     OCSYNC_EXPORT QString timeAgoInWords(const QDateTime &dt, const QDateTime &from = QDateTime());
 
+    OCSYNC_EXPORT QMimeType mimeTypeForFile(const QString &filePath);
+
     class OCSYNC_EXPORT StopWatch
     {
     private:

src/common/utility_mac.mm

@@ -266,4 +266,9 @@
 
 void Utility::registerUriHandlerForLocalEditing() { /* URI handler is registered via MacOSXBundleInfo.plist.in */ }
 
+QMimeType Utility::mimeTypeForFile(const QString &filePath)
+{
+    return QMimeDatabase().mimeTypeForFile(filePath);
+}
+
 } // namespace OCC

src/common/utility_unix.cpp

@@ -150,4 +150,9 @@ void Utility::registerUriHandlerForLocalEditing()
     process.waitForFinished();
 }
 
+QMimeType Utility::mimeTypeForFile(const QString &filePath)
+{
+    return QMimeDatabase().mimeTypeForFile(filePath);
+}
+
 } // namespace OCC

src/common/utility_win.cpp

@@ -27,6 +27,7 @@
 #include <shlguid.h>
 #include <shlobj.h>
 #include <string>
+#include <urlmon.h>
 #include <winbase.h>
 #include <windows.h>
 #include <winerror.h>
@@ -44,6 +45,19 @@ static const char systemRunPathC[] = R"(HKEY_LOCAL_MACHINE\Software\Microsoft\Wi
 static const char runPathC[] = R"(HKEY_CURRENT_USER\Software\Microsoft\Windows\CurrentVersion\Run)";
 
 namespace OCC {
+
+QString mimeNameForFile(const QString &filePath)
+{
+    LPWSTR mimeTypeFound = NULL;
+    HRESULT hr = FindMimeFromData(NULL, filePath.toStdWString().c_str(), NULL, 0, NULL, FMFD_URLASFILENAME, &mimeTypeFound, 0);
+    if (SUCCEEDED(hr)) {
+        const auto result = QString::fromWCharArray(mimeTypeFound);
+        CoTaskMemFree(mimeTypeFound);
+        return result;
+    }
+
+    return {};
+}
 
 QVector<Utility::ProcessInfosForOpenFile> Utility::queryProcessInfosKeepingFileOpen(const QString &filePath)
 {
@@ -538,4 +552,9 @@ Utility::NtfsPermissionLookupRAII::~NtfsPermissionLookupRAII()
     qt_ntfs_permission_lookup--;
 }
 
+QMimeType Utility::mimeTypeForFile(const QString &filePath)
+{
+    return QMimeDatabase().mimeTypeForName(mimeNameForFile(filePath));
+}
+
 } // namespace OCC

src/csync/ConfigureChecks.cmake

@@ -31,7 +31,7 @@ if (NOT LINUX)
 endif (NOT LINUX)
 
 if(WIN32)
-  set(CMAKE_REQUIRED_LIBRARIES ${CMAKE_REQUIRED_LIBRARIES} psapi kernel32 Rstrtmgr)
+  set(CMAKE_REQUIRED_LIBRARIES ${CMAKE_REQUIRED_LIBRARIES} psapi kernel32 Rstrtmgr urlmon)
 endif()
 
 check_function_exists(utimes HAVE_UTIMES)

src/gui/caseclashfilenamedialog.cpp

@@ -88,7 +88,10 @@ CaseClashFilenameDialog::CaseClashFilenameDialog(AccountPtr account,
 
     _relativeFilePath = filePathFileInfo.path() + QStringLiteral("/");
     _relativeFilePath = _relativeFilePath.replace(folder->path(), QLatin1String());
-    _relativeFilePath = _relativeFilePath.isEmpty() ? QString() : _relativeFilePath + QStringLiteral("/");
+    _relativeFilePath = _relativeFilePath.isEmpty() ? QString() : _relativeFilePath;
+    if (!_relativeFilePath.isEmpty() && !_relativeFilePath.endsWith(QStringLiteral("/"))) {
+        _relativeFilePath += QStringLiteral("/");
+    }
 
     _originalFileName = _relativeFilePath + conflictFileName;
 
@@ -208,7 +211,7 @@ void CaseClashFilenameDialog::updateFileWidgetGroup(const QString &filePath,
     const auto fileSizeString = locale().formattedDataSize(filePathFileInfo.size());
     const auto fileUrl = QUrl::fromLocalFile(filePath).toString();
     const auto linkString = QStringLiteral("<a href='%1'>%2</a>").arg(fileUrl, linkText);
-    const auto mime = QMimeDatabase().mimeTypeForFile(_filePath);
+    const auto mime = Utility::mimeTypeForFile(_filePath);
     QIcon fileTypeIcon;
 
     qCDebug(lcCaseClashConflictFialog) << filePath << filePathFileInfo.exists() << filename << lastModifiedString << fileSizeString << fileUrl << linkString << mime;