From: Kai Uwe Broulik <kde@privat.broulik.de>
Date: Wed, 5 Sep 2018 10:25:39 +0000 (+0200)
Subject: [DolphinContextMenu] Use the view's already existing root KFileItem
X-Git-Url: https://cloud.milkyroute.net/gitweb/dolphin.git/commitdiff_plain/8f10b5540c091b47df1b7a92aa96baaf126ef2dc

[DolphinContextMenu] Use the view's already existing root KFileItem

Saves creating a new KFileItem from scratch which stats the file.
Only when the root item isn't available (as can happen when you're currently looking at an non-existing directory)
will it create a new KFileItem

Differential Revision: https://phabricator.kde.org/D15266
---

diff --git a/src/dolphincontextmenu.cpp b/src/dolphincontextmenu.cpp
index 8fdaab1d0..a521551e5 100644
--- a/src/dolphincontextmenu.cpp
+++ b/src/dolphincontextmenu.cpp
@@ -341,7 +341,12 @@ void DolphinContextMenu::openViewportContextMenu()
     addSeparator();
 
     // Insert 'Open With' entries
-    const KFileItemListProperties baseUrlProperties(KFileItemList() << baseFileItem());
+    KFileItem baseItem = view->rootItem();
+    if (baseItem.isNull() || baseItem.url() != m_baseUrl) {
+        baseItem = baseFileItem();
+    }
+
+    const KFileItemListProperties baseUrlProperties(KFileItemList() << baseItem);
     KFileItemActions fileItemActions;
     fileItemActions.setParentWidget(m_mainWindow);
     fileItemActions.setItemListProperties(baseUrlProperties);