Bug 357100

Summary:	The destUrl of 'Show original directory' of a link is wrong in DesktopView showing desktop folder
Product:	[Plasma] plasmashell	Reporter:	Cjacker <cjacker>
Component:	Desktop icons & Folder View widget	Assignee:	Eike Hein <hein>
Status:	RESOLVED FIXED
Severity:	normal	CC:	notuxius, plasma-bugs-null, zhaixiang
Priority:	NOR
Version First Reported In:	5.4.3
Target Milestone:	1.0
Platform:	Other
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description Cjacker 2015-12-23 14:06:28 UTC

As topic.

Reproducible: Always

Steps to Reproduce:
1.set DesktopView to 'Show the Desktop folder'.

2.then create a link in ~/Desktop, link to ~.

3.Right click the link in DesktopView and activate the 'Show Original Directory' menu item

It will open "desktop:/home" instead of "/home".

Comment 1 Cjacker 2015-12-23 14:07:09 UTC

Patch to fix it:

diff -Nur plasma-desktop-5.4.3/containments/desktop/plugins/folder/internallibkonq/konq_popupmenu.cpp plasma-desktop-5.4.3.fix/containments/desktop/plugins/folder/internallibkonq/konq_popupmenu.cpp
--- plasma-desktop-5.4.3/containments/desktop/plugins/folder/internallibkonq/konq_popupmenu.cpp 2015-12-23 21:47:37.329252500 +0800
+++ plasma-desktop-5.4.3.fix/containments/desktop/plugins/folder/internallibkonq/konq_popupmenu.cpp 2015-12-23 21:47:09.163251227 +0800
@@ -623,7 +623,7 @@
 {
     const KFileItem item = m_popupItemProperties.items().first();
     const QString dest = item.linkDest();
-    QUrl destUrl = m_sViewURL;
+    QUrl destUrl = m_sViewURL.toLocalFile();
     if (dest.startsWith('/')) {
         destUrl.setPath(dest);
     } else {

Comment 2 Leslie Zhai 2015-12-24 02:36:12 UTC

fixed in git master

Comment 3 Alexander Mentyu 2017-08-31 15:49:40 UTC

Cannot reproduce/no longer relevant? i:

Distribution: KDE neon Developer Edition Unstable Branches
KDE Plasma Version: 5.10.90
KDE Frameworks Version: 5.37.0
Qt Version: 5.9.1
Kernel Version: 4.10.0-32-generic

Comment 4 Christoph Feck 2017-09-19 20:30:09 UTC

Fixed with the FolderView rewrite for Plasma 5.10.

Comment 5 Eike Hein 2017-09-20 09:00:44 UTC

There was no Folder View rewrite in 5.10.

Comment 6 Christoph Feck 2017-10-04 18:01:59 UTC

Well, maybe 5.9 then?