Patch: Cover manager fix for 1.4.10

[Benjamin Johnson]

Please disregard my previous patch -- I have an updated version attached.   
(I completely forgot I'd mailed that last one...)  This one is a lot  
cleaner, and thoroughly removes the old Amazon code.  So far I've tested  
without problems on my own collection, although I agree about the  
necessity of cross-platform testing.

There is one part I'm not sure about, around line 302 of  
amarok/src/coverfetcher.cpp.  When using the "Fetch missing covers" button  
it will pop up a whole bunch of dialogs for the ones it can't find.  At  
any rate, the fault's entirely mine for messing with code I don't fully  
understand.

By the way, I also posted this patch to the forum a little while ago.   
Apologies for unintentional spamming; I'm a bit disorganized at the moment  
 from back-to-school craziness.

On Sat, 22 Aug 2009 14:46:40 -0400, Jeff Mitchell <mitchell at kde.org> wrote:

> We could possibly send this out to package maintainers, in case they want
> to apply it to their packages, but I'd like to get some more people
> verifying functionality first (not that I don't trust you, it's just  
> always
> good to test on multiple systems).
>
> Anyone running 1.4 that can apply this patch and let us know how it goes?
>
> Thanks,
> Jeff

-- 
Using Opera's revolutionary e-mail client: http://www.opera.com/mail/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: amarok-1.4.10.coverfetcher.diff
Type: application/octet-stream
Size: 62420 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/amarok/attachments/20090822/52116f4a/attachment.obj>