diff options
| author | Kai Uwe Broulik <[email protected]> | 2020-05-05 12:05:18 +0200 |
|---|---|---|
| committer | Kai Uwe Broulik <[email protected]> | 2020-05-05 12:05:18 +0200 |
| commit | 9f268f5f76a4a4f0c91abcc213e159c8bde4869c (patch) | |
| tree | 56b336266807c81d7986dc35e4a8d662d3e5f1ef /src | |
| parent | 2c72b18661f0225a7c4b0d9485c2186e6ff09e62 (diff) | |
Don't use local URL for KMoreTool
This method is called a billion times and spawns a blocking(!) StatJob.
KFind can search local URLs, too, so I don't see why this is necessary.
Either way, I'd rather have a non-KDE search tool broken than the app.
Ideally, I'd like this entire thing redone to use an async StatJob, if any.
Fwiw the KMoreTools in the free space bar doesn't use a local URL either.
BUG: 420911
FIXED-IN: 20.04.1
Differential Revision: https://phabricator.kde.org/D29442
Diffstat (limited to 'src')
| -rw-r--r-- | src/dolphinmainwindow.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/dolphinmainwindow.cpp b/src/dolphinmainwindow.cpp index f88bc3f44..7ad136fe7 100644 --- a/src/dolphinmainwindow.cpp +++ b/src/dolphinmainwindow.cpp @@ -956,7 +956,7 @@ QPointer<QAction> DolphinMainWindow::preferredSearchTool() { m_searchTools.clear(); KMoreToolsMenuFactory("dolphin/search-tools").fillMenuFromGroupingNames( - &m_searchTools, { "files-find" }, QUrl::fromLocalFile(activeContainerLocalPath()) + &m_searchTools, { "files-find" }, m_activeViewContainer->url() ); QList<QAction*> actions = m_searchTools.actions(); if (actions.isEmpty()) { |