[Konversation-devel] [Bug 219916] Certain URL is mangled in browser after clicking it from Konversation

Peter Simonsson peter.simonsson at gmail.com
Tue Jan 5 17:55:26 CET 2010


https://bugs.kde.org/show_bug.cgi?id=219916





--- Comment #3 from Peter Simonsson <peter simonsson gmail com>  2010-01-05 17:55:26 ---
commit a5f926d10386544c94a0f91068a03b2c288c7af1
Author: Peter Simonsson <peter.simonsson at gmail.com>
Date:   Tue Jan 5 17:51:51 2010 +0100

    Don't decode URLs before sending them to browser.
    BUG:219916

diff --git a/src/viewer/ircview.cpp b/src/viewer/ircview.cpp
index 35c6d84..1195c12 100644
--- a/src/viewer/ircview.cpp
+++ b/src/viewer/ircview.cpp
@@ -1366,6 +1366,10 @@ void IRCView::mouseMoveEvent(QMouseEvent* ev)

         return;
     }
+    else
+    {
+        m_urlToCopy = anchorAt(ev->pos());
+    }

     KTextBrowser::mouseMoveEvent(ev);
 }
@@ -1411,7 +1415,7 @@ void IRCView::mouseReleaseEvent(QMouseEvent *ev)

 void IRCView::anchorClicked(const QUrl& url)
 {
-    openLink(url.toString());
+    openLink(url.toEncoded());
 }

 // FIXME do we still care about newtab? looks like konqi has lots of config
now..
@@ -1516,7 +1520,7 @@ void IRCView::highlightedSlot(const QString& _link)
                 m_bookmark->setVisible( true );
                 m_saveUrl->setVisible( true );
             m_copyUrlMenu = true;
-            m_urlToCopy = link;
+//            m_urlToCopy = link;
         }
     }
     else if (link.startsWith('#') && !link.startsWith(QLatin1String("##")))

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


More information about the Konversation-devel mailing list