From: Kai Uwe Broulik <kde@privat.broulik.de>
Date: Tue, 3 Jan 2017 08:46:43 +0000 (+0100)
Subject: Explicitly pass iconNameForUrl when creating new places entry
X-Git-Url: https://cloud.milkyroute.net/gitweb/dolphin.git/commitdiff_plain/996f8520e7003326ade6ea96929dd41958ba308d

Explicitly pass iconNameForUrl when creating new places entry

Once KIO::iconNameForUrl returns special icons for standard paths (e.g. Video folder)
this will ensure when adding such a folder to places it will automatically get the proper icon.

Since KIO already depends on KBookmarks it can't depend the other way round, otherwise
this would have been fixed in KBookmarks directly.

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

diff --git a/src/dolphincontextmenu.cpp b/src/dolphincontextmenu.cpp
index ed3f64379..666b81c8d 100644
--- a/src/dolphincontextmenu.cpp
+++ b/src/dolphincontextmenu.cpp
@@ -307,7 +307,7 @@ void DolphinContextMenu::openItemContextMenu()
             if (selectedUrl.isValid()) {
                 PlacesItemModel model;
                 const QString text = selectedUrl.fileName();
-                PlacesItem* item = model.createPlacesItem(text, selectedUrl);
+                PlacesItem* item = model.createPlacesItem(text, selectedUrl, KIO::iconNameForUrl(selectedUrl));
                 model.appendItemToGroup(item);
                 model.saveBookmarks();
             }
@@ -373,7 +373,8 @@ void DolphinContextMenu::openViewportContextMenu()
         if (container->url().isValid()) {
             PlacesItemModel model;
             PlacesItem* item = model.createPlacesItem(container->placesText(),
-                                                      container->url());
+                                                      container->url(),
+                                                      KIO::iconNameForUrl(container->url()));
             model.appendItemToGroup(item);
             model.saveBookmarks();
         }