Funkwhale
/
Clementine-audio-player-Mac-Linux-Windows
mirror of https://github.com/clementine-player/Clementine
1
0
Fork 0
Code Issues Releases Wiki Activity

player: Emit request URL in SongChangeRequestProcessed 

Use the request URL rather than the media URL when emitting the
SongChangeRequestProcessed signal. The request URL is the URL that the
PlaylistManager, which is the only consumer of this signal, knows about.
However, in the current cases, the url is only checked by the playlist
when the two URLs should be the same.
This commit is contained in:
Jim Broadus 2021-03-11 23:26:11 -08:00 committed by John Maguire
parent bde6d9cc5b
commit dfb953a78f
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/core/player.cpp
View File

@ -670,12 +670,12 @@ void Player::TrackAboutToEnd() {
void Player::IntroPointReached() { NextInternal(Engine::Intro); }
void Player::ValidMediaRequested(const MediaPlaybackRequest& req) {
emit SongChangeRequestProcessed(req.url_, true);
emit SongChangeRequestProcessed(req.RequestUrl(), true);
}
void Player::InvalidMediaRequested(const MediaPlaybackRequest& req) {
// first send the notification to others...
emit SongChangeRequestProcessed(req.url_, false);
emit SongChangeRequestProcessed(req.RequestUrl(), false);
// ... and now when our listeners have completed their processing of the
// current item we can change the current item by skipping to the next song