Bug 471456 - Modifying playlist position of current song breaks pause
Summary: Modifying playlist position of current song breaks pause
Status: RESOLVED FIXED
Alias: None
Product: Elisa
Classification: Applications
Component: general (show other bugs)
Version: 23.04.2
Platform: Fedora RPMs Linux
: NOR normal
Target Milestone: ---
Assignee: Matthieu Gallien
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-06-26 05:16 UTC by Quinten Kock
Modified: 2023-09-18 16:51 UTC (History)
1 user (show)

See Also:
Latest Commit: https://invent.kde.org/multimedia/elisa/-/commit/0e6fdd58908fd86d924ce040902be4635e90545a
Version Fixed In:
Sentry Crash Report:

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Quinten Kock 2023-06-26 05:16:53 UTC 
SUMMARY
When the current song is moved in the playlist (either by moving another song over it, or by moving the song itself), the pause button stops playback, instead of pausing it.

STEPS TO REPRODUCE
1. Play a song in a playlist
2. Move the playing song, or move another song 'over' the playing song, so that its index in the playlist changes
3. Pause

OBSERVED RESULT
Playback stops

EXPECTED RESULT
Playback pauses (Remembers current song and position in song)

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: Fedora Linux 38
(available in About System)
KDE Plasma Version: 5.27.5
KDE Frameworks Version: 5.107.0
Qt Version: 5.15.10

ADDITIONAL INFORMATION
Comment 1 Nate Graham 2023-09-14 20:37:05 UTC 
Can reproduce.
Comment 2 Bug Janitor Service 2023-09-15 16:42:40 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/multimedia/elisa/-/merge_requests/488
Comment 3 Nate Graham 2023-09-18 16:51:47 UTC 
Git commit 0e6fdd58908fd86d924ce040902be4635e90545a by Nate Graham, on behalf of Jack Hill.
Committed on 18/09/2023 at 18:46.
Pushed by ngraham into branch 'master'.

mediaplaylistproxymodel: update playlist indices when moving the current track

Update previous/next track
Update mCurrentPlayListPosition
Emit currentTrackRowChanged
Related: bug 471429, bug 472284

M  +14   -2    src/mediaplaylistproxymodel.cpp
M  +2    -0    src/mediaplaylistproxymodel.h

https://invent.kde.org/multimedia/elisa/-/commit/0e6fdd58908fd86d924ce040902be4635e90545a