Review Request: Last.fm scrobbling using composer (back again)

[Nicholas Wilson]

> On May 25, 2011, 7:28 p.m., Ralf Engels wrote:
> > The description of the configuration option seems to be wrong.
> > It says "use composer field" where in reality it is "using the composer field as artist" which only makes sense for classical tracks.
> > 
> > Am I missing something here?
> 
> Nicholas Wilson wrote:
>     Nope, not missing anything. That's right. The only way to use the composer field in scrobbling is to send it as the artist. That's why it was suggested that there be an option for it. Note that this is how last.fm itself stores the data: it picks album art and pictures of the composers etc based on composer as artist.
> 
> Ralf Engels wrote:
>     Ok.
>     If you agree then I will submit the patch and update the description of the configuration option.

Thanks. Maybe I'll get round to submitting the patches for the random albums navigator bugs now that the last few patches have gone through. That's pleasing—once I get the patches for all the bugs bothering me upstream I can go back to using distro packages again.


- Nicholas


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


On March 6, 2011, 12:34 a.m., Nicholas Wilson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100806/
> -----------------------------------------------------------
> 
> (Updated March 6, 2011, 12:34 a.m.)
> 
> 
> Review request for Amarok.
> 
> 
> Summary
> -------
> 
> There was a discussion a few months ago about using composer tags in last.fm scrobbling. The patch did what I wanted it to, but after being checked in it was reverted when some people complained. Fair enough. I have got round to adding a config option for this behaviour.
> 
> Feedback in particular: I am new to amarok, so might be using the config system incorrectly. It does work though.
> 
> Behaviour implemented: when config option is set, scrobble composer preferentially to artist; when not set, always use artist data (default, and same behaviour as before)
> 
> 
> This addresses bug 254580.
>     https://bugs.kde.org/show_bug.cgi?id=254580
> 
> 
> Diffs
> -----
> 
>   src/amarokconfig.kcfg 1268c33 
>   src/services/lastfm/LastFmConfigWidget.ui a6e50d3 
>   src/services/lastfm/LastFmService.h 8c30a5d 
>   src/services/lastfm/LastFmService.cpp b61a8f4 
>   src/services/lastfm/LastFmServiceConfig.h 99b46c2 
>   src/services/lastfm/LastFmServiceConfig.cpp 6ac3ff6 
>   src/services/lastfm/LastFmServiceSettings.cpp d464c89 
>   src/services/lastfm/ScrobblerAdapter.cpp 940d0c4 
> 
> Diff: http://git.reviewboard.kde.org/r/100806/diff
> 
> 
> Testing
> -------
> 
> It seems to work. Scrobbles as expected with option set and not set, and config is correctly saved between restarts of amarok when set and not set.
> 
> 
> Thanks,
> 
> Nicholas
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/amarok-devel/attachments/20110525/2f7cbf84/attachment-0001.htm