Review Request 116741: Fixed Bug 331934 (JJ- Jamendo API change causes the service not to work anymore) and corrects update button behaviour

[yash ladia]

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

(Updated May 8, 2014, 12:06 a.m.)


Status
------

This change has been marked as submitted.


Review request for Amarok.


Bugs: 331934
    https://bugs.kde.org/show_bug.cgi?id=331934


Repository: amarok


Description
-------

I have corrected the URL to the new one.
Also, when there was an error in downloading the relatively large (17.3 mb) file, it just used to quit silently. With my patch it still does, but the "update" button becomes enabled so that the download can be attempted again without restarting Amarok. I had to make sure that the 'm_listDownloadJob = 0;' (line 205) was called so I placed it before the return statement. Otherwise, it was giving a segfault error when Amarok was being quit.


Diffs
-----

  src/services/jamendo/JamendoService.cpp e4a7ecc 
  src/services/jamendo/JamendoXmlParser.h 6bdc67d 

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


Testing
-------

Jamendo works now. File downloads as expected.
Also, I testing the working of the new update button behavior by running the update without any internet connection. The "update" button became enabled in this case.


Thanks,

yash ladia

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20140508/9739cc91/attachment-0001.html>