Bug 10374 – need method for differentiating between fab4, squeezeplay, and controller

Bug 10374 - need method for differentiating between fab4, squeezeplay, and controller

Summary:

need method for differentiating between fab4, squeezeplay, and controller

Status:	RESOLVED DUPLICATE of bug 10470

Product:	SB Touch
Classification:	Unclassified
Component:	Setup
Version:	unspecified
Platform:	PC Other

Importance:	-- normal (vote)
Target Milestone:	---
Assigned To:	Richard Titmuss

URL:
Keywords:

Depends on:
Blocks:	10963
	Show dependency tree / graph

Reported:	2008-12-17 13:35 UTC by Ben Klaas
Modified:	2009-09-08 09:18 UTC (History)
CC List:	2 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-12-17 13:35:55 UTC

Choose Player shouldn't show up, and the fab4 "local player" needs to be selected behind the scenes automatically.

note- I think LocalPlayer.lua needs updating to support fab4 as a member of that class

Comment 1 Blackketter Dean 2008-12-17 13:48:12 UTC

And local playback defaulted on (with no way to turn off!)

Comment 2 Blackketter Dean 2008-12-17 13:48:53 UTC

And one more thing:  It should be called Squeezebox, like any other Squeezebox, with the right icon.

Comment 3 Richard Titmuss 2009-01-05 02:13:24 UTC

Local playback is now defaulted to on. The other changes are still needed.

Comment 4 Ben Klaas 2009-02-05 08:04:42 UTC

changing summary to reflect remaining issue

setup auto selects the local player during Fab4 setup. However, it calls it "squeezeplay X" rather than "squeezebox touch X" (or whatever).

player:getModel() returns squeezeplay for all squeezeplay-based players now. Need to be able to tell them apart.

Comment 5 Blackketter Dean 2009-02-05 15:07:59 UTC

I suspect we'll need to know whether a player is SqueezePlay based as well as what specific model it is.

Comment 6 Richard Titmuss 2009-02-06 04:33:06 UTC

I'm working on this today. It needed some SC changes to be in place first, and should now be a trivial change.

Comment 7 Richard Titmuss 2009-02-06 04:47:04 UTC


*** This bug has been marked as a duplicate of bug 10470 ***