<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/105722/">http://git.reviewboard.kde.org/r/105722/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Amarok, Bart Cerneels and Myriam Schweingruber.</div>
<div>By Matěj Laitl.</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">MultiTrack, EngineController: Fix MultiTrack API
In previous API, the only way to get next track url is
multiTrack->next(), which also advances the current index.
EngineController doesn't work this way, it needs to know next url in
the slotAboutToFinish(), but only should only advance the index in
slotNewTrackPlaying() (because aboutToFinish() may be even emitted
multiple times etc).
Correct the api so that the only non-const method is
setSource( int index ). Also change m_multiSource handling in
EngineController so that it is created and destroyed in
slotNewTrackPlaying(), which is a correct place to do that.
Also contains clean-ups to the MultiTrack classes.
This change fixes a bug for MultiTrack streams where metadata wouldn't
be updated because EngineController thinks different source is playing.
Therefore I think it fixes bug 300189. Reporters, please reopen the bug
if this patch doesn't fix the bug and provide full `amarok --debug`
output in that case.
BUG: 300189
FIXED-IN: 2.7
Rewrite Last.fm ScrobblerAdapter
* use new EngineController signals, don't track streams on its own
* better error handling, use liblastfm 1.0 features
* add delay before calling nowPlaying
* call removeNowPlaying when it's appropriate
* send much more detailed track info to Last.fm
* Last.fm notifies us about suggested tag corrections; relay these to
the user. (currently Components::logger()->longMessage(), will need
something more persistent to be useful. Or Components::logger() must
log into a file as A 1.4 did
* prepare for split into ScrobblingService/Controller
EngineController: better docs for trackMetadataChanged() signal
ScrobblerAdapter, MainWindow: make banTrack() cleaner using loveTrack() example
EngineController: emit trackFinishedPlaying() in the middle of streams, too
This will be needed for cleaner Last.fm scrobbling. Also ensure that we
update length of just played track so that Last.fm knows whether it can
scrobble.
StatSyncing: ChooseProvidersPage.ui fixuifiles
ScrobblerAdapter: remove unused methods</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="https://bugs.kde.org/show_bug.cgi?id=300189">300189</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>src/EngineController.h <span style="color: grey">(4851e2830977787521a041a74f7874bb92e0e647)</span></li>
<li>src/EngineController.cpp <span style="color: grey">(4f565141ecfe41af7fc66083981dea347ffc5ef5)</span></li>
<li>src/MainWindow.h <span style="color: grey">(d49776476ce8b23013c1a8880a640d9aef35e817)</span></li>
<li>src/MainWindow.cpp <span style="color: grey">(7297559a363377f9b91129fb89b19895beaa02bc)</span></li>
<li>src/core-impl/capabilities/multisource/MultiSourceCapabilityImpl.h <span style="color: grey">(4a472c2072193f809b2c9b6dd06f7caa6df2b991)</span></li>
<li>src/core-impl/capabilities/multisource/MultiSourceCapabilityImpl.cpp <span style="color: grey">(28152e61c93ce3cdd2d5a26f8adf33c87571e245)</span></li>
<li>src/core-impl/meta/multi/MultiTrack.h <span style="color: grey">(63a8651628eb6335cd0a3e635150bfae95e83235)</span></li>
<li>src/core-impl/meta/multi/MultiTrack.cpp <span style="color: grey">(e270d0a211dc1dd61575918c62279649e4e84f27)</span></li>
<li>src/core-impl/meta/stream/Stream.cpp <span style="color: grey">(b0fcfad5808b9ae428cb6612bec1737b3af82d3b)</span></li>
<li>src/core/capabilities/MultiSourceCapability.h <span style="color: grey">(819a24ff79f8d741db30c34ed42a38610885bf4c)</span></li>
<li>src/core/capabilities/MultiSourceCapability.cpp <span style="color: grey">(4c46ff45f7f55c3460704f7058c9323a947529cc)</span></li>
<li>src/services/lastfm/ScrobblerAdapter.h <span style="color: grey">(613ad9973b0d768a0345acbe801a72e2652c9c90)</span></li>
<li>src/services/lastfm/ScrobblerAdapter.cpp <span style="color: grey">(fa8635bb8c5492a2a434f6f947afa5f215124fee)</span></li>
<li>src/statsyncing/ui/ChooseProvidersPage.ui <span style="color: grey">(PRE-CREATION)</span></li>
<li>tests/core-impl/meta/multi/TestMetaMultiTrack.h <span style="color: grey">(61e37627351163a7f2bbae729779900d2abb6ca9)</span></li>
<li>tests/core-impl/meta/multi/TestMetaMultiTrack.cpp <span style="color: grey">(b9d64dee9f8f0880981550aab29b882c4b4bbc31)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/105722/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>