[Kde-hardware-devel] Review Request 121693: Wait for notification service before emitting them

[Kai Uwe Broulik]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121693/
-----------------------------------------------------------

Review request for Solid.


Repository: powerdevil


Description
-------

This patch makes PowerDevil postpone its notifications (battery broken, battery low) until the notification service has been registered to avoid ugly oldschool notifications from appearing ontop of KSplash.

The notifications are emitted when:
- The notification service is already registered (kded restart)
- The service becomes available (plasma startup)
- After 30 seconds (to ensure the notifications are shown)

This makes the "your battery is broken" and "battery low" notifications appear right after Plasma is started on login whereas the former previously had a 30s hardcoded delay whereas the latter appeared right on ksplash


Diffs
-----

  daemon/powerdevilcore.h a27e2ba 
  daemon/powerdevilcore.cpp 05ee2e6 

Diff: https://git.reviewboard.kde.org/r/121693/diff/


Testing
-------

Logged in and out, notifications fire. Restarted kded while plasma was running (emit right away), while it was not running (30s timeout), and started plasma shortly after kded (emit once service is there). Works fine, looks much sleeker now, and notifications also only seem to be fired once regardless of when they have been for the first time.


Thanks,

Kai Uwe Broulik

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-hardware-devel/attachments/20141227/6efdf72b/attachment.html>