Bug 130676 - URLs containing non-ASCII characters are displayed wrong in side panel
Summary: URLs containing non-ASCII characters are displayed wrong in side panel
Status: RESOLVED UNMAINTAINED
Alias: None
Product: kab3
Classification: Miscellaneous
Component: general (show other bugs)
Version: unspecified
Platform: OpenBSD Linux
: NOR normal
Target Milestone: ---
Assignee: Tobias Koenig
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-07-12 09:27 UTC by Roland Kempf
Modified: 2009-08-05 16:30 UTC (History)
0 users

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 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.