Bug 354678

Summary:	location bar shows wrong name for folders containing "%" symbol
Product:	[Frameworks and Libraries] frameworks-kio	Reporter:	kyklops <kyklops>
Component:	general	Assignee:	David Faure <faure>
Status:	RESOLVED FIXED
Severity:	normal	CC:	emmanuelpescosta099, kdelibs-bugs, kyklops
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Kubuntu
OS:	Linux
Latest Commit:	http://commits.kde.org/kio/9dbe36f734b5b839b2a6a934fad29d639e954498	Version Fixed In:	5.17
Sentry Crash Report:

Description kyklops 2015-11-01 10:25:35 UTC

Forlders containing "%" symbol in their names get shown in the location bar with "%25" instead when opened.

Reproducible: Always

Steps to Reproduce:
1. create a new folder named asd%
2. open it 
3. asd%/ is shown as "asd%25" in the location bar
4. clicking on "asd%25" appends a further "25" to the name shown, an error message (correctly) says that folder asd%25 does not exists.

Actual Results:  
asd%/ is shown as "asd%25" in the location bar.
Clicking on "asd%25" appends a further "25" to the name shown, an error message (correctly) says that folder asd%25 does not exists.

Comment 1 Emmanuel Pescosta 2015-11-08 19:24:03 UTC

Thanks for the bug report!
I can reproduce the problem.

Looks like a KUrl -> QUrl porting bug (url encoding failure). The relevant code is in KIO, so I'll reassign this bug report to KIO.

Comment 2 Emmanuel Pescosta 2015-11-09 19:10:22 UTC

Review-Request: https://git.reviewboard.kde.org/r/126009/

Comment 3 Emmanuel Pescosta 2015-11-09 22:15:26 UTC

Git commit 9dbe36f734b5b839b2a6a934fad29d639e954498 by Emmanuel Pescosta.
Committed on 09/11/2015 at 22:14.
Pushed by emmanuelp into branch 'master'.

Fix url encoding problems (e.g. when the path contains a '%') in KUrlNavigator

These changes also fix a weird behavior of KUrlNavigator when button 0 is
pressed. KUrlNavigator always used the local path '/' for button 0 and ignored
the scheme and authority.
REVIEW: 126009
FIXED-IN: 5.17

M  +37   -0    autotests/kurlnavigatortest.cpp
M  +3    -0    autotests/kurlnavigatortest.h
M  +4    -3    src/filewidgets/kurlnavigator.cpp

http://commits.kde.org/kio/9dbe36f734b5b839b2a6a934fad29d639e954498