From: Emmanuel Pescosta <emmanuelpescosta099@gmail.com>
Date: Tue, 3 Dec 2013 18:00:28 +0000 (+0100)
Subject: Fix Bug 328262 - rename bug if you cancel when folder already exists
X-Git-Url: https://cloud.milkyroute.net/gitweb/dolphin.git/commitdiff_plain/c4c999d2431285a8c49f43130773ae562dc06b8b

Fix Bug 328262 - rename bug if you cancel when folder already exists

Only connect the renamingFailed signal if there is no item with the new name in the model yet.

BUG: 328262
FIXED-IN: 4.11.5
REVIEW: 114228
---

diff --git a/src/views/dolphinview.cpp b/src/views/dolphinview.cpp
index 20bc9f522..db9d8d22f 100644
--- a/src/views/dolphinview.cpp
+++ b/src/views/dolphinview.cpp
@@ -1492,7 +1492,9 @@ void DolphinView::slotRoleEditingFinished(int index, const QByteArray& role, con
             }
 
             KonqOperations* op = KonqOperations::renameV2(this, oldUrl, newName);
-            if (op) {
+            if (op && !newNameExistsAlready) {
+                // Only connect the renamingFailed signal if there is no item with the new name
+                // in the model yet, see bug 328262.
                 connect(op, SIGNAL(renamingFailed(KUrl,KUrl)), SLOT(slotRenamingFailed(KUrl,KUrl)));
             }
         }