Bug 158746

Summary: URLs have commas added to them when displayed
Product: [Applications] konversation Reporter: Stuart Prescott <bugs.kde.org>
Component: generalAssignee: Konversation Developers <konversation-devel>
Status: RESOLVED WORKSFORME    
Severity: normal CC: anselmolsm, hein
Priority: NOR    
Version: 1.0.1   
Target Milestone: ---   
Platform: Debian stable   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description Stuart Prescott 2008-03-04 01:11:05 UTC
Version:           1.0.1 (using KDE 3.5.7)
Installed from:    Debian stable Packages
OS:                Linux

Some automagical parsing of the text is going on in Konversation -- I am finding that URLs are having a comma appended to them when I type them in and they are displayed to me in a query, but the recipient and the logfile for the query do not have the comma.

Example (I am "themill" and I was talking to the bot "dpkg" in #debian):

I typed: 

no, skype is <reply> skype is a proprietary cross-platform VoIP program is an evil obfuscated black-box that could be communicating absolutely any of your data with third parties. It is not in debian because we are not allowed to distribute it. Packages are available from http://www.skype.com/download/skype/linux/, or use the sources.list line "deb http://download.skype.com/linux/repos/debian/ stable non-free"

The backlog showed:

[23:48] <themill> no, skype is <reply> skype is a proprietary cross-platform VoIP program is an evil obfuscated black-box that could be communicating absolutely any of your data with third parties. It is not in debian because we are not allowed to distribute it. Packages are available from http://www.skype.com/download/skype/linux/, or use the sources.list line "deb http://download.skype.com/linux/repos/debian/, stable non-free"

Note the extra comma in the last line (just before the word "stable"), which would completely break your sources.list)

The logfile for that query does not show the comma though:

[Mon Mar 3 2008] [23:48:59] <themill> no, skype is <reply> skype is a proprietary cross-platform VoIP program is an evil obfuscated black-box that could be communicating absolutely any of your data with third parties. It is not in debian because we are not allowed to distribute it. Packages are available from http://www.skype.com/download/skype/linux/, or use the sources.list line "deb http://download.skype.com/linux/repos/debian/ stable non-free"

The actual text as received by the dpkg did not contain the comma.


So the text that I am being shown as what I wrote and what the recipient actually receives differ by a comma here.
Comment 1 Eike Hein 2008-03-04 14:09:58 UTC
This appears to be fixed in the SVN version, at least I can't reproduce it with the sample text.

Peter, Eli?
Comment 2 Peter Simonsson 2008-03-20 19:53:57 UTC
Unable to reproduce with the svn version.
Comment 3 Peter Simonsson 2008-04-18 21:27:04 UTC
The bug is actually the other way around now... adding a comma after the url that has none
Comment 4 Anselmo L. S. Melo (anselmolsm) 2009-07-15 03:08:20 UTC
I can't reproduce this bug here, using:

Konversation Version 1.2-alpha4+ #3414
Using KDE 4.3.60 (KDE 4.3.60 (KDE 4.4 >= 20090706)) - svn r992924
Qt 4.5.2 from kde-qt, hash 2d1727
Comment 5 Eike Hein 2009-07-15 03:21:26 UTC
Peter, could you post some new steps to reproduce?
Comment 6 Eike Hein 2013-04-14 20:45:17 UTC
Closing - can't reproduce, Peter never chimed in, and the URL code was rewritten anyway.