Bug 16062 – Settings page in Classic skin broken

Bug 16062 - Settings page in Classic skin broken

Summary:

Settings page in Classic skin broken

Status:	RESOLVED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Web Interface
Version:	7.5.x
Platform:	PC Other

Importance:	P2 normal (vote)
Target Milestone:	7.5.1
Assigned To:	Michael Herger

URL:	http://forums.slimdevices.com/showthr...
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2010-04-13 23:58 UTC by Erland Isaksson
Modified:	2010-04-14 20:25 UTC (History)
CC List:	1 user (show)

See Also:
Category:	Bug

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 Erland Isaksson 2010-04-13 23:58:55 UTC

After revision 30485 it no longer works to select the different setting sections in the Classic skin in the web interface. It works in 30484 but in 30485 I just get an our glass when I select something else than "Basic Settings" which is shown by default.

Comment 1 Michael Herger 2010-04-14 00:06:43 UTC

yep, passing playerid to settings.js seems to be the problem

Comment 2 SVN Bot 2010-04-14 00:32:01 UTC

 == Auto-comment from SVN commit #30576 to the slim repo by mherger ==
 == http://svn.slimdevices.com/slim?view=revision&revision=30576 ==

Fixed Bug: 16062
Description: use needsClient to decide whether we want client settings or not. playerid is now set for all settings pages, not only player relevant pages.

Comment 3 SVN Bot 2010-04-14 20:25:59 UTC

 == Auto-comment from SVN commit #30609 to the slim repo by kdf ==
 == http://svn.slimdevices.com/slim?view=revision&revision=30609 ==

Bug: 16062
Description: backport of a fix in 7.6 to keep the save settings button on all settings pages.