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

[Commit Hook]

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


This review has been submitted with commit 01b3ab9fd70b158bfc6f4529ce22a671c5d2820c by Daniel Meltzer to branch master.

- Commit Hook


On March 11, 2014, 8:45 p.m., yash ladia wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/116741/
> -----------------------------------------------------------
> 
> (Updated March 11, 2014, 8:45 p.m.)
> 
> 
> 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/bd4b0860/attachment.html>