Bug 6206 – Default skin Settings link messed up when no player attached

Bug 6206 - Default skin Settings link messed up when no player attached

Summary:

Default skin Settings link messed up when no player attached

Status:	RESOLVED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Web Interface
Version:	7.0
Platform:	PC Windows XP

Importance:	P2 normal (vote)
Target Milestone:	---
Assigned To:	Unassigned bug - please assign me!

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2007-11-20 22:44 UTC by Ceejay
Modified:	2007-11-21 00:15 UTC (History)
CC List:	0 users

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 Ceejay 2007-11-20 22:44:20 UTC

Using 7.0 nightly from Nov 20 on Win XP SP2, Firefox

With no player attached, the behaviour of the "Settings" button breaks.

Start up SC7 with no player attached. You can click "Settings" and it will work but the URL will be something like http://127.0.0.1:9000/settings/index.html?player=&player=&player=

Move around SC7 for a bit, then click "Settings" again. You will find many more instances of &player= added to the string. At least once I found the URL getting so long that it became invalid and wouldn't work at all.

Closing and restarting the browser window seems to be enough to reset the string.

Comment 1 Michael Herger 2007-11-21 00:15:27 UTC

change 14890 - don't even try to fix the url unless a player is defined