From 98363db9134fc42e67f6aa2434dc5a03516ad1a6 Mon Sep 17 00:00:00 2001
From: Peter Penz <peter.penz19@gmail.com>
Date: Mon, 11 May 2009 18:07:05 +0000
Subject: [PATCH] select items after they have been copied/moved by a drag &
 drop operation

svn path=/trunk/KDE/kdebase/apps/; revision=966685
---
 src/dolphinview.cpp | 37 ++++++++++++++++++++-----------------
 src/dolphinview.h   | 19 +++++++++++++++++--
 2 files changed, 37 insertions(+), 19 deletions(-)

diff --git a/src/dolphinview.cpp b/src/dolphinview.cpp
index e947508f0..e59f9b282 100644
--- a/src/dolphinview.cpp
+++ b/src/dolphinview.cpp
@@ -67,6 +67,8 @@
 #include "viewproperties.h"
 #include "zoomlevelinfo.h"
 
+#include <kdebug.h>
+
 /**
  * Helper function for sorting items with qSort() in
  * DolphinView::renameSelectedItems().
@@ -90,7 +92,6 @@ DolphinView::DolphinView(QWidget* parent,
     m_isContextMenuOpen(false),
     m_ignoreViewProperties(false),
     m_assureVisibleCurrentIndex(false),
-    m_selectClipboardItems(false),
     m_mode(DolphinView::IconsView),
     m_topLayout(0),
     m_controller(0),
@@ -108,6 +109,7 @@ DolphinView::DolphinView(QWidget* parent,
     m_currentItemUrl(),
     m_createdItemUrl(),
     m_selectedItems(),
+    m_newFileNames(),
     m_expandedDragSource(0)
 {
     m_topLayout = new QVBoxLayout(this);
@@ -607,8 +609,9 @@ QString DolphinView::statusBarText() const
 
 void DolphinView::setUrl(const KUrl& url)
 {
+    m_newFileNames.clear();
+
     // remember current item candidate (see slotDirListerCompleted())
-    m_selectClipboardItems = false;
     m_currentItemUrl = url;
     updateView(url, KUrl());
 }
@@ -981,6 +984,7 @@ void DolphinView::dropUrls(const KFileItem& destItem,
                            const KUrl& destPath,
                            QDropEvent* event)
 {
+    addNewFileNames(event->mimeData());
     DragAndDropHelper::instance().dropUrls(destItem, destPath, event, this);
 }
 
@@ -1198,29 +1202,21 @@ void DolphinView::slotDirListerCompleted()
         m_currentItemUrl.clear();
     }
 
-    if (m_selectClipboardItems) {
-        m_selectClipboardItems = false;
-
-        // select all items that have been pasted from the clipboard to
-        // the current directory
-        const QMimeData* mimeData = QApplication::clipboard()->mimeData();
-        const KUrl::List copiedUrls = KUrl::List::fromMimeData(mimeData);
-
-        QSet<QString> fileNames;
-        foreach (const KUrl& url, copiedUrls) {
-            fileNames.insert(url.fileName());
-        }
-
+    if (!m_newFileNames.isEmpty()) {
+        // select all newly added items created by a paste operation or
+        // a drag & drop operation
         QItemSelectionModel* selectionModel = itemView()->selectionModel();
         const int rowCount = m_proxyModel->rowCount();
         for (int row = 0; row < rowCount; ++row) {
             const QModelIndex proxyIndex = m_proxyModel->index(row, 0);
             const QModelIndex dirIndex = m_proxyModel->mapToSource(proxyIndex);
             const KUrl url = m_dolphinModel->itemForIndex(dirIndex).url();
-            if (fileNames.contains(url.fileName())) {
+            if (m_newFileNames.contains(url.fileName())) {
                 selectionModel->select(proxyIndex, QItemSelectionModel::Select);
             }
         }
+
+        m_newFileNames.clear();
     }
 }
 
@@ -1500,7 +1496,7 @@ QAbstractItemView* DolphinView::itemView() const
 
 void DolphinView::pasteToUrl(const KUrl& url)
 {
-    m_selectClipboardItems = true;
+    addNewFileNames(QApplication::clipboard()->mimeData());
     KonqOperations::doPaste(this, url);
 }
 
@@ -1534,5 +1530,12 @@ QMimeData* DolphinView::selectionMimeData() const
     return m_dolphinModel->mimeData(selection.indexes());
 }
 
+void DolphinView::addNewFileNames(const QMimeData* mimeData)
+{
+    const KUrl::List urls = KUrl::List::fromMimeData(mimeData);
+    foreach (const KUrl& url, urls) {
+        m_newFileNames.insert(url.fileName());
+    }
+}
 
 #include "dolphinview.moc"
diff --git a/src/dolphinview.h b/src/dolphinview.h
index 122e0e1f5..24831e03b 100644
--- a/src/dolphinview.h
+++ b/src/dolphinview.h
@@ -36,6 +36,7 @@
 #include <QKeyEvent>
 #include <QLinkedList>
 #include <QListView>
+#include <QSet>
 #include <QWidget>
 
 typedef KIO::FileUndoManager::CommandType CommandType;
@@ -747,6 +748,14 @@ private:
      */
     QMimeData* selectionMimeData() const;
 
+    /**
+     * Is invoked after a paste operation or a drag & drop
+     * operation and adds the filenames of all URLs from \a mimeData to
+     * m_newFileNames. This allows to select all newly added
+     * items in slotDirListerCompleted().
+     */
+    void addNewFileNames(const QMimeData* mimeData);
+
 private:
     bool m_active : 1;
     bool m_showPreview : 1;
@@ -756,7 +765,6 @@ private:
     bool m_isContextMenuOpen : 1;   // TODO: workaround for Qt-issue 207192
     bool m_ignoreViewProperties : 1;
     bool m_assureVisibleCurrentIndex : 1;
-    bool m_selectClipboardItems : 1;
 
     Mode m_mode;
 
@@ -780,7 +788,14 @@ private:
     KUrl m_rootUrl;
     KUrl m_currentItemUrl;
     KUrl m_createdItemUrl; // URL for a new item that got created by the "Create New..." menu
-    KFileItemList m_selectedItems; //this is used for making the View to remember selections after F5
+    KFileItemList m_selectedItems; // this is used for making the View to remember selections after F5
+
+    /**
+     * Remembers the filenames that have been added by a paste operation
+     * or a drag & drop operation. Allows to select the items in
+     * slotDirListerCompleted().
+     */
+    QSet<QString> m_newFileNames;
 
     QAbstractItemView* m_expandedDragSource;
 };
-- 
2.47.3