[SCM] kdeconnect packaging branch, master, updated. debian/0.9g-1-1183-g9d69498
Maximiliano Curia
maxy at moszumanska.debian.org
Fri Oct 14 14:28:37 UTC 2016
Gitweb-URL: http://git.debian.org/?p=pkg-kde/kde-extras/kdeconnect.git;a=commitdiff;h=e6402f7
The following commit has been merged in the master branch:
commit e6402f7e97796a8a17a45bc6e3b85799330005a0
Author: Albert Vaca <albertvaka at gmail.com>
Date: Sun Jun 14 18:59:53 2015 -0700
We were not clearing notifications when a device disconnected
---
plugins/notifications/notificationsdbusinterface.cpp | 5 +++++
plugins/notifications/notificationsdbusinterface.h | 1 +
plugins/notifications/notificationsplugin.cpp | 3 +--
3 files changed, 7 insertions(+), 2 deletions(-)
diff --git a/plugins/notifications/notificationsdbusinterface.cpp b/plugins/notifications/notificationsdbusinterface.cpp
index 26323ae..66d6790 100644
--- a/plugins/notifications/notificationsdbusinterface.cpp
+++ b/plugins/notifications/notificationsdbusinterface.cpp
@@ -46,6 +46,11 @@ NotificationsDbusInterface::NotificationsDbusInterface(KdeConnectPlugin* plugin)
NotificationsDbusInterface::~NotificationsDbusInterface()
{
+ clearNotifications();
+}
+
+void NotificationsDbusInterface::clearNotifications()
+{
qDeleteAll(mNotifications);
}
diff --git a/plugins/notifications/notificationsdbusinterface.h b/plugins/notifications/notificationsdbusinterface.h
index 5c1244e..e8d5c2a 100644
--- a/plugins/notifications/notificationsdbusinterface.h
+++ b/plugins/notifications/notificationsdbusinterface.h
@@ -43,6 +43,7 @@ public:
virtual ~NotificationsDbusInterface();
void processPackage(const NetworkPackage& np);
+ void clearNotifications();
public Q_SLOTS:
Q_SCRIPTABLE QStringList activeNotifications();
diff --git a/plugins/notifications/notificationsplugin.cpp b/plugins/notifications/notificationsplugin.cpp
index 568a052..b0d089c 100644
--- a/plugins/notifications/notificationsplugin.cpp
+++ b/plugins/notifications/notificationsplugin.cpp
@@ -49,9 +49,8 @@ NotificationsPlugin::~NotificationsPlugin()
// is that disabling the plugin does not remove the interface (that will
// return outdated values) and that enabling it again instantiates a second
// adaptor.
-
//notificationsDbusInterface->deleteLater();
-
+ notificationsDbusInterface->clearNotifications();
}
bool NotificationsPlugin::receivePackage(const NetworkPackage& np)
--
kdeconnect packaging
More information about the pkg-kde-commits
mailing list