Bug 60522 - mis-escape: <http://bla/> becomes <http://bla/&gt;
Summary: mis-escape: <http://bla/> becomes <http://bla/&gt;
Status: RESOLVED WORKSFORME
Alias: None
Product: kdelibs
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: unspecified
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: Andrew Stanley-Jones
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-06-30 13:01 UTC by kris
Modified: 2024-05-01 20:39 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description kris 2003-06-30 13:01:13 UTC
Version:           1.2.4 (using KDE 3.1.1)
Installed from:    SuSE
Compiler:          gcc version 3.3 20030226 (prerelease) (SuSE Linux)
OS:          Linux (i686) release 2.4.20-4GB

ksirc automatically highlights urls in channels. If that url is bracketed in <> signs (less-than, greater-than), the trailing > sign (greater-than) is being escaped into an entity &gt; (ampersand gt semicolon), and the amp gt part of that text becomes part of the clickable urls.

This only happens to brackets around urls, not other backets.
Comment 1 kris 2003-06-30 13:33:40 UTC
kris@valiant:~/Source/kde/kdelibs/kdecore> diff -u kstringhandler.cpp kstringhan 
dler.cpp.patched 
--- kstringhandler.cpp  2003-06-30 13:28:18.000000000 +0200 
+++ kstringhandler.cpp.patched  2003-06-30 13:27:56.000000000 +0200 
@@ -613,7 +613,7 @@ 
  QString 
 KStringHandler::tagURLs( const QString& text ) 
 { 
-    /*static*/ QRegExp 
urlEx("(www\\.(?!\\.)|(f|ht)tp(|s)://)[\\d\\w\\./,:_~\\?=&;#@\\-\\+\\%]+[\\d\\w/]"); 
+    /*static*/ QRegExp 
urlEx("(?:^|\\s)(www\\.(?!\\.)|(f|ht)tp(|s)://)[\\d\\w\\./,:_~\\?=&;#@\\-\\+\\%]+[\\d\\w/]"); 
 
     QString richText( text ); 
     int urlPos = 0, urlLen; 
 
Please test 
Comment 2 Andrew Stanley-Jones 2003-07-20 05:08:02 UTC
That patch doesn't actually solve the problem.  It just makes <url> not 
higlight the url, but the problem still occurs for < url>, or any string that 
ends in ; 
Comment 3 Andrew Stanley-Jones 2003-07-24 05:56:27 UTC
This is a KStringHander problem, and tagURLs needs a port of the kdepim link 
locator class. 
Comment 4 kris 2004-08-22 20:16:28 UTC
[20:14:31] <Isotopp> <http://blog.koehntopp.de&gt; -- ein test, bitte ignorieren

The problem is still present in kde 3.3 release, using ksirc 1.3.11 (Suse Linux 9.1 precompiled binaries).
Comment 5 lexual 2010-01-23 06:25:37 UTC
Is this still relevant to recent KDE version?
If not please close the bug.

Thanks,

Lex.
Comment 6 Christoph Cullmann 2024-05-01 20:39:54 UTC
Hi,

the KDE libaries and applications version 4.x are obsolete and got replaced
with Frameworks 5 and now 6 based variants that are still maintained.

The issue in this bug doesn't seem to happen for me with these versions.