Bug 8338 – If player's name exeeds char limit, the rest of chars looks weired on SC.

Bug 8338 - If player's name exeeds char limit, the rest of chars looks weired on SC.

Summary:

If player's name exeeds char limit, the rest of chars looks weired on SC.

Status:	CLOSED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Web Interface
Version:	7.2
Platform:	PC Windows (legacy)

Importance:	-- minor (vote)
Target Milestone:	7.x
Assigned To:	Michael Herger

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2008-06-05 12:02 UTC by Wallace Lai
Modified:	2009-07-31 10:22 UTC (History)
CC List:	1 user (show)

See Also:
Category:	---

Attachments
limit maxlength of pref_playername to 32 (570 bytes, patch) 2008-06-21 13:49 UTC, Ben Klaas	Details \| Diff
View All 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 Wallace Lai 2008-06-05 12:02:34 UTC

SqueezeCenter Version: 7.2 - 20005 @ Thu May 22 00:51:54 PDT 2008 - Windows 2000 - EN - cp1252
Server IP address:
Perl Version: 5.8.8 MSWin32-x86-multi-thread
MySQL Version: 5.0.22-community-nt 

Steps to Reproduce:
1.  Start SqueezeCenter.
2.  Go to classic skin.
3.  Make sure a SqueezeBox or other player is connected to SC.
4.  Go to the Player Settings to make a really long name for the player.
5.  Notice the characters after number 33 become weired.  SC accepts the name without warning, it is not shown properly.
6.  Example: SqueezeBoxWithMACAddress00:04:20:¬¼n

Suggestion:  The text field should just stop accepting chars after certain length.

Note:  Already showed this bug to Michael Herger.

Comment 1 Chris Owens 2008-06-20 17:00:19 UTC

This should probably be assigned to Ben, I think.

Comment 2 Ben Klaas 2008-06-21 13:49:39 UTC

Created attachment 3470 [details]
limit maxlength of pref_playername to 32

This is how I'd fix it, but this is really Michael's territory

Comment 3 Michael Herger 2008-06-22 23:34:32 UTC

thanks Ben - applied in 21030 (and don't worry about me being touchy if you want to fix this kind of bug :-))

Comment 4 James Richardson 2008-12-15 12:04:56 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.

Comment 5 Chris Owens 2009-07-31 10:22:11 UTC

Reduce number of active targets for SC