[SCM] kdeconnect packaging branch, master, updated. debian/0.9g-1-1183-g9d69498

Maximiliano Curia maxy at moszumanska.debian.org
Fri Oct 14 14:27:40 UTC 2016


Gitweb-URL: http://git.debian.org/?p=pkg-kde/kde-extras/kdeconnect.git;a=commitdiff;h=8c7c95f

The following commit has been merged in the master branch:
commit 8c7c95f98bd280a75d469a08cde0e98bffb3ed87
Author: Albert Vaca <albertvaka at gmail.com>
Date:   Sun Jun 15 21:39:23 2014 +0200

    Workaround for the bug where old notifications could not be erased
---
 plugins/notifications/notificationsdbusinterface.cpp | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/plugins/notifications/notificationsdbusinterface.cpp b/plugins/notifications/notificationsdbusinterface.cpp
index c5bdaeb..75172b2 100644
--- a/plugins/notifications/notificationsdbusinterface.cpp
+++ b/plugins/notifications/notificationsdbusinterface.cpp
@@ -137,8 +137,11 @@ void NotificationsDbusInterface::dismissRequested(Notification* notification)
     np.set<QString>("cancel", internalId);
     mPlugin->sendPackage(np);
 
-    //This should be called automatically back from server
-    //removeNotification(internalId);
+    //Workaround: we erase notifications without waiting a repsonse from the
+    //phone because we won't receive a response if we are out of sync and this
+    //notification no longer exists. Ideally, each time we reach the phone
+    //after some time disconnected we should re-sync all the notifications.
+    removeNotification(internalId);
 }
 
 QString NotificationsDbusInterface::newId()

-- 
kdeconnect packaging



More information about the pkg-kde-commits mailing list