From 8045ccc382ac91c14849e10f37d9a8d0605dc562 Mon Sep 17 00:00:00 2001
From: Friedemann Kleint <Friedemann.Kleint@qt.io>
Date: Wed, 3 Apr 2019 15:35:30 +0200
Subject: Windows QPA/File dialog: Pass up http[s] URLs

The native Windows allows for typing in http[s] URLs directly (without
checking existence). Pass these up.

Fixes: QTBUG-71785
Change-Id: I60237bab596ca3f52e6f513f17544ff94e9080da
Reviewed-by: Andre de la Rocha <andre.rocha@qt.io>
Reviewed-by: Oliver Wolff <oliver.wolff@qt.io>
---
 .../platforms/windows/qwindowsdialoghelpers.cpp    | 39 +++++++++++++++-------
 1 file changed, 27 insertions(+), 12 deletions(-)

(limited to 'src/plugins')

diff --git a/src/plugins/platforms/windows/qwindowsdialoghelpers.cpp b/src/plugins/platforms/windows/qwindowsdialoghelpers.cpp
index ba441a1921..9de3268fc8 100644
--- a/src/plugins/platforms/windows/qwindowsdialoghelpers.cpp
+++ b/src/plugins/platforms/windows/qwindowsdialoghelpers.cpp
@@ -1453,26 +1453,41 @@ static QString createTemporaryItemCopy(QWindowsShellItem &qItem, QString *errorM
     return result;
 }
 
+static QUrl itemToDialogUrl(QWindowsShellItem &qItem, QString *errorMessage)
+{
+    QUrl url = qItem.url();
+    if (url.isLocalFile() || url.scheme().startsWith(QLatin1String("http")))
+        return url;
+    const QString path = qItem.path();
+    if (path.isEmpty() && !qItem.isDir() && qItem.canStream()) {
+        const QString temporaryCopy = createTemporaryItemCopy(qItem, errorMessage);
+        if (temporaryCopy.isEmpty()) {
+            QDebug(errorMessage).noquote() << "Unable to create a local copy of"
+                << qItem << ": " << errorMessage;
+            return QUrl();
+        }
+        return QUrl::fromLocalFile(temporaryCopy);
+    }
+    if (!url.isValid())
+        QDebug(errorMessage).noquote() << "Invalid URL obtained from" << qItem;
+    return url;
+}
+
 QList<QUrl> QWindowsNativeOpenFileDialog::dialogResult() const
 {
     QList<QUrl> result;
     IShellItemArray *items = nullptr;
     if (SUCCEEDED(openFileDialog()->GetResults(&items)) && items) {
+        QString errorMessage;
         for (IShellItem *item : QWindowsShellItem::itemsFromItemArray(items)) {
             QWindowsShellItem qItem(item);
-            const QString path = qItem.path();
-            if (path.isEmpty() && !qItem.isDir()) {
-                QString errorMessage;
-                const QString temporaryCopy = createTemporaryItemCopy(qItem, &errorMessage);
-                if (temporaryCopy.isEmpty()) {
-                    qWarning().noquote() << "Unable to create a local copy of" << qItem
-                        << ": " << errorMessage;
-                } else {
-                    result.append(QUrl::fromLocalFile(temporaryCopy));
-                }
-            } else {
-                result.append(qItem.url());
+            const QUrl url = itemToDialogUrl(qItem, &errorMessage);
+            if (!url.isValid()) {
+                qWarning("%s", qPrintable(errorMessage));
+                result.clear();
+                break;
             }
+            result.append(url);
         }
     }
     return result;
-- 
cgit v1.2.3