Bug 157447 - KUrl::prettyUrl should strip the 'file://' part (jmho)
Summary: KUrl::prettyUrl should strip the 'file://' part (jmho)
Status: RESOLVED WORKSFORME
Alias: None
Product: kdelibs
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR wishlist
Target Milestone: ---
Assignee: kdelibs bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-02-08 19:07 UTC by Marcel Partap
Modified: 2008-02-11 14:26 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Marcel Partap 2008-02-08 19:07:14 UTC 
Version:           unknown (using 4.00.61 (KDE 4.0.61 >= 20080207), Gentoo)
Compiler:          x86_64-pc-linux-gnu-gcc
OS:                Linux (x86_64) release 2.6.24-git4

Hi,
this is highly debatable but while the file:// scheme might be correct to use internally, imho those file:///foo/bar urls look confusing and unpleasant with those triple slashes. As the URL parses just fine without the file:// which is obviously the prerequisite for an URL returned by prettyUrl it should not break anything. There is a piece of leftover code in kurl.cpp which seems to have done just that. Why was it left there if not with the intent to make prettyUrl strip file:// part again? David, clues, suggestions? Opinions?
Comment 1 David Faure 2008-02-11 14:26:08 UTC 
The behavior that you want is given by pathOrUrl().