Review Request 126722: notifications: also sync icon-data from notfication hints

[Holger Kaelberer]

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

(Updated Aug. 27, 2016, noon)


Status
------

This change has been marked as submitted.


Review request for kdeconnect.


Changes
-------

Submitted with commit c122312c65ae4d2350d93e4d403200f9750926c8 by Holger Kaelberer to branch master.


Repository: kdeconnect-kde


Description
-------

- follows the priorities defined in the notifications spec version 1.2
- seems that payload can only be attached via QIODevices/files, therefore needed to create a temp file from the in-memory image data first; if there is another way - teach me pls!
- QDBusArgument-s can't be faked for testing without a real DBUS-call because they can be created either only for reading or for writing. Therefore added the virtual parseImageData function to use another datatype in the unit-tests


Diffs
-----

  plugins/sendnotifications/notificationslistener.h 0612c34 
  plugins/sendnotifications/notificationslistener.cpp 4da1770 
  tests/testnotificationlistener.cpp ad36dac 

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


Testing
-------

sure


Thanks,

Holger Kaelberer

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20160827/8b39e2d7/attachment.html>