[Slim-Checkins] r9304 - /trunk/server/Slim/Player/Player.pm

adrian at svn.slimdevices.com adrian at svn.slimdevices.com
Wed Aug 30 17:02:14 PDT 2006


Author: adrian
Date: Wed Aug 30 17:02:12 2006
New Revision: 9304

URL: http://svn.slimdevices.com?rev=9304&view=rev
Log:
Bug: N/A
Description: r9289 didn't return param 'visu' to its original state correctly

Modified:
    trunk/server/Slim/Player/Player.pm

Modified: trunk/server/Slim/Player/Player.pm
URL: http://svn.slimdevices.com/trunk/server/Slim/Player/Player.pm?rev=9304&r1=9303&r2=9304&view=diff
==============================================================================
--- trunk/server/Slim/Player/Player.pm (original)
+++ trunk/server/Slim/Player/Player.pm Wed Aug 30 17:02:12 2006
@@ -741,8 +741,7 @@
 	my $scale = $client->mixerConstant($feature, 'scale');
 
 	my $headerValue = '';
-	my $parts;
-	my $oldvisu;
+	my ($parts, $oldvisu, $savedvisu);
 
 	if ($client->mixerConstant($feature, 'balanced')) {
 
@@ -775,6 +774,7 @@
 	if (blessed($client->display) eq 'Slim::Display::Squeezebox2') {
 		# XXXX hack attack: turn off visualizer when showing volume, etc.		
 		$oldvisu = $client->modeParam('visu');
+		$savedvisu = 1;
 		$client->modeParam('visu', [0]);
 	}
 
@@ -788,7 +788,7 @@
 	$client->display->showBriefly($parts, { 'name' => 'mixer' } );
 
 	# Turn the visualizer back to it's old value.
-	if (defined $oldvisu) {
+	if ($savedvisu) {
 		$client->modeParam('visu', $oldvisu);
 	}
 }



More information about the checkins mailing list