Bug 158746 - URLs have commas added to them when displayed
Summary: URLs have commas added to them when displayed
Status: RESOLVED WORKSFORME
Alias: None
Product: konversation
Classification: Applications
Component: general (show other bugs)
Version: 1.0.1
Platform: Debian stable Linux
: NOR normal
Target Milestone: ---
Assignee: Konversation Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-03-04 01:11 UTC by Stuart Prescott
Modified: 2013-04-14 20:45 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 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.