blumia/pineapple-pictures
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix: proper check for filemanager1 dbus

Browse Source
This commit is contained in:
Gary Wang
2022-11-01 20:58:48 +08:00
parent 51e49ab8a8
commit d1fbf64011
1 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
app/mainwindow.cpp
View File

@ -38,6 +38,7 @@
#ifdef HAVE_QTDBUS
#include <QDBusInterface>
#include <QDBusConnectionInterface>
#endif // HAVE_QTDBUS
MainWindow::MainWindow(QWidget *parent)
@ -748,13 +749,18 @@ void MainWindow::on_actionLocateInFileManager_triggered()
QProcess::startDetached("explorer", QStringList() << "/select," << QDir::toNativeSeparators(fileInfo.absoluteFilePath()));
#elif defined(Q_OS_LINUX) and defined(HAVE_QTDBUS)
// Use https://www.freedesktop.org/wiki/Specifications/file-manager-interface/ if possible
const QDBusConnectionInterface * dbusIface = QDBusConnection::sessionBus().interface();
if (!dbusIface || !dbusIface->isServiceRegistered(QLatin1String("org.freedesktop.FileManager1"))) {
QDesktopServices::openUrl(folderUrl);
return;
}
QDBusInterface fm1Iface(QStringLiteral("org.freedesktop.FileManager1"),
QStringLiteral("/org/freedesktop/FileManager1"),
QStringLiteral("org.freedesktop.FileManager1"));
fm1Iface.setTimeout(500);
fm1Iface.setTimeout(1000);
fm1Iface.callWithArgumentList(QDBus::Block, "ShowItems", {
QStringList{currentFileUrl.toString()},
QString::number(QDateTime::currentSecsSinceEpoch())
QString()
});
if (fm1Iface.lastError().isValid()) {
QDesktopServices::openUrl(folderUrl);