From: Andreas Krutzler <andi.krutzler@gmail.com>
Date: Sat, 11 Nov 2017 02:32:53 +0000 (-0700)
Subject: Prevent "Two clicks renaming" if the selected file/folder is not movable
X-Git-Url: https://cloud.milkyroute.net/gitweb/dolphin.git/commitdiff_plain/fa04e37569a316b3aaab3ca9fc42d50501347fd8

Prevent "Two clicks renaming" if the selected file/folder is not movable

Summary:
Two clicks renaming doesn't check if the user is actually allowed to rename a file/folder. With this patch, this get fixed.
Depends on D7647

Reviewers: rkflx, #dolphin, ngraham

Reviewed By: #dolphin, ngraham

Subscribers: elvisangelaccio, ngraham, #dolphin

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

diff --git a/src/views/dolphinview.cpp b/src/views/dolphinview.cpp
index 4485c1707..cd802eec1 100644
--- a/src/views/dolphinview.cpp
+++ b/src/views/dolphinview.cpp
@@ -1125,8 +1125,12 @@ void DolphinView::slotMouseButtonPressed(int itemIndex, Qt::MouseButtons buttons
 void DolphinView::slotSelectedItemTextPressed(int index)
 {
     if (GeneralSettings::renameInline()) {
-        m_twoClicksRenamingItemUrl = m_model->fileItem(index).url();
-        m_twoClicksRenamingTimer->start(QApplication::doubleClickInterval());
+        const KFileItem item = m_model->fileItem(index);
+        const KFileItemListProperties capabilities(KFileItemList() << item);
+        if (capabilities.supportsMoving()) {
+            m_twoClicksRenamingItemUrl = item.url();
+            m_twoClicksRenamingTimer->start(QApplication::doubleClickInterval());
+        }
     }
 }