Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Playlists
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Daily build (which?)
  • Due in Version Undecided
  • Due Date Undecided
  • Votes 1
  • Private
Attached to Project: Rockbox
Opened by Nephiel - 2013-07-28
Last edited by speachy - 2019-07-24

FS#12887 - Fix for wrong playlist order after moving tracks before current

When moving a track within a playlist to any position before the currently playing track, the index of the currently playing track on the playlist gets shifted back by 1. This causes two problems:

-The playlist shows a wrong track as the current track being played
-The currently playing track is played once more when finished

The only exception is when moving to the first position in the playlist, since that is handled as a prepend and works as expected.

AFAIK this issue affects all players and has been present at least since 3.6 (see #FS11376 http://www.rockbox.org/tracker/task/11376)

Attached is a patch against 49bcf35 that (hopefully) fixes this issue by comparing the display indexes of the positions we’re moving from/to and the currently playing track, incrementing the playlist index when neccessary.

Closed by  speachy
2019-07-24 18:25
Reason for closing:  Accepted
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

Better late than never, this has finally been committed as 7bef453e.

Moving this to gerrit: http://gerrit.rockbox.org/r/#/c/530/

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing