Differential D23112 Diff 63647 src/service/plugins/gtk-eventspy/GtkEventSpy.cpp

Changeset View

Standalone View

View Options

src/service/plugins/gtk-eventspy/GtkEventSpy.cpp

  • This file was added.
1​/*
2​ * Copyright (C) 2019 Méven Car (meven.car@kdemail.net)
3​ *
4​ * This program is free software; you can redistribute it and/or
5​ * modify it under the terms of the GNU General Public License as
6​ * published by the Free Software Foundation; either version 2 of
7​ * the License, or (at your option) any later version.
8​ *
9​ * This program is distributed in the hope that it will be useful,
10​ * but WITHOUT ANY WARRANTY; without even the implied warranty of
11​ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12​ * GNU General Public License for more details.
13​ *
14​ * You should have received a copy of the GNU General Public License
15​ * along with this program. If not, see <http://www.gnu.org/licenses/>.
16​ */
17​
18​#include "GtkEventSpy.h"
19​
20​#include <QStringList>
21​#include <QString>
22​#include <QFileInfo>
23​#include <QXmlSimpleReader>
24​#include <QXmlInputSource>
25​
26​#include <KIOCore/KRecentDocument>
27​#include <KCoreAddons/KDirWatch>
28​#include <KConfigCore/KDesktopFile>
29​#include <KConfigCore/KConfigGroup>
30​#include <KServiceTypeTrader>
31​
32​KAMD_EXPORT_PLUGIN(GtkEventSpyPlugin, GtkEventSpyPlugin, "kactivitymanagerd-plugin-gtk-eventspy.json")
33​
34​GtkEventSpyPlugin::GtkEventSpyPlugin(QObject *parent, const QVariantList &args)
35​ : Plugin(parent)
36​ , m_resources(nullptr)
37​ , m_dirWatcher(new KDirWatch(this))
38​ , m_lastUpdate(QDateTime::currentDateTime())
39​{
40​ Q_UNUSED(args);
41​
42​ // gtk xml history file
43​ // usually $HOME/.local/share/recently-used.xbel
44​ QString filename = QStandardPaths::writableLocation(QStandardPaths::GenericDataLocation) + QLatin1String("/recently-used.xbel");
45​
46​ m_dirWatcher->addFile(filename);
47​ connect(m_dirWatcher.get(), &KDirWatch::dirty,
48​ this, &GtkEventSpyPlugin::fileUpdated);
49​ connect(m_dirWatcher.get(), &KDirWatch::created,
50​ this, &GtkEventSpyPlugin::fileUpdated);
51​
52​ m_dirWatcher->setDirty(filename);
53​}
54​
55​struct Application {
56​ QString name;
57​ QDateTime modified;
58​};
59​
60​class Bookmark {
61​
62​public:
63​ QUrl href;
64​ QDateTime added;
65​ QDateTime modified;
66​ QDateTime visited;
67​ QList<Application> *applications;
68​
69​ Bookmark(){
ivanUnsubmitted
Done

No need for this to be a pointer to a list. Make it just QList<Application> applications.

ivan: No need for this to be a pointer to a list. Make it just `QList<Application> applications`.
70​ applications = new QList<Application>();
71​ }
ivanUnsubmitted
Done

The constructor will not be needed once applications stops being a pointer.

ivan: The constructor will not be needed once `applications` stops being a pointer.
72​ QString latestApplication() const;
73​};
74​
75​QString Bookmark::latestApplication() const {
76​ Application current = applications->first();
77​ for (const Application &app : *applications) {
ivanUnsubmitted
Done

{ which starts a function should be on a new line (I don't care much about this, but let's follow the KF5 style)

ivan: `{` which starts a function should be on a new line (I don't care much about this, but let's…
78​ if (app.modified > current.modified) {
79​ current = app;
ivanUnsubmitted
Done

When you make applications not to be a pointer qAsConst will not be needed as this is a const member function.

ivan: When you make `applications` not to be a pointer `qAsConst` will not be needed as this is a…
80​ }
81​ }
82​ return current.name;
83​}
84​
85​class BookmarkHandler : public QXmlDefaultHandler
86​{
87​public:
88​ BookmarkHandler(){
89​ current = nullptr;
90​ marks = QList<Bookmark>();
ivanUnsubmitted
Done

Replace with:

BookmarkHandler()
    : current(nullptr)
{
}
ivan: Replace with: BookmarkHandler() : current(nullptr) { }
91​ }
92​
ivanUnsubmitted
Done

No need for this - marks are already default-constructed.

ivan: No need for this - marks are already default-constructed.
93​ bool startElement(const QString &namespaceURI, const QString &localName,
94​ const QString &qName, const QXmlAttributes &attributes) override;
95​ bool endElement(const QString &namespaceURI, const QString &localName, const QString &qName) override;
96​
97​ QList<Bookmark> bookmarks() const;
98​private:
99​ QList<Bookmark> marks;
100​ Bookmark* current;
101​};
102​
103​QList<Bookmark> BookmarkHandler::bookmarks() const {
104​ return marks;
105​}
106​
107​bool BookmarkHandler::startElement(const QString &/*namespaceURI*/, const QString &/*localName*/,
108​ const QString &qName, const QXmlAttributes &attributes)
109​{
110​ // new bookmark
111​ if (qName == QStringLiteral("bookmark")) {
112​ current = new Bookmark();
113​ current->href = QUrl(attributes.value("href"));
114​ QString added = attributes.value("added");
ivanUnsubmitted
Done

No need for dynamic allocation. Make it a normal variable instead of a pointer.

ivan: No need for dynamic allocation. Make it a normal variable instead of a pointer.
115​ QString modified = attributes.value("modified");
116​ QString visited = attributes.value("visited");
117​ current->added = QDateTime::fromString(added, Qt::ISODate);
118​ current->modified = QDateTime::fromString(modified, Qt::ISODate);
119​ current->visited = QDateTime::fromString(visited, Qt::ISODate);
120​
121​ // application for the current bookmark
122​ } else if (qName == QStringLiteral("bookmark:application")) {
123​ Application app;
124​
125​ QString exec = attributes.value("exec");
126​
broulikUnsubmitted
Done

It stores the full Exec line of the desktop file, including the %u.
What you probably want do to is use KServiceTypeTrader to find the application with said Exec line.

auto services = KServiceTypeTrader::self()->query(QStringLiteral("Application"), QStringLiteral("exist Exec and ('%1' == Exec)").arg(cmd));

and then operate on the KService desktop file name, e.g. org.gnome.evince instead of the executable.

broulik: It stores the full `Exec` line of the desktop file, including the `%u`. What you probably want…
127​ // remove "'" caracters wrapping the command
ivanUnsubmitted
Done

No need to compare chars with strings:

if (!exec.isEmpty() && exec[0] == '\'' && exec.back() == '\'')
ivan: No need to compare chars with strings: ``` if (!exec.isEmpty() && exec[0] == '\'' && exec.back…
brunsUnsubmitted
Done

missing QChar('\''), otherwise it breaks with QT_NO_CAST_FROM_ASCII

-> if (exec.startsWith(QChar('\'') && exec.endsWith(QChar('\'')) {...

bruns: missing `QChar('\'')`, otherwise it breaks with QT_NO_CAST_FROM_ASCII -> `if (exec.startsWith…
128​ exec = exec.mid(1, exec.size() -2 );
129​
130​ // Search for applications which are executable and case-insensitively match the search term
ivanUnsubmitted
Done

It might be dangerous to assume that the command is wrapped with single quotes

ivan: It might be dangerous to assume that the command is wrapped with single quotes
131​ // See https://techbase.kde.org/Development/Tutorials/Services/Traders#The_KTrader_Query_Language
132​ QString query = QString("exist Exec and ((Exec ~~ '%1') or (exist GenericName and '%1' ~~ GenericName) or (exist Name and '%1' ~~ Name))").arg(exec);
133​ KService::List services = KServiceTypeTrader::self()->query(QStringLiteral("Application"), query);
134​
ivanUnsubmitted
Done

const auto query

ivan: const auto query
135​ if (!services.isEmpty()) {
ivanUnsubmitted
Done

This should also be const, other local variables as well

ivan: This should also be `const`, other local variables as well
136​ // use the first item matching
broulikUnsubmitted
Done

Isn't that a bit coarse? Doesn't matching the Exec line exactly work?

broulik: Isn't that a bit coarse? Doesn't matching the `Exec` line exactly work?
137​ auto service = services.first();
138​ app.name = service->desktopEntryName();
139​ } else {
140​ // when no services are found, sanitize a little the exec
141​ // remove space and any caracter after
142​ int spaceIdex = exec.indexOf(" ");
143​ if (spaceIdex != -1) {
144​ exec = exec.mid(0, spaceIdex);
ivanUnsubmitted
Done

Let's hope commands will never have spaces in them :)

ivan: Let's hope commands will never have spaces in them :)
mevenAuthorUnsubmitted
Done

It is to just extract the executable name, we don't want to have an exploding number of initiatingAgent for every argument and parameter that might come through here.

meven: It is to just extract the executable name, we don't want to have an exploding number of…
ivanUnsubmitted
Done

I meant it will be a problem if someone decides to have a space in the executable like my\ aweomse\ binary. But this should not be an issue at the moment.

ivan: I meant it will be a problem if someone decides to have a space in the executable like `my\…
145​ }
brunsUnsubmitted
Done

spaceI_n_dex

bruns: `spaceI_n_dex`
146​ app.name = exec;
147​ }
148​
149​ app.modified = QDateTime::fromString(attributes.value("modified"), Qt::ISODate);
150​
151​ current->applications->append(app);
152​ }
153​ return true;
154​}
155​
156​bool BookmarkHandler::endElement(const QString &namespaceURI, const QString &localName, const QString &qName) {
157​ Q_UNUSED(namespaceURI);
158​ Q_UNUSED(localName);
159​
160​ if (qName == QStringLiteral("bookmark")) {
161​ // keep track of the finished parsed bookmark
162​ marks << *current;
163​ }
164​
165​ return true;
166​}
167​
168​void GtkEventSpyPlugin::fileUpdated(const QString &filename)
169​{
170​ QFile file(filename);
171​ if ( ! file.open(QFile::ReadOnly | QFile::Text) ) {
172​ qWarning() << "Could not read" << filename;
173​ return;
174​ }
175​
176​ // must parse the xbel xml file
177​ BookmarkHandler bookmarkHandler;
178​
179​ QXmlSimpleReader reader;
180​ reader.setContentHandler(&bookmarkHandler);
181​ reader.setErrorHandler(&bookmarkHandler);
182​ QXmlInputSource *source = new QXmlInputSource(&file);
183​
184​ if ( ! reader.parse(source) ) {
185​ qWarning() << "could not parse" << file << "error was " << bookmarkHandler.errorString();
186​ return ;
ivanUnsubmitted
Done

No need for dynamic allocation. Make it a normal variable instead of a pointer.

ivan: No need for dynamic allocation. Make it a normal variable instead of a pointer.
187​ }
188​
189​ // then find the files that were accessed since last run
190​ const QList<Bookmark> bookmarks = bookmarkHandler.bookmarks();
191​ for (Bookmark mark : bookmarks) {
192​ if (mark.added > m_lastUpdate || mark.modified > m_lastUpdate || mark.visited > m_lastUpdate) {
193​ addDocument(mark.href, mark.latestApplication());
194​ }
195​ }
196​
197​ m_lastUpdate = QDateTime::currentDateTime();
198​}
199​
200​void GtkEventSpyPlugin::addDocument(const QUrl &url, const QString &application)
201​{
202​ const QString name = url.fileName();
203​
204​ Plugin::invoke<Qt::QueuedConnection>(
205​ m_resources, "RegisterResourceEvent",
206​ Q_ARG(QString, application), // Application
207​ Q_ARG(uint, 0), // Window ID
208​ Q_ARG(QString, url.toString()), // URI
209​ Q_ARG(uint, 0) // Event Activities::Accessed
210​ );
211​}
212​
213​GtkEventSpyPlugin::~GtkEventSpyPlugin()
214​{
215​}
216​
217​bool GtkEventSpyPlugin::init(QHash<QString, QObject *> &modules)
218​{
219​ Plugin::init(modules);
220​
221​ m_resources = modules["resources"];
222​
223​ return true;
224​}
225​
226​#include "GtkEventSpy.moc"
227​