Review Request: Fix Permission denied when playing a track from a Rhythmbox DAAP collection

[Daniel Stöckel]

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

Review request for Amarok.


Description
-------

This is a followup to https://git.reviewboard.kde.org/r/106366/

Rhythmbox seems to require that the session-id field is present in a track get request.
This patch adds an additional parameter to the DaapTrack class constructor and appends the session id to the Track URL.


Diffs
-----

  src/core-impl/collections/daap/DaapMeta.h 5278b57 
  src/core-impl/collections/daap/DaapMeta.cpp a2429d7 
  src/core-impl/collections/daap/daapreader/Reader.cpp b6196e1 

Diff: http://git.reviewboard.kde.org/r/106370/diff/


Testing
-------

- Tracks published by Rhythmbox now play correctly
- Daap collections published by mt-daap still work


Thanks,

Daniel Stöckel

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20120907/10821182/attachment.html>