Bug 204052

Summary: Encoding problem with copy/paste an url from different a application
Product: [Applications] konversation Reporter: Mehmet Nur Olcay <mnurolcay>
Component: generalAssignee: Konversation Developers <konversation-devel>
Status: RESOLVED FIXED    
Severity: normal CC: hein, mnurolcay, peter.simonsson
Priority: NOR    
Version: 1.2-alpha6   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In:

Description Mehmet Nur Olcay 2009-08-16 15:03:45 UTC 
Version:           1.2-alpha6 (using KDE 4.2.4)
Compiler:          Gcc 4.3.3 
OS:                Linux
Installed from:    Compiled From Sources

The problem occurs with non-english characters. Here with Turkish ones.

Here are the steps to reproduce:

1. Run firefox, open this page in firefox: http://tinyurl.com/m4qh9l

2. Copy the exact link from firefox address bar, paste it to any channel in konversation

3. See how encoding fails, the link becomes totally useless
Comment 1 Peter Simonsson 2009-08-16 19:07:32 UTC 
It's not a problem with encoding... the problem is that firefox url encodes the url which results in the following string:
http://tr.pardus-wiki.org/NASIL:Unutulan_kullan%C4%B1c%C4%B1_parolalar%C4%B1n%C4%B1_yeniden_olu%C5%9Fturmak

Which contains %Cnumber and %B which konversation by default replaces with text color and bold. So should this be on by default? That's the question...
Comment 2 Mehmet Nur Olcay 2009-08-16 19:20:55 UTC 
(In reply to comment #1)
> It's not a problem with encoding... the problem is that firefox url encodes the
> url which results in the following string:
> http://tr.pardus-wiki.org/NASIL:Unutulan_kullan%C4%B1c%C4%B1_parolalar%C4%B1n%C4%B1_yeniden_olu%C5%9Fturmak
> 
> Which contains %Cnumber and %B which konversation by default replaces with text
> color and bold. So should this be on by default? That's the question...

Arora does the same as Firefox while Chromium and Opera doesn't.

What if adding some dedection code what turns off color or offer to paste as simple text if there's an url ?
Comment 3 Eike Hein 2009-10-24 20:00:35 UTC 
Fixed as of SVN revision 1039827. Variable expansion (%B, %C, %I, etc.) is no longer done in text segments recognized as URLs to avoid clashes with percent-encoded characters in URLs copied from web browsers, such as German umlauts.