Version: 2.4.3 (using KDE 4.6.5) OS: Linux When the new setting "Use composer instead of performer for scrobbling" is on, amaroK should still scrobble performer when composer tag is empty or absent. But that doesn't happen, amarok happily scrobbles empty composer, liblastfm turns it into [unknown] and last.fm dropps the scrobble on the floor. Annoying bug - if you checked the option accidentally, scrobbling would be silently not working and troubleshooting isn't easy. Reproducible: Always Steps to Reproduce: Turn on the checkbox. Listen to some track. Actual Results: last.fm wouldn't show that you listen to anything The post body would be like s=xxxxx&a=%5Bunknown%5D&t=%D0%A1%D0%BE%D0%B1%D0%B0%D1%87%D0%B8%D0%B9%20%D0%B0%D0%BD%D0%B3%D0%B5%D0%BB&b=%D0%A1%D1%82%D1%80%D0%B0%D0%B6%D0%B8%20%D0%BF%D0%BE%D1%81%D0%BB%D0%B5%D0%B4%D0%BD%D0%B5%D0%B3%D0%BE%20%D1%80%D1%83%D0%B1%D0%B5%D0%B6%D0%B0&l=159&n=0&m= Expected Results: it scrobbles.
This is really a corner case. I listen to classical music a lot, but don't use the composer scrobbling, as it usually doesn't give coherent results in last.fm, their datsbase is not really made for classical music.
The problem is clicking the option accidentallly. You'll spend a week never knowing what hit you.
Created attachment 66122 [details] This patch fixes the issue and also refactors a bit This patch fixes the issue by checking whether the composer is empty and falling back to performer. I've also refactored out repetitive track-populating code into a method.
Thank you for the patch. Please submit it for review to http://reviewboard.kde.org. You will need to get an identity from http://identity.kde.org first.
https://git.reviewboard.kde.org/r/103279/
Git commit f9feaf51acfca9eb9994123ffb1929c65d99aa23 by Sam Lade. Committed on 28/11/2011 at 03:40. Pushed by lade into branch 'master'. Fix last.fm scrobble composer fallback to artist Now correctly scrobbles with the artist name if the composer field is empty and the scrobble composer option is enabled, rather than silently failing. Patch by Ilya Kasnacheev. BUG:286215 FIXED-IN:2.5 REVIEW:103279 M +1 -0 ChangeLog M +27 -25 src/services/lastfm/ScrobblerAdapter.cpp M +2 -1 src/services/lastfm/ScrobblerAdapter.h http://commits.kde.org/amarok/f9feaf51acfca9eb9994123ffb1929c65d99aa23