Bug 7128 - SqueezeCenter asks users to update Ray's FW by holding brightness button.
: SqueezeCenter asks users to update Ray's FW by holding brightness button.
Status: CLOSED WORKSFORME
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 7.0
: PC Windows (legacy)
: -- normal (vote)
: 7.x
Assigned To: Wallace Lai
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-02-13 14:49 UTC by Wallace Lai
Modified: 2009-09-08 09:21 UTC (History)
0 users

See Also:
Category: ---


Attachments
ScreenShot. (384.45 KB, image/bmp)
2008-02-13 14:50 UTC, Wallace Lai
Details
removing the "press and hold" string (4.85 KB, patch)
2008-02-14 00:54 UTC, Michael Herger
Details | Diff
This is in default skin. (308.80 KB, image/bmp)
2008-02-14 13:31 UTC, Wallace Lai
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Wallace Lai 2008-02-13 14:49:28 UTC
The playlist frame on SqueezeCenter asks users to update the SqueezeBox firmware by holding holding down the brightness button.  This message needs to have info for Rays also.  The player in this case is a Ray.  Users with Jive and Rays only will be confused.
Comment 1 Wallace Lai 2008-02-13 14:50:07 UTC
Created attachment 2875 [details]
ScreenShot.
Comment 2 Wallace Lai 2008-02-13 14:51:43 UTC
This may not be a stop ship bug.
Comment 3 Blackketter Dean 2008-02-13 15:49:27 UTC
Wallace, how long is this up on the web UI?

Also, does it appear in the new default skin or only in the classic skin.

(finally, if you can, use PNG or JPEG for your attachments, BMP aren't compressed)
Comment 4 Wallace Lai 2008-02-13 16:19:05 UTC
This message stayed on the Web UI for about 3 minutes.  Then the UI went back to the playlist.
It is in classic skin.
Comment 5 Blackketter Dean 2008-02-13 16:48:53 UTC
Wallace: can you reproduce this in the Default skin?
Comment 6 Blackketter Dean 2008-02-13 17:08:24 UTC
The correct fix is to have a unique string for players that are in the process of being updated.

"Updating Squeezebox firmware.  This may take a minute."

How about:

	DE	Squeezebox-Firmware wird aktualisiert. Neustart wird durchgeführt.
	EN	Updating Squeezebox firmware. This might take a minute.
	ES	Actualizando firmware de Squeezebox. Puede tardar un minuto.
	FR	Mise à jour du micrologiciel Squeezebox. Cela peut durer une minute.
	IT	Aggiornamento firmware Squeezebox. L'operazione può richiedere un minuto.
	NL	Squeezebox-firmware wordt bijgewerkt. Dit kan even duren.

Comment 7 Michael Herger 2008-02-14 00:54:12 UTC
Created attachment 2878 [details]
removing the "press and hold" string

The quick fix. This might be ok if players auto-update by default. Do they?

A better fix would be having per-playertype strings (we already have short versions of these).
Comment 8 Blackketter Dean 2008-02-14 07:29:01 UTC
Is it possible to display the new string if a firmware update has been sent? 

 I don't like the idea of removing the old, working string.
Comment 9 Michael Herger 2008-02-14 07:36:01 UTC
We don't remove the working string. We just remove the part about pressing BRIGHTNESS on the remote to get the update done, because it doesn't apply to SBR.
Comment 10 Blackketter Dean 2008-02-14 07:49:53 UTC
Right.  I like the current string for TR and SB1/2/3.  What I want is for a new string to be displayed after the update process has begun, for all devices.  Since SBR always auto-updates, the first string would never be seen.

Comment 11 Wallace Lai 2008-02-14 09:00:03 UTC
So far, this issue is not showing up with default skin.  In fact, this string has not shown itself at all again since it was replaced by the playlist.  I will keep trying.
Comment 12 Wallace Lai 2008-02-14 09:04:00 UTC
By the way, I wonder if it would be best to just append a string like "if you have a duet, do this and this to upgrade your firmware".  If possible, it may be a good idea to avoid any code change at this stage.
Comment 13 James Richardson 2008-02-14 09:46:44 UTC
Please retest this bug
Comment 14 Wallace Lai 2008-02-14 11:39:01 UTC
After many attempts, this was finally reproduced on both classic and default skin.
At this stage, it may be a good append a string to the existing one to handle the duet situation.
Comment 15 Wallace Lai 2008-02-14 13:31:18 UTC
Created attachment 2887 [details]
This is in default skin.
Comment 16 Wallace Lai 2008-02-14 13:41:57 UTC
Note:  It is easier to reproduce this bug is the Ray is connected with ethernet.  However, it happens either way.

Here are the steps to reproduce:
1.  Have version 18 of Ray's FW ready in the firmware folder.
2.  Stop squeezeCenter.
3.  Change the numbers in version file to 18.
4.  Start SqueezeCenter.
5.  Go to the default skin on the web UI.
6.  After Web UI finishes loading, the message will be there for a minute or two while Ray is downgrading.
7.  AFter Web UI goes back to playlist, stop SqueezeCenter.
8.  Chnage the version file to 22 or the latest FW version for Ray.
9.  Start up SqueezzeCenter again.
10.  Ray should start upgrading.
11.  After the Web UI finishes refreshing itself, the message will show for about a minute or two.
Comment 17 Wallace Lai 2008-04-01 14:33:40 UTC
Pinging anybody:
Have not seen this bug recently.  Has anyone fixed this?
Comment 18 Chris Owens 2008-04-14 10:06:58 UTC
Michael says he thinks this is fixed.   Please reopen if you still see it, Wallace.
Comment 19 Wallace Lai 2008-04-30 13:51:52 UTC
Unable to reproduce this bug any more with 7.0.1-19251.  I am convinced it is fixed.  Closing.
Comment 20 Chris Owens 2009-07-31 10:17:09 UTC
Reduce number of active targets for SC