Bug 3182 – Random Mix plugin will not show 2 upcoming songs

Bug 3182 - Random Mix plugin will not show 2 upcoming songs

Summary:

Random Mix plugin will not show 2 upcoming songs

Status:	RESOLVED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Plugin
Version:	6.2.2
Platform:	PC Windows XP

Importance:	P5 trivial (vote)
Target Milestone:	---
Assigned To:	Blackketter Dean

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2006-03-18 17:44 UTC by Mark Lanctot
Modified:	2006-04-22 21:50 UTC (History)
CC List:	1 user (show)

See Also:
Category:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Mark Lanctot 2006-03-18 17:44:51 UTC

When I select "Now Playing will show 2 upcoming songs" I only get one upcoming song on the playlist.

Any other number seems to work fine.

Comment 1 Max Spicer 2006-03-19 13:09:41 UTC

I don't think this is right.  If you ask for 2 songs, you'll get the current one and the next one.  If you ask for three you'll get the current one and the next 2.  etc.  You seem to be saying that if you asked for three you would get the current song and then THREE after it.  This shouldn't be the case.  Please post here to confirm the behaviour.

Basically, the code is doing what it's meant to be doing - upcoming songs includes the current one.  I admit that the wording is therefore confusing or just plain incorrect.  I'm open to suggestions about whether to fix the wording or the behaviour!

Comment 2 Mark Lanctot 2006-03-19 14:59:01 UTC

How I interpreted it was that showing 2 upcoming songs should show...2 upcoming songs.

i.e. This shouldn't include the current song.

If it's just a wording/interpretation problem I'm OK with leaving it fixed.  Thanks.