Commit ada8fca9 authored by Алексей Шилин's avatar Алексей Шилин
Browse files

Remove D10890 workaround

Summary:
D10890 was a workaround related to KRecentDocument::add() bug (see D10863)
which is fixed in Frameworks 5.46. Given that Plasma 5.13 depends on that
Frameworks version, it's safe to remove this no longer needed workaround
without breaking anything.

Reviewers: #plasma, broulik, hein

Reviewed By: #plasma, hein

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D12701
parent 20a8dcb8
......@@ -241,13 +241,6 @@ QVariantList recentDocumentActions(KService::Ptr service)
| Activity::current()
| Url::file();
// Due to KRecentDocument::add() bug, application name "<app>" could
// be stored instead of its desktop entry name "org.kde.<app>". Let's
// check for both in order to get all results for the application.
if (storageId.startsWith("org.kde.")) {
query = query | Agent(storageId.mid(8));
}
ResultSet results(query);
ResultSet::const_iterator resultIt;
......@@ -305,13 +298,6 @@ bool handleRecentDocumentAction(KService::Ptr service, const QString &actionId,
| Activity::current()
| Url::file();
// Due to KRecentDocument::add() bug, application name "<app>" could
// be stored instead of its desktop entry name "org.kde.<app>". Let's
// check for both in order to get all results for the application.
if (storageId.startsWith("org.kde.")) {
query = query | Agent(storageId.mid(8));
}
KAStats::forgetResources(query);
return false;
......
......@@ -311,13 +311,6 @@ QVariantList Backend::recentDocumentActions(const QUrl &launcherUrl, QObject *pa
| Activity::current()
| Url::file();
// Due to KRecentDocument::add() bug, application name "<app>" could
// be stored instead of its desktop entry name "org.kde.<app>". Let's
// check for both in order to get all results for the application.
if (storageId.startsWith(QLatin1String("org.kde."))) {
query = query | Agent(storageId.mid(8));
}
ResultSet results(query);
ResultSet::const_iterator resultIt = results.begin();
......@@ -408,13 +401,6 @@ void Backend::handleRecentDocumentAction() const
| Activity::current()
| Url::file();
// Due to KRecentDocument::add() bug, application name "<app>" could
// be stored instead of its desktop entry name "org.kde.<app>". Let's
// check for both in order to get all results for the application.
if (agent.startsWith(QLatin1String("org.kde."))) {
query = query | Agent(agent.mid(8));
}
KAStats::forgetResources(query);
return;
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment