Fixed Dolphin's extension

It was not working because it used the model (that is now async) instead
of dbus interfaces.

CCMAIL: aleixpol@kde.org
This commit is contained in:
Albert Vaca 2015-03-21 22:50:00 -07:00
parent e20c3c907d
commit 4477ff29d9

View file

@ -19,23 +19,21 @@
*/ */
#include "sendfileitemaction.h" #include "sendfileitemaction.h"
#include <interfaces/devicesmodel.h>
#include <interfaces/dbusinterfaces.h>
#include <QList> #include <QList>
#include <QMenu> #include <QMenu>
#include <QAction> #include <QAction>
#include <QWidget> #include <QWidget>
#include <QMessageBox>
#include <QVariantList> #include <QVariantList>
#include <QUrl>
#include <QIcon> #include <QIcon>
#include <KPluginFactory> #include <KPluginFactory>
#include <KPluginLoader> #include <KPluginLoader>
#include <KProcess>
#include <KLocalizedString> #include <KLocalizedString>
#include <QUrl>
#include <interfaces/devicesmodel.h>
#include <interfaces/dbusinterfaces.h>
K_PLUGIN_FACTORY(SendFileItemActionFactory, registerPlugin<SendFileItemAction>();) K_PLUGIN_FACTORY(SendFileItemActionFactory, registerPlugin<SendFileItemAction>();)
@ -47,22 +45,31 @@ SendFileItemAction::SendFileItemAction(QObject* parent, const QVariantList& ): K
QList<QAction*> SendFileItemAction::actions(const KFileItemListProperties& fileItemInfos, QWidget* parentWidget) QList<QAction*> SendFileItemAction::actions(const KFileItemListProperties& fileItemInfos, QWidget* parentWidget)
{ {
DevicesModel m;
QList<QAction*> actions; QList<QAction*> actions;
for(int i = 0; i<m.rowCount(); ++i) { DaemonDbusInterface iface;
QModelIndex idx = m.index(i); if (!iface.isValid()) {
DeviceDbusInterface* dev = m.getDevice(idx); return actions;
if(dev->isReachable() && dev->isPaired()) { }
QAction* action = new QAction(QIcon::fromTheme(dev->iconName()), dev->name(), parentWidget);
action->setProperty("id", idx.data(DevicesModel::IdModelRole)); QDBusPendingReply<QStringList> reply = iface.devices(true, true);
reply.waitForFinished();
const QStringList devices = reply.value();
foreach (const QString& id, devices) {
DeviceDbusInterface deviceIface(id);
if (!deviceIface.isValid()) {
continue;
}
if (!deviceIface.hasPlugin("kdeconnect_share")) {
continue;
}
QAction* action = new QAction(QIcon::fromTheme(deviceIface.iconName()), deviceIface.name(), parentWidget);
action->setProperty("id", id);
action->setProperty("urls", QVariant::fromValue(fileItemInfos.urlList())); action->setProperty("urls", QVariant::fromValue(fileItemInfos.urlList()));
action->setProperty("parentWidget", QVariant::fromValue(parentWidget)); action->setProperty("parentWidget", QVariant::fromValue(parentWidget));
connect(action, SIGNAL(triggered(bool)), this, SLOT(sendFile())); connect(action, SIGNAL(triggered(bool)), this, SLOT(sendFile()));
actions += action; actions += action;
} }
}
if (actions.count() > 1) { if (actions.count() > 1) {
QAction *menuAction = new QAction(QIcon::fromTheme("preferences-system-network"), i18n("Send via KDE Connect"), parentWidget); QAction *menuAction = new QAction(QIcon::fromTheme("preferences-system-network"), i18n("Send via KDE Connect"), parentWidget);
@ -81,10 +88,10 @@ QList<QAction*> SendFileItemAction::actions(const KFileItemListProperties& fileI
void SendFileItemAction::sendFile() void SendFileItemAction::sendFile()
{ {
QList<QUrl> urls = sender()->property("urls").value<QList<QUrl>>(); QList<QUrl> urls = sender()->property("urls").value<QList<QUrl>>();
QString id = sender()->property("id").toString();
foreach(const QUrl& url, urls) { foreach(const QUrl& url, urls) {
QDBusMessage msg = QDBusMessage::createMethodCall("org.kde.kdeconnect", "/modules/kdeconnect/devices/"+sender()->property("id").toString()+"/share", "org.kde.kdeconnect.device.share", "shareUrl"); QDBusMessage msg = QDBusMessage::createMethodCall("org.kde.kdeconnect", "/modules/kdeconnect/devices/"+id+"/share", "org.kde.kdeconnect.device.share", "shareUrl");
msg.setArguments(QVariantList() << url.toString()); msg.setArguments(QVariantList() << url.toString());
QDBusConnection::sessionBus().call(msg); QDBusConnection::sessionBus().call(msg);
} }
} }