Bug 357100 - The destUrl of 'Show original directory' of a link is wrong in DesktopView showing desktop folder
Summary: The destUrl of 'Show original directory' of a link is wrong in DesktopView s...
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Desktop icons & Folder View widget (other bugs)
Version First Reported In: 5.4.3
Platform: Other Linux
: NOR normal
Target Milestone: 1.0
Assignee: Eike Hein
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-12-23 14:06 UTC by Cjacker
Modified: 2017-10-04 18:01 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In:
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
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?