Bug 130676

Summary:	URLs containing non-ASCII characters are displayed wrong in side panel
Product:	[Unmaintained] kab3	Reporter:	Roland Kempf <kde>
Component:	general	Assignee:	Tobias Koenig <tokoe>
Status:	RESOLVED UNMAINTAINED
Severity:	normal
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	OpenBSD
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description Roland Kempf 2006-07-12 09:27:11 UTC

Version:           3.5.3 (using KDE KDE 3.5.3)
Installed from:    OpenBSD Packages

The bug is easy to reproduce:
1) Type in an non-ASCII URL in the contact data (e.g. "www.münchen.de").
2) The information is displayed as "http://www.xn-mnchen-3ya.de" in the side panel.

Further comments:
- The URL is also saved as "http://www.xn-mnchen-3ya.de" in the "std.vcf" file.
- The URL is interpreted correctly ("http://www.münchen.de") in the input mask for the contact data and when the web site is opened by clicking the URL in the side.
- Non-ASCII characters in e-mail addresses (e.g. "info@münchen.de") are saved correctly and displayed correctly in the side panel.

Here are my questions and suggestions:
Does the vcard definition allow non-ASCII characters in email and URLs?
If the answer is "yes": Please treat URLs in general in the same way as e-mails.
If the answer is "no":  Please interprete the URLs in the side panel in the same way as in the input masks.

Comment 1 Roland Kempf 2006-07-12 09:29:31 UTC

Ohh, the platform is not OpenBSD Packages but OpenSUSE Packages. sorry...

Comment 2 Tobias Koenig 2007-02-13 19:57:49 UTC

SVN commit 633301 by tokoe:

Use KURL::prettyURL() for nicer output and not confuse the user by punnyCode encoding

BUG:130676


 M  +3 -3      addresseeview.cpp  


--- branches/KDE/3.5/kdepim/libkdepim/addresseeview.cpp #633300:633301
@@ -268,11 +268,11 @@
     if ( !addr.url().url().isEmpty() ) {
       QString url;
       if ( linkMask & URLLinks ) {
-        url = (addr.url().url().startsWith( "http://" ) || addr.url().url().startsWith( "https://" ) ? addr.url().url() :
-          "http://" + addr.url().url());
+        url = (addr.url().url().startsWith( "http://" ) || addr.url().url().startsWith( "https://" ) ? addr.url().prettyURL() :
+          "http://" + addr.url().prettyURL());
         url = KStringHandler::tagURLs( url );
       } else {
-        url = addr.url().url();
+        url = addr.url().prettyURL();
       }
       dynamicPart += rowFmtStr.arg( i18n("Homepage") ).arg( url );
     }

Comment 3 Tobias Koenig 2009-08-05 16:30:28 UTC

The development of the old KAddressBook will be discontinued for KDE 4.4.
Since the new application has the same name, but a completly new code base we close all bug reports against the old version and ask the submitters to resend there reports against the new product.