This should fix "border" protocols handling like apt.

And it should also be a nice semplification :)

2 files changed, 14 insertions, 12 deletions

diff --git a/src/protocolhandler.cpp b/src/protocolhandler.cpp
index 945f5b48..c5e0ef96 100644
--- a/src/protocolhandler.cpp
+++ b/src/protocolhandler.cpp
@@ -190,7 +190,7 @@ bool ProtocolHandler::postHandling(const QNetworkRequest &request, QWebFrame *fr
     _frame = frame;
 
     kDebug() << "URL PROTOCOL: " << _url;
-
+    
     // "http(s)" (fast) handling
     if (_url.protocol() == QL1S("http") || _url.protocol() == QL1S("https"))
         return false;
@@ -228,6 +228,15 @@ bool ProtocolHandler::postHandling(const QNetworkRequest &request, QWebFrame *fr
         }
     }
 
+    // we cannot handle this protocol in any way.
+    // Try KRunning it...
+    if (KProtocolInfo::isKnownProtocol(_url))
+    {
+        kDebug() << "WARNING: launching a new app...";
+        (void)new KRun(_url, rApp->mainWindow(), 0, _url.isLocalFile());
+        return true;
+    }
+
     return false;
 }
 
diff --git a/src/webpage.cpp b/src/webpage.cpp
index 128d2e34..9499d6f7 100644
--- a/src/webpage.cpp
+++ b/src/webpage.cpp
@@ -409,17 +409,6 @@ void WebPage::handleUnsupportedContent(QNetworkReply *reply)
 
     KIO::Integration::AccessManager::putReplyOnHold(reply);
 
-    // get reply url...
-    KUrl replyUrl = reply->url();
-
-    bool isLocal = replyUrl.isLocalFile();
-    if(isLocal && KProtocolInfo::isKnownProtocol(replyUrl))
-    {
-        kDebug() << "WARNING: launching a new app...";
-        (void)new KRun(replyUrl, rApp->mainWindow(), 0, replyUrl.isLocalFile());
-        return;
-    }
-
     // Get suggested file name...
     extractSuggestedFileName(reply, _suggestedFileName);
 
@@ -436,6 +425,9 @@ void WebPage::handleUnsupportedContent(QNetworkReply *reply)
 
     KService::Ptr appService = KMimeTypeTrader::self()->preferredService(_mimeType);
 
+    KUrl replyUrl = reply->url();
+    bool isLocal = replyUrl.isLocalFile();
+    
     if (appService.isNull())  // no service can handle this. We can just download it..
     {
         kDebug() << "no service can handle this. We can just download it..";
@@ -519,6 +511,7 @@ void WebPage::loadStarted()
 {
 }
 
+
 void WebPage::loadFinished(bool ok)
 {
     Q_UNUSED(ok);