iiiypuk/cantata
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Use QGuiApplication::applicationDisplayName() as application name

Browse Source 
Using a consistent spelling of the application name (w.r.t. case-sensitivity)
whenever communicating with the desktop environment is common practice to
always have the same name displayed (as window title, in a window switcher
dialog, etc.). That is also what already has been done in Notify::show().
This commit is contained in:
Thomas Uhle
2020-06-06 18:55:36 +02:00
parent 9f5e3bfa33
commit 1d25d0dbfc
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
dbus/powermanagement.cpp
View File

@@ -28,6 +28,7 @@
#include "upowerinterface.h"
#include "login1interface.h"
#include "mpd-interface/mpdstatus.h"
#include <QGuiApplication>
#include <QString>
GLOBAL_STATIC(PowerManagement, instance)
@@ -77,17 +78,17 @@ void PowerManagement::beginSuppressingSleep()
QString reason=tr("Cantata is playing a track");
QDBusReply<uint> reply;
if (policy->isValid()) {
reply = policy->AddInhibition((uint)1, QCoreApplication::applicationName(), reason);
reply = policy->AddInhibition((uint)1, QGuiApplication::applicationDisplayName(), reason);
} else {
// Fallback to the fd.o Inhibit interface
reply = inhibit->Inhibit(QCoreApplication::applicationName(), reason);
reply = inhibit->Inhibit(QGuiApplication::applicationDisplayName(), reason);
}
cookie=reply.isValid() ? reply : -1;
QString types=QStringLiteral("sleep");
QString mode=QStringLiteral("block");
QDBusPendingReply<QDBusUnixFileDescriptor> futureReply;
futureReply = login1->Inhibit(types, QCoreApplication::applicationName(), reason, mode);
futureReply = login1->Inhibit(types, QGuiApplication::applicationDisplayName(), reason, mode);
futureReply.waitForFinished();
if (futureReply.isValid()) {
descriptor=futureReply.value();