[systemsettings] Updated udisks2 defines and add it to devel package.…

… Contributes to JB#42530
sailfishos · Aug 30, 2018 · 8ee508e · 8ee508e
1 parent 43908cd
commit 8ee508e
Show file tree

Hide file tree

Showing 6 changed files with 24 additions and 18 deletions.
diff --git a/src/src.pro b/src/src.pro
@@ -57,6 +57,7 @@ PUBLIC_HEADERS = \
     connmanvpnconnectionproxy.h \
     developermodesettings.h \
     batterystatus.h \
+    udisks2defines.h \
     diskusage.h \
     partition.h \
     partitionmanager.h \
@@ -77,7 +78,6 @@ HEADERS += \
     partition_p.h \
     partitionmanager_p.h \
     udisks2block_p.h \
-    udisks2defines.h \
     udisks2job_p.h \
     udisks2monitor_p.h
 

diff --git a/src/udisks2block.cpp b/src/udisks2block.cpp
@@ -7,7 +7,7 @@
 #include <QDBusPendingCallWatcher>
 #include <QDBusPendingReply>
 
-UDisks2::Block::Block(const QString &path, const InterfacePropertyMap &interfacePropertyMap, QObject *parent)
+UDisks2::Block::Block(const QString &path, const UDisks2::InterfacePropertyMap &interfacePropertyMap, QObject *parent)
     : QObject(parent)
     , m_path(path)
     , m_interfacePropertyMap(interfacePropertyMap)

diff --git a/src/udisks2block_p.h b/src/udisks2block_p.h
@@ -36,9 +36,7 @@
 #include <QVariantMap>
 #include <QDBusConnection>
 
-typedef QMap<QString, QVariantMap> InterfacePropertyMap;
-
-Q_DECLARE_METATYPE(InterfacePropertyMap)
+#include "udisks2defines.h"
 
 class QDBusPendingCallWatcher;
 
@@ -49,7 +47,7 @@ class Block : public QObject
     Q_OBJECT
 
 public:
-    Block(const QString &path, const InterfacePropertyMap &interfacePropertyMap, QObject *parent = nullptr);
+    Block(const QString &path, const UDisks2::InterfacePropertyMap &interfacePropertyMap, QObject *parent = nullptr);
     Block& operator=(const Block& other);
 
     ~Block();
@@ -112,7 +110,7 @@ private slots:
     void getEncryptedInterface();
 
     QString m_path;
-    InterfacePropertyMap m_interfacePropertyMap;
+    UDisks2::InterfacePropertyMap m_interfacePropertyMap;
     QVariantMap m_data;
     QDBusConnection m_connection;
     QString m_mountPath;

diff --git a/src/udisks2defines.h b/src/udisks2defines.h
@@ -32,11 +32,18 @@
 #ifndef UDISKS2_DEFINES
 #define UDISKS2_DEFINES
 
+#include <QVariantMap>
+
 namespace UDisks2 {
     static const auto propertiesChangedSignal = QStringLiteral("PropertiesChanged");
-    static const auto cryptoBackingDeviceKey = QStringLiteral("CryptoBackingDevice");
+    static const auto interfacesAddedSignal   = QStringLiteral("InterfacesAdded");
+    static const auto interfacesRemovedSignal = QStringLiteral("InterfacesRemoved");
+    static const auto cryptoBackingDeviceKey  = QStringLiteral("CryptoBackingDevice");
+
+    typedef QMap<QString, QVariantMap> InterfacePropertyMap;
 }
 
+Q_DECLARE_METATYPE(UDisks2::InterfacePropertyMap)
 
 #define DBUS_OBJECT_MANAGER_INTERFACE    QLatin1String("org.freedesktop.DBus.ObjectManager")
 #define DBUS_OBJECT_PROPERTIES_INTERFACE QLatin1String("org.freedesktop.DBus.Properties")
