After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 131150 - Deleting playing song should go to next song
Deleting playing song should go to next song
Status: RESOLVED FIXED
Product: rhythmbox
Classification: Other
Component: general
0.6.4
Other Linux
: Normal normal
: ---
Assigned To: RhythmBox Maintainers
RhythmBox Maintainers
Depends on:
Blocks:
 
 
Reported: 2004-01-11 11:49 UTC by Sebastien Bacher
Modified: 2006-02-19 03:16 UTC
See Also:
GNOME target: ---
GNOME version: 2.3/2.4


Attachments
patch (2.14 KB, patch)
2006-01-25 12:15 UTC, James "Doc" Livingston
reviewed Details | Review
fixed patch (2.39 KB, patch)
2006-01-30 10:38 UTC, James "Doc" Livingston
committed Details | Review

Description Sebastien Bacher 2004-01-11 11:49:14 UTC
This bug was originally reported in the Debian BTS:
http://bugs.debian.org/205976

"When the currently playing song is deleted, rhythmbox should go onto the
next song."
Comment 1 James "Doc" Livingston 2006-01-25 12:15:21 UTC
Created attachment 58079 [details] [review]
patch

This patch fixed this by queuing a Next operation when the playing song is deleted, rather than stopping.
Comment 2 Alex Lancaster 2006-01-28 05:20:33 UTC
With this patch it appears that upon a "remove" action, it always skips to the song at the top of the current list of songs in the library song list.  This is OK only if you are deleting the first song in the place, it should skip to the song immediately below.
Comment 3 Alex Lancaster 2006-01-28 05:22:39 UTC
(In reply to comment #2)
> only if you are deleting the first song in the place, it should skip to the
> song immediately below.

Oops, should read: 

"only if you are deleting the first song in the list, otherwise it should skip to the song immediately below."

Comment 4 James "Doc" Livingston 2006-01-30 10:38:26 UTC
Created attachment 58382 [details] [review]
fixed patch

This now chooses the entry to play correctly.
Comment 5 Alex Lancaster 2006-01-30 11:01:49 UTC
Works for me.
Comment 6 Alex Lancaster 2006-02-17 14:16:29 UTC
Has this patch been committed yet?  Switching patch status to "none".
Comment 7 James "Doc" Livingston 2006-02-18 05:39:33 UTC
Thanks for reminding me, I'd forgotten. Committed to cvs.
Comment 8 Alex Lancaster 2006-02-19 03:16:32 UTC
*** Bug 331732 has been marked as a duplicate of this bug. ***