CopyTo and MoveTo actions aware of Dolphin settings (needed for fixing bug #181910)

Alessandro Diaferia alediaferia at gmail.com
Thu Apr 16 18:03:32 BST 2009


The patch comes from the suggestion made by dfaure here:
https://bugs.kde.org/show_bug.cgi?id=181910 .
Hope it is OK to commit since it helps me fixing the folderview bug.

Cheers :)

-- 
Alessandro Diaferia
KDE Developer
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20090416/9e9e616f/attachment.htm>
-------------- next part --------------
Index: konq_popupmenu.cpp
===================================================================
--- konq_popupmenu.cpp	(revision 951970)
+++ konq_popupmenu.cpp	(working copy)
@@ -410,8 +410,13 @@ void KonqPopupMenuPrivate::init(KonqPopu
 
     q->addSeparator();
 
+    // Use the Dolphin setting for showing the "Copy To" and "Move To" actions
+    KConfig dolphin("dolphinrc");
+
     // CopyTo/MoveTo menus
-    if (m_itemFlags & KParts::BrowserExtension::ShowUrlOperations) {
+    if (m_itemFlags & KParts::BrowserExtension::ShowUrlOperations &&
+        KConfigGroup(&dolphin, "General").readEntry("ShowCopyMoveMenu", false)) {
+
         m_copyToMenu.setItems(lstItems);
         m_copyToMenu.setReadOnly(sMoving == false);
         m_copyToMenu.addActionsTo(q);


More information about the kfm-devel mailing list