Bug 10123 – playlist_cur_index sometimes coming in from SC as a string instead of number

Bug 10123 - playlist_cur_index sometimes coming in from SC as a string instead of number

Summary:

playlist_cur_index sometimes coming in from SC as a string instead of number

Status:	CLOSED FIXED

Product:	SB Controller
Classification:	Unclassified
Component:	SB Server
Version:	unspecified
Platform:	PC Other

Importance:	-- normal (vote)
Target Milestone:	7.3
Assigned To:	Ben Klaas

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2008-11-24 13:59 UTC by Ben Klaas
Modified:	2008-12-15 12:10 UTC (History)
CC List:	3 users (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 Ben Klaas 2008-11-24 13:59:43 UTC

I was seeing a situation where sometimes the playlist_cur_index variable from the playerstatus update was coming in from SC as a string instead of a number. This would cause NowPlaying to flop:

154853:11408 ERROR (RequestHttp.lua:67) - Response sink:...zeplay/share/applets/NowPlaying/NowPlayingApplet.lua:379: attempt to compare number with string

Restarted SC and the problem went away, but I'm going to put a tonumber() in NowPlaying._updatePlaylist to make sure.

Comment 1 Ben Klaas 2008-11-24 14:01:54 UTC

fixed, r3434

QA, this is a very low risk fix. To test make sure that the "X of Y" in the upper right of the Now Playing window is always correct.

Comment 2 James Richardson 2008-12-15 12:10:05 UTC

This bug has been fixed in the 7.3.0 release version of SqueezeCenter!

Please download the new version from http://www.slimdevices.com/su_downloads.html if you haven't already.  

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.