From 8c7c95f98bd280a75d469a08cde0e98bffb3ed87 Mon Sep 17 00:00:00 2001 From: Albert Vaca Date: Sun, 15 Jun 2014 21:39:23 +0200 Subject: [PATCH] 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 c5bdaeb74..75172b20f 100644 --- a/plugins/notifications/notificationsdbusinterface.cpp +++ b/plugins/notifications/notificationsdbusinterface.cpp @@ -137,8 +137,11 @@ void NotificationsDbusInterface::dismissRequested(Notification* notification) np.set("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()