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

[Ralf Engels]

> 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.

Ok.
If you agree then I will submit the patch and update the description of the configuration option.


- Ralf


-----------------------------------------------------------
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/8a598d9c/attachment.htm