Bug 4953 – Menu Option "Start SlimServer" replaced with "Starting SlimServer..."

Bug 4953 - Menu Option "Start SlimServer" replaced with "Starting SlimServer..."

Summary:

Menu Option "Start SlimServer" replaced with "Starting SlimServer..."

Status:	CLOSED FIXED

Product:	Logitech Media Server
Classification:	Unclassified
Component:	Slimtray
Version:	6.5.2
Platform:	PC Windows XP

Importance:	P2 normal (vote)
Target Milestone:	---
Assigned To:	Squeezebox QA Team email alias

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2007-05-01 15:49 UTC by Spies Steven
Modified:	2008-12-18 11:11 UTC (History)
CC List:	3 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 Spies Steven 2007-05-01 15:49:09 UTC

To reproduce install the nightly SlimServer.
Once installed make sure SlimServer is stopped.
Click on the Slimtray icon.
Notice the first menu option is "Starting SlimServer..." in plain type.
It used to be "Start SlimServer" in bold type.


SlimServer Version: 6.5.2 - 11874 - Windows XP - EN - cp1252
Perl Version: 5.8.8 MSWin32-x86-multi-thread
MySQL Version: 5.0.22-community-nt

Comment 1 KDF 2007-05-01 18:45:42 UTC

a slip in the recent slimtray.pl changes, perhaps?
possibly change 11873 around line 79

Comment 2 KDF 2007-05-01 20:33:43 UTC

Index: SlimTray.pl
===================================================================
--- SlimTray.pl (revision 11880)
+++ SlimTray.pl (working copy)
@@ -87,7 +87,7 @@
                push @menu, [string('STARTING_SLIMSERVER'), ""];
        }
        else {
-               push @menu, [string('STARTING_SLIMSERVER'), \&startSlimServer] if (!$vistaUser || $type =~ /none|login/);
+               push @menu, [string('START_SLIMSERVER'), \&startSlimServer] if (!$vistaUser || $type =~ /none|login/);
        }
 
        my $serviceString = string('RUN_AT_BOOT');

Comment 3 KDF 2007-05-01 20:34:33 UTC

cc'ing michael in case he's online first.

Comment 4 Michael Herger 2007-05-01 22:21:18 UTC

Change 11881 - please test

Comment 5 KDF 2007-05-02 22:27:48 UTC

restored the boldface in change 11890

Marking as fixed, but it can be reopened if there are problems.  

note: someone should give Michael access to marking bug as fixed :)

Comment 6 Chris Owens 2007-05-22 12:10:29 UTC

Fixed in 6.5.2, which is now released and available for download at http://www.slimdevices.com/su_downloads.html

If you're still experiencing this bug, please re-open it!