@@ -49,6 +56,7 @@ namespace UDisks2 {
 // Interfaces
 #define UDISKS2_MANAGER_INTERFACE    QLatin1String("org.freedesktop.UDisks2.Manager")
 #define UDISKS2_ENCRYPTED_INTERFACE  QLatin1String("org.freedesktop.UDisks2.Encrypted")
+#define UDISKS2_DRIVE_INTERFACE      QLatin1String("org.freedesktop.UDisks2.Drive")
 #define UDISKS2_BLOCK_INTERFACE      QLatin1String("org.freedesktop.UDisks2.Block")
 #define UDISKS2_FILESYSTEM_INTERFACE QLatin1String("org.freedesktop.UDisks2.Filesystem")
 #define UDISKS2_PARTITION_INTERFACE  QLatin1String("org.freedesktop.UDisks2.Partition")

diff --git a/src/udisks2monitor.cpp b/src/udisks2monitor.cpp
@@ -85,7 +85,7 @@ UDisks2::Monitor::Monitor(PartitionManagerPrivate *manager, QObject *parent)
     Q_ASSERT(!sharedInstance);
     sharedInstance = this;
 
-    qDBusRegisterMetaType<InterfacePropertyMap>();
+    qDBusRegisterMetaType<UDisks2::InterfacePropertyMap>();
     QDBusConnection systemBus = QDBusConnection::systemBus();
 
     connect(systemBus.interface(), &QDBusConnectionInterface::callWithCallbackFailed, this, [this](const QDBusError &error, const QDBusMessage &call) {
@@ -104,17 +104,17 @@ UDisks2::Monitor::Monitor(PartitionManagerPrivate *manager, QObject *parent)
                 UDISKS2_SERVICE,
                 UDISKS2_PATH,
                 DBUS_OBJECT_MANAGER_INTERFACE,
-                QStringLiteral("InterfacesAdded"),
+                interfacesAddedSignal,
                 this,
-                SLOT(interfacesAdded(QDBusObjectPath, InterfacePropertyMap)))) {
+                SLOT(interfacesAdded(QDBusObjectPath, UDisks2::InterfacePropertyMap)))) {
         qCWarning(lcMemoryCardLog) << "Failed to connect to interfaces added signal:" << qPrintable(systemBus.lastError().message());
     }
 
     if (!systemBus.connect(
                 UDISKS2_SERVICE,
                 UDISKS2_PATH,
                 DBUS_OBJECT_MANAGER_INTERFACE,
-                QStringLiteral("InterfacesRemoved"),
+                interfacesRemovedSignal,
                 this,
                 SLOT(interfacesRemoved(QDBusObjectPath, QStringList)))) {
         qCWarning(lcMemoryCardLog) << "Failed to connect to interfaces removed signal:" << qPrintable(systemBus.lastError().message());
@@ -202,7 +202,7 @@ void UDisks2::Monitor::format(const QString &deviceName, const QString &type, co
     doFormat(deviceName, objectPath, type, arguments);
 }
 
-void UDisks2::Monitor::interfacesAdded(const QDBusObjectPath &objectPath, const InterfacePropertyMap &interfaces)
+void UDisks2::Monitor::interfacesAdded(const QDBusObjectPath &objectPath, const UDisks2::InterfacePropertyMap &interfaces)
 {
     QString path = objectPath.path();
     qCInfo(lcMemoryCardLog) << "UDisks interface added:" << path << interfaces << externalBlockDevice(path);
@@ -561,7 +561,7 @@ void UDisks2::Monitor::createPartition(const UDisks2::Block *block)
     m_manager->add(addedPartitions);
 }
 
-void UDisks2::Monitor::createBlockDevice(const QString &path, const InterfacePropertyMap &interfacePropertyMap)
+void UDisks2::Monitor::createBlockDevice(const QString &path, const UDisks2::InterfacePropertyMap &interfacePropertyMap)
 {
     if (m_blockDevices.contains(path)) {
         return;
@@ -698,7 +698,7 @@ void UDisks2::Monitor::getBlockDevices()
             for (const QDBusObjectPath &dbusObjectPath : blockDevicePaths) {
                 QString path = dbusObjectPath.path();
                 if (externalBlockDevice(path)) {
-                    createBlockDevice(path, InterfacePropertyMap());
+                    createBlockDevice(path, UDisks2::InterfacePropertyMap());
                 }
             }
         } else if (watcher->isError()) {

diff --git a/src/udisks2monitor_p.h b/src/udisks2monitor_p.h
@@ -41,7 +41,7 @@
 
 #include "partitionmodel.h"
 #include "partitionmanager_p.h"
-#include "udisks2block_p.h"
+#include "udisks2defines.h"
 
 class PartitionManagerPrivate;
 
@@ -96,7 +96,7 @@ class Monitor : public QObject
     void formatError(Partition::Error error);
 
 private slots:
-    void interfacesAdded(const QDBusObjectPath &objectPath, const InterfacePropertyMap &interfaces);
+    void interfacesAdded(const QDBusObjectPath &objectPath, const UDisks2::InterfacePropertyMap &interfaces);
     void interfacesRemoved(const QDBusObjectPath &objectPath, const QStringList &interfaces);
 
 private:
@@ -110,7 +110,7 @@ private slots:
     void lookupPartitions(PartitionManagerPrivate::Partitions &affectedPartitions, const QStringList &objects);
 
     void createPartition(const Block *block);
-    void createBlockDevice(const QString &path, const InterfacePropertyMap &interfacePropertyMap);
+    void createBlockDevice(const QString &path, const UDisks2::InterfacePropertyMap &interfacePropertyMap);
 
     void doFormat(const QString &deviceName, const QString &dbusObjectPath, const QString &type, const QVariantHash &arguments);
     void getBlockDevices();