Make use of the error messages coming from the places model.

[dolphin.git] / src / dolphincontextmenu.cpp

diff --git a/src/dolphincontextmenu.cpp b/src/dolphincontextmenu.cpp
index a18f88a4180ae0f65d4f3217d414bcab073f31e0..038902e6b5b5ac8cc98ad531c4a1e5ba66785300 100644 (file)
--- a/src/dolphincontextmenu.cpp
+++ b/src/dolphincontextmenu.cpp
@@ -23,11 +23,10 @@
 #include "dolphinmainwindow.h"
 #include "dolphinsettings.h"
 #include "dolphinview.h"
 #include "dolphinmainwindow.h"
 #include "dolphinsettings.h"
 #include "dolphinview.h"

-#include "editbookmarkdialog.h"
+#include "dolphinviewcontainer.h"

 
 #include <kactioncollection.h>
 
 #include <kactioncollection.h>

-#include <kbookmarkmanager.h>
-#include <kbookmark.h>
+#include <kfileplacesmodel.h>

 #include <kdesktopfile.h>
 #include <kglobal.h>
 #include <kiconloader.h>
 #include <kdesktopfile.h>
 #include <kglobal.h>
 #include <kiconloader.h>


@@ -36,6 +35,7 @@
 #include <kmessagebox.h>
 #include <kmimetypetrader.h>
 #include <knewmenu.h>
 #include <kmessagebox.h>
 #include <kmimetypetrader.h>
 #include <knewmenu.h>

+#include <konqmimedata.h>

 #include <konq_operations.h>
 #include <klocale.h>
 #include <kpropertiesdialog.h>
 #include <konq_operations.h>
 #include <klocale.h>
 #include <kpropertiesdialog.h>


@@ -43,34 +43,28 @@
 #include <kstandardaction.h>
 #include <kstandarddirs.h>
 
 #include <kstandardaction.h>
 #include <kstandarddirs.h>
 

-#include <QDir>
+#include <QtGui/QApplication>
+#include <QtGui/QClipboard>
+#include <QtCore/QDir>
+#include <Qt3Support/Q3ValueList>

 
 DolphinContextMenu::DolphinContextMenu(DolphinMainWindow* parent,
                                        KFileItem* fileInfo,
 
 DolphinContextMenu::DolphinContextMenu(DolphinMainWindow* parent,
                                        KFileItem* fileInfo,

-                                       const KUrl& baseUrl,
-                                       ViewType viewType) :
+                                       const KUrl& baseUrl) :

     m_mainWindow(parent),
     m_fileInfo(fileInfo),
     m_baseUrl(baseUrl),
     m_mainWindow(parent),
     m_fileInfo(fileInfo),
     m_baseUrl(baseUrl),

-    m_viewType(viewType),

     m_context(NoContext)
 {
     m_context(NoContext)
 {

-    if (viewType == ItemsView) {
-        // The context menu either accesses the URLs of the selected items
-        // or the items itself. To increase the performance both lists are cached.
-        DolphinView* view = m_mainWindow->activeView();
-        m_selectedUrls = view->selectedUrls();
-        m_selectedItems = view->selectedItems();
-    }
-    else if (fileInfo != 0) {
-        m_selectedUrls.append(fileInfo->url());
-        m_selectedItems.append(fileInfo);
-    }
+    // The context menu either accesses the URLs of the selected items
+    // or the items itself. To increase the performance both lists are cached.
+    DolphinView* view = m_mainWindow->activeViewContainer()->view();
+    m_selectedUrls = view->selectedUrls();
+    m_selectedItems = view->selectedItems();

 }
 
 DolphinContextMenu::~DolphinContextMenu()
 }
 
 DolphinContextMenu::~DolphinContextMenu()

-{
-}
+{}

 
 void DolphinContextMenu::open()
 {
 
 void DolphinContextMenu::open()
 {


@@ -82,60 +76,23 @@ void DolphinContextMenu::open()
     if (m_fileInfo != 0) {
         m_context |= ItemContext;
         // TODO: handle other use cases like devices + desktop files
     if (m_fileInfo != 0) {
         m_context |= ItemContext;
         // TODO: handle other use cases like devices + desktop files

-   }
+    }

 
     // open the corresponding popup for the context
     if (m_context & TrashContext) {
         if (m_context & ItemContext) {
             openTrashItemContextMenu();
 
     // open the corresponding popup for the context
     if (m_context & TrashContext) {
         if (m_context & ItemContext) {
             openTrashItemContextMenu();

-        }
-        else {
+        } else {

             openTrashContextMenu();
         }
             openTrashContextMenu();
         }

-    }
-    else if (m_context & ItemContext) {
+    } else if (m_context & ItemContext) {

         openItemContextMenu();
         openItemContextMenu();

-    }
-    else {
+    } else {

         Q_ASSERT(m_context == NoContext);
         openViewportContextMenu();
     }
 }
 
         Q_ASSERT(m_context == NoContext);
         openViewportContextMenu();
     }
 }
 

-void DolphinContextMenu::cut()
-{
-    // TODO
-}
-
-void DolphinContextMenu::copy()
-{
-    // TODO
-}
-
-void DolphinContextMenu::paste()
-{
-    // TODO
-}
-
-void DolphinContextMenu::rename()
-{
-    // TODO
-}
-
-void DolphinContextMenu::moveToTrash()
-{
-    // TODO
-}
-
-void DolphinContextMenu::deleteItem()
-{
-    // TODO
-}
-
-void DolphinContextMenu::showProperties()
-{
-    new KPropertiesDialog(m_fileInfo->url());
-}

 
 void DolphinContextMenu::openTrashContextMenu()
 {
 
 void DolphinContextMenu::openTrashContextMenu()
 {


@@ -143,7 +100,7 @@ void DolphinContextMenu::openTrashContextMenu()
 
     KMenu* popup = new KMenu(m_mainWindow);
 
 
     KMenu* popup = new KMenu(m_mainWindow);
 

-    QAction* emptyTrashAction = new QAction(KIcon("user-trash"), i18n("Emtpy Trash"), popup);
+    QAction* emptyTrashAction = new QAction(KIcon("emptytrash"), i18n("Empty Trash"), popup);

     KConfig trashConfig("trashrc", KConfig::OnlyLocal);
     emptyTrashAction->setEnabled(!trashConfig.group("Status").readEntry("Empty", true));
     popup->addAction(emptyTrashAction);
     KConfig trashConfig("trashrc", KConfig::OnlyLocal);
     emptyTrashAction->setEnabled(!trashConfig.group("Status").readEntry("Empty", true));
     popup->addAction(emptyTrashAction);


@@ -154,10 +111,10 @@ void DolphinContextMenu::openTrashContextMenu()
     if (popup->exec(QCursor::pos()) == emptyTrashAction) {
         const QString text(i18n("Do you really want to empty the Trash? All items will get deleted."));
         const bool del = KMessageBox::warningContinueCancel(m_mainWindow,
     if (popup->exec(QCursor::pos()) == emptyTrashAction) {
         const QString text(i18n("Do you really want to empty the Trash? All items will get deleted."));
         const bool del = KMessageBox::warningContinueCancel(m_mainWindow,

-                                                            text,
-                                                            QString(),
-                                                            KGuiItem(i18n("Empty Trash"), KIcon("user-trash"))
-                                                            ) == KMessageBox::Continue;
+                         text,
+                         QString(),
+                         KGuiItem(i18n("Empty Trash"), KIcon("user-trash"))
+                                                           ) == KMessageBox::Continue;

         if (del) {
             KonqOperations::emptyTrash(m_mainWindow);
         }
         if (del) {
             KonqOperations::emptyTrash(m_mainWindow);
         }


@@ -198,10 +155,10 @@ void DolphinContextMenu::openItemContextMenu()
 
     popup->addSeparator();
 
 
     popup->addSeparator();
 

-    // insert 'Bookmark this folder' entry if exactly one item is selected
+    // insert 'Bookmark This Folder' entry if exactly one item is selected

     QAction* bookmarkAction = 0;
     if (m_fileInfo->isDir() && (m_selectedUrls.count() == 1)) {
     QAction* bookmarkAction = 0;
     if (m_fileInfo->isDir() && (m_selectedUrls.count() == 1)) {

-        bookmarkAction = popup->addAction(KIcon("bookmark-folder"), i18n("Bookmark folder"));
+        bookmarkAction = popup->addAction(KIcon("bookmark-folder"), i18n("Bookmark Folder..."));

     }
 
     // Insert 'Open With...' sub menu
     }
 
     // Insert 'Open With...' sub menu


@@ -209,48 +166,32 @@ void DolphinContextMenu::openItemContextMenu()
     const QList<QAction*> openWithActions = insertOpenWithItems(popup, openWithVector);
 
     // Insert 'Actions' sub menu
     const QList<QAction*> openWithActions = insertOpenWithItems(popup, openWithVector);
 
     // Insert 'Actions' sub menu

-    QVector<KDEDesktopMimeType::Service> actionsVector;
+    QVector<KDesktopFileActions::Service> actionsVector;

     const QList<QAction*> serviceActions = insertActionItems(popup, actionsVector);
     popup->addSeparator();
 
     // insert 'Properties...' entry
     const QList<QAction*> serviceActions = insertActionItems(popup, actionsVector);
     popup->addSeparator();
 
     // insert 'Properties...' entry

-    QAction* propertiesAction = 0;
-    if (m_viewType == SidebarView) {
-        propertiesAction = new QAction(i18n("Properties..."), this);
-        connect(this, SIGNAL(triggered()), this, SLOT(showProperties()));
-    }
-    else {
-        propertiesAction = m_mainWindow->actionCollection()->action("properties");
-    }
+    QAction* propertiesAction = m_mainWindow->actionCollection()->action("properties");

     popup->addAction(propertiesAction);
 
     QAction* activatedAction = popup->exec(QCursor::pos());
 
     popup->addAction(propertiesAction);
 
     QAction* activatedAction = popup->exec(QCursor::pos());
 

-    if ((bookmarkAction!= 0) && (activatedAction == bookmarkAction)) {
+    if ((bookmarkAction != 0) && (activatedAction == bookmarkAction)) {

         const KUrl selectedUrl(m_fileInfo->url());
         const KUrl selectedUrl(m_fileInfo->url());

-        KBookmark bookmark = EditBookmarkDialog::getBookmark(i18n("Add folder as bookmark"),
-                                                             selectedUrl.fileName(),
-                                                             selectedUrl,
-                                                             "bookmark");
-        if (!bookmark.isNull()) {
-            KBookmarkManager* manager = DolphinSettings::instance().bookmarkManager();
-            KBookmarkGroup root = manager->root();
-            root.addBookmark(manager, bookmark);
-            manager->emitChanged(root);
+        if (selectedUrl.isValid()) {
+            DolphinSettings::instance().placesModel()->addPlace(selectedUrl.fileName(),
+                    selectedUrl);

         }
         }

-    }
-    else if (serviceActions.contains(activatedAction)) {
+    } else if (serviceActions.contains(activatedAction)) {

         // one of the 'Actions' items has been selected
         int id = serviceActions.indexOf(activatedAction);
         // one of the 'Actions' items has been selected
         int id = serviceActions.indexOf(activatedAction);

-        KDEDesktopMimeType::executeService(m_selectedUrls, actionsVector[id]);
-    }
-    else if (openWithActions.contains(activatedAction)) {
+        KDesktopFileActions::executeService(m_selectedUrls, actionsVector[id]);
+    } else if (openWithActions.contains(activatedAction)) {

         // one of the 'Open With' items has been selected
         if (openWithActions.last() == activatedAction) {
             // the item 'Other...' has been selected
             KRun::displayOpenWithDialog(m_selectedUrls, m_mainWindow);
         // one of the 'Open With' items has been selected
         if (openWithActions.last() == activatedAction) {
             // the item 'Other...' has been selected
             KRun::displayOpenWithDialog(m_selectedUrls, m_mainWindow);

-        }
-        else {
+        } else {

             int id = openWithActions.indexOf(activatedAction);
             KService::Ptr servicePtr = openWithVector[id];
             KRun::run(*servicePtr, m_selectedUrls, m_mainWindow);
             int id = openWithActions.indexOf(activatedAction);
             KService::Ptr servicePtr = openWithVector[id];
             KRun::run(*servicePtr, m_selectedUrls, m_mainWindow);


@@ -286,33 +227,38 @@ void DolphinContextMenu::openViewportContextMenu()
     QAction* detailsMode = m_mainWindow->actionCollection()->action("details");
     viewModeMenu->addAction(detailsMode);
 
     QAction* detailsMode = m_mainWindow->actionCollection()->action("details");
     viewModeMenu->addAction(detailsMode);
 

+    QAction* columnsMode = m_mainWindow->actionCollection()->action("columns");
+    viewModeMenu->addAction(columnsMode);
+

     QAction* previewsMode = m_mainWindow->actionCollection()->action("previews");
     viewModeMenu->addAction(previewsMode);
 
     popup->addMenu(viewModeMenu);
     QAction* previewsMode = m_mainWindow->actionCollection()->action("previews");
     viewModeMenu->addAction(previewsMode);
 
     popup->addMenu(viewModeMenu);

-    popup->addSeparator();

 
 

-    QAction* bookmarkAction = popup->addAction(KIcon("bookmark-folder"), i18n("Bookmark this folder"));
+    QAction* toggleViewsAction = 0;
+    if (m_mainWindow->isSplit()) {
+        toggleViewsAction = popup->addAction(i18n("Toggle Views"));
+    }
+

     popup->addSeparator();
 
     popup->addSeparator();
 

-    QAction* propertiesAction = popup->addAction(i18n("Properties..."));
+    QAction* bookmarkAction = popup->addAction(KIcon("bookmark-folder"), i18n("Bookmark This Folder..."));
+    popup->addSeparator();

 
 

-    QAction* activatedAction = popup->exec(QCursor::pos());
-    if (activatedAction == propertiesAction) {
-        new KPropertiesDialog(m_mainWindow->activeView()->url());
-    }
-    else if (activatedAction == bookmarkAction) {
-        const KUrl& url = m_mainWindow->activeView()->url();
-        KBookmark bookmark = EditBookmarkDialog::getBookmark(i18n("Add folder as bookmark"),
-                                                             url.fileName(),
-                                                             url,
-                                                             "bookmark");
-        if (!bookmark.isNull()) {
-            KBookmarkManager* manager = DolphinSettings::instance().bookmarkManager();
-            KBookmarkGroup root = manager->root();
-            root.addBookmark(manager, bookmark);
-            manager->emitChanged(root);
+    QAction* propertiesAction = popup->addAction(i18n("Properties"));
+
+    QAction* action = popup->exec(QCursor::pos());
+    if (action == propertiesAction) {
+        const KUrl& url = m_mainWindow->activeViewContainer()->url();
+        KPropertiesDialog dialog(url);
+        dialog.exec();
+    } else if (action == bookmarkAction) {
+        const KUrl& url = m_mainWindow->activeViewContainer()->url();
+        if (url.isValid()) {
+            DolphinSettings::instance().placesModel()->addPlace(url.fileName(), url);

         }
         }

+    } else if ((toggleViewsAction != 0) && (action == toggleViewsAction)) {
+        m_mainWindow->toggleViews();

     }
 
     popup->deleteLater();
     }
 
     popup->deleteLater();


@@ -322,29 +268,11 @@ void DolphinContextMenu::insertDefaultItemActions(KMenu* popup)
 {
     Q_ASSERT(popup != 0);
     const KActionCollection* collection = m_mainWindow->actionCollection();
 {
     Q_ASSERT(popup != 0);
     const KActionCollection* collection = m_mainWindow->actionCollection();

-    const bool insertSidebarActions = (m_viewType == SidebarView);

 
     // insert 'Cut', 'Copy' and 'Paste'
 
     // insert 'Cut', 'Copy' and 'Paste'

-    QAction* cutAction = 0;
-    QAction* copyAction = 0;
-    QAction* pasteAction = 0;
-    if (insertSidebarActions) {
-        cutAction   = new QAction(KIcon("edit-cut"), i18n("Cut"), this);
-        connect(cutAction, SIGNAL(triggered()), this, SLOT(cut()));
-
-        copyAction  = new QAction(KIcon("edit-copy"), i18n("Copy"), this);
-        connect(copyAction, SIGNAL(triggered()), this, SLOT(copy()));
-
-        const QAction* menuPasteAction = collection->action(KStandardAction::stdName(KStandardAction::Paste));
-        pasteAction = new QAction(KIcon("edit-paste"), menuPasteAction->text(), this);
-        pasteAction->setEnabled(menuPasteAction->isEnabled());
-        connect(pasteAction, SIGNAL(triggered()), this, SLOT(paste()));
-    }
-    else {
-        cutAction   = collection->action(KStandardAction::stdName(KStandardAction::Cut));
-        copyAction  = collection->action(KStandardAction::stdName(KStandardAction::Copy));
-        pasteAction = collection->action(KStandardAction::stdName(KStandardAction::Paste));
-    }
+    QAction* cutAction = collection->action(KStandardAction::stdName(KStandardAction::Cut));
+    QAction* copyAction  = collection->action(KStandardAction::stdName(KStandardAction::Copy));
+    QAction* pasteAction = collection->action(KStandardAction::stdName(KStandardAction::Paste));

 
     popup->addAction(cutAction);
     popup->addAction(copyAction);
 
     popup->addAction(cutAction);
     popup->addAction(copyAction);


@@ -352,52 +280,29 @@ void DolphinContextMenu::insertDefaultItemActions(KMenu* popup)
     popup->addSeparator();
 
     // insert 'Rename'
     popup->addSeparator();
 
     // insert 'Rename'

-    QAction* renameAction = 0;
-    if (insertSidebarActions) {
-        renameAction = new QAction(i18n("Rename"), this);
-        connect(renameAction, SIGNAL(triggered()), this, SLOT(paste()));
-    }
-    else {
-        collection->action("rename");
-    }
+    QAction* renameAction = collection->action("rename");

     popup->addAction(renameAction);
 
     // insert 'Move to Trash' and (optionally) 'Delete'
     const KSharedConfig::Ptr globalConfig = KSharedConfig::openConfig("kdeglobals", KConfig::NoGlobals);
     const KConfigGroup kdeConfig(globalConfig, "KDE");
     bool showDeleteCommand = kdeConfig.readEntry("ShowDeleteCommand", false);
     popup->addAction(renameAction);
 
     // insert 'Move to Trash' and (optionally) 'Delete'
     const KSharedConfig::Ptr globalConfig = KSharedConfig::openConfig("kdeglobals", KConfig::NoGlobals);
     const KConfigGroup kdeConfig(globalConfig, "KDE");
     bool showDeleteCommand = kdeConfig.readEntry("ShowDeleteCommand", false);

-    const KUrl& url = insertSidebarActions ? m_fileInfo->url():
-                                             m_mainWindow->activeView()->url();
+    const KUrl& url = m_mainWindow->activeViewContainer()->url();

     if (url.isLocalFile()) {
     if (url.isLocalFile()) {

-        QAction* moveToTrashAction = 0;
-        if (insertSidebarActions) {
-            moveToTrashAction = new QAction(KIcon("edit-trash"), i18n("Move To Trash"), this);
-            connect(moveToTrashAction, SIGNAL(triggered()), this, SLOT(moveToTrash()));
-        }
-        else {
-            moveToTrashAction = collection->action("move_to_trash");
-        }
+        QAction* moveToTrashAction = collection->action("move_to_trash");

         popup->addAction(moveToTrashAction);
         popup->addAction(moveToTrashAction);

-    }
-    else {
+    } else {

         showDeleteCommand = true;
     }
 
     if (showDeleteCommand) {
         showDeleteCommand = true;
     }
 
     if (showDeleteCommand) {

-        QAction* deleteAction = 0;
-        if (insertSidebarActions) {
-            deleteAction = new QAction(KIcon("edit-delete"), i18n("Delete"), this);
-            connect(deleteAction, SIGNAL(triggered()), this, SLOT(deleteItem()));
-        }
-        else {
-            deleteAction = collection->action("delete");
-        }
+        QAction* deleteAction = collection->action("delete");

         popup->addAction(deleteAction);
     }
 }
 
 QList<QAction*> DolphinContextMenu::insertOpenWithItems(KMenu* popup,
         popup->addAction(deleteAction);
     }
 }
 
 QList<QAction*> DolphinContextMenu::insertOpenWithItems(KMenu* popup,

-                                                        QVector<KService::Ptr>& openWithVector)
+        QVector<KService::Ptr>& openWithVector)

 {
     // Parts of the following code have been taken
     // from the class KonqOperations located in
 {
     // Parts of the following code have been taken
     // from the class KonqOperations located in


@@ -422,12 +327,12 @@ QList<QAction*> DolphinContextMenu::insertOpenWithItems(KMenu* popup,
         // fill the 'Open with' sub menu with application types
         const KMimeType::Ptr mimePtr = KMimeType::findByUrl(m_fileInfo->url());
         KService::List offers = KMimeTypeTrader::self()->query(mimePtr->name(),
         // fill the 'Open with' sub menu with application types
         const KMimeType::Ptr mimePtr = KMimeType::findByUrl(m_fileInfo->url());
         KService::List offers = KMimeTypeTrader::self()->query(mimePtr->name(),

-                                                               "Application",
-                                                               "Type == 'Application'");
+                                "Application",
+                                "Type == 'Application'");

         if (offers.count() > 0) {
             KService::List::Iterator it;
             KMenu* openWithMenu = new KMenu(i18n("Open With"));
         if (offers.count() > 0) {
             KService::List::Iterator it;
             KMenu* openWithMenu = new KMenu(i18n("Open With"));

-            for(it = offers.begin(); it != offers.end(); ++it) {
+            for (it = offers.begin(); it != offers.end(); ++it) {

                 // The offer list from the KTrader returns duplicate
                 // application entries. Although this seems to be a configuration
                 // problem outside the scope of Dolphin, duplicated entries just
                 // The offer list from the KTrader returns duplicate
                 // application entries. Although this seems to be a configuration
                 // problem outside the scope of Dolphin, duplicated entries just


@@ -446,16 +351,14 @@ QList<QAction*> DolphinContextMenu::insertOpenWithItems(KMenu* popup,
 
             openWithActions << action;
             popup->addMenu(openWithMenu);
 
             openWithActions << action;
             popup->addMenu(openWithMenu);

-        }
-        else {
+        } else {

             // No applications are registered, hence just offer
             // a "Open With..." item instead of a sub menu containing
             // only one entry.
             QAction* action = popup->addAction(i18n("Open With..."));
             openWithActions << action;
         }
             // No applications are registered, hence just offer
             // a "Open With..." item instead of a sub menu containing
             // only one entry.
             QAction* action = popup->addAction(i18n("Open With..."));
             openWithActions << action;
         }

-    }
-    else {
+    } else {

         // At least one of the selected items has a different MIME type. In this case
         // just show a disabled "Open With..." entry.
         QAction* action = popup->addAction(i18n("Open With..."));
         // At least one of the selected items has a different MIME type. In this case
         // just show a disabled "Open With..." entry.
         QAction* action = popup->addAction(i18n("Open With..."));


@@ -466,7 +369,7 @@ QList<QAction*> DolphinContextMenu::insertOpenWithItems(KMenu* popup,
 }
 
 QList<QAction*> DolphinContextMenu::insertActionItems(KMenu* popup,
 }
 
 QList<QAction*> DolphinContextMenu::insertActionItems(KMenu* popup,

-                                                      QVector<KDEDesktopMimeType::Service>& actionsVector)
+        QVector<KDesktopFileActions::Service>& actionsVector)

 {
     // Parts of the following code have been taken
     // from the class KonqOperations located in
 {
     // Parts of the following code have been taken
     // from the class KonqOperations located in


@@ -488,7 +391,7 @@ QList<QAction*> DolphinContextMenu::insertActionItems(KMenu* popup,
         QStringList entries = dir.entryList(QDir::Files);
 
         for (QStringList::ConstIterator entryIt = entries.begin(); entryIt != entries.end(); ++entryIt) {
         QStringList entries = dir.entryList(QDir::Files);
 
         for (QStringList::ConstIterator entryIt = entries.begin(); entryIt != entries.end(); ++entryIt) {

-            KConfigGroup cfg(KSharedConfig::openConfig( *dirIt + *entryIt, KConfig::OnlyLocal), "Desktop Entry" );
+            KConfigGroup cfg(KSharedConfig::openConfig(*dirIt + *entryIt, KConfig::OnlyLocal), "Desktop Entry");

             if ((cfg.hasKey("Actions") || cfg.hasKey("X-KDE-GetActionMenu")) && cfg.hasKey("ServiceTypes")) {
                 //const QStringList types = cfg.readListEntry("ServiceTypes");
                 QStringList types;
             if ((cfg.hasKey("Actions") || cfg.hasKey("X-KDE-GetActionMenu")) && cfg.hasKey("ServiceTypes")) {
                 //const QStringList types = cfg.readListEntry("ServiceTypes");
                 QStringList types;


@@ -529,24 +432,23 @@ QList<QAction*> DolphinContextMenu::insertActionItems(KMenu* popup,
                     if (insert) {
                         menu = actionsMenu;
 
                     if (insert) {
                         menu = actionsMenu;
 

-                        const QString submenuName = cfg.readEntry( "X-KDE-Submenu" );
+                        const QString submenuName = cfg.readEntry("X-KDE-Submenu");

                         if (!submenuName.isEmpty()) {
                             menu = new KMenu(submenuName);
                             actionsMenu->addMenu(menu);
                         }
 
                         if (!submenuName.isEmpty()) {
                             menu = new KMenu(submenuName);
                             actionsMenu->addMenu(menu);
                         }
 

-                        Q3ValueList<KDEDesktopMimeType::Service> userServices =
-                            KDEDesktopMimeType::userDefinedServices(*dirIt + *entryIt, true);
+                        Q3ValueList<KDesktopFileActions::Service> userServices =
+                            KDesktopFileActions::userDefinedServices(*dirIt + *entryIt, true);

 
 

-                        Q3ValueList<KDEDesktopMimeType::Service>::Iterator serviceIt;
+                        Q3ValueList<KDesktopFileActions::Service>::Iterator serviceIt;

                         for (serviceIt = userServices.begin(); serviceIt != userServices.end(); ++serviceIt) {
                         for (serviceIt = userServices.begin(); serviceIt != userServices.end(); ++serviceIt) {

-                            KDEDesktopMimeType::Service service = (*serviceIt);
+                            KDesktopFileActions::Service service = (*serviceIt);

                             if (!service.m_strIcon.isEmpty()) {
                             if (!service.m_strIcon.isEmpty()) {

-                                QAction* action = menu->addAction(SmallIcon(service.m_strIcon),
+                                QAction* action = menu->addAction(KIcon(service.m_strIcon),

                                                                   service.m_strName);
                                 serviceActions << action;
                                                                   service.m_strName);
                                 serviceActions << action;

-                            }
-                            else {
+                            } else {

                                 QAction *action = menu->addAction(service.m_strName);
                                 serviceActions << action;
                             }
                                 QAction *action = menu->addAction(service.m_strName);
                                 serviceActions << action;
                             }


@@ -579,21 +481,18 @@ QList<QAction*> DolphinContextMenu::insertActionItems(KMenu* popup,
                 QAction* action = popup->addAction(text);
                 serviceActions.clear();
                 serviceActions << action;
                 QAction* action = popup->addAction(text);
                 serviceActions.clear();
                 serviceActions << action;

-            }
-            else {
+            } else {

                 QAction* action = popup->addAction(icon, text);
                 serviceActions.clear();
                 serviceActions << action;
             }
                 QAction* action = popup->addAction(icon, text);
                 serviceActions.clear();
                 serviceActions << action;
             }

-        }
-        else {
+        } else {

             // The item is a sub menu, hence show the sub menu in the root menu.
             popup->addMenu(menu);
         }
         actionsMenu->deleteLater();
         actionsMenu = 0;
             // The item is a sub menu, hence show the sub menu in the root menu.
             popup->addMenu(menu);
         }
         actionsMenu->deleteLater();
         actionsMenu = 0;

-    }
-    else {
+    } else {

         popup->addMenu(actionsMenu);
     }
 
         popup->addMenu(actionsMenu);
     }