[Slim-Checkins] r10077 - in /trunk/server: Changelog6.html Slim/Buttons/Home.pm

[kdf at svn.slimdevices.com]

Author: kdf
Date: Wed Sep 27 22:38:06 2006
New Revision: 10077

URL: http://svn.slimdevices.com?rev=10077&view=rev
Log:
Bug: 4245
Description: reduce scope for pressing play to pushLeft on any BROWSE_* menu option, as BROWSE_MUSIC is the top level


Modified:
    trunk/server/Changelog6.html
    trunk/server/Slim/Buttons/Home.pm

Modified: trunk/server/Changelog6.html
URL: http://svn.slimdevices.com/trunk/server/Changelog6.html?rev=10077&r1=10076&r2=10077&view=diff
==============================================================================
--- trunk/server/Changelog6.html (original)
+++ trunk/server/Changelog6.html Wed Sep 27 22:38:06 2006
@@ -23,6 +23,7 @@
 		<li><a href="http://bugs.slimdevices.com/show_bug.cgi?id=4221">#4221</a> - 6.5 crashes on artist breadcrumb</li>
 		<li><a href="http://bugs.slimdevices.com/show_bug.cgi?id=4228">#4228</a> - Fishbone skin status doesn't update from an empty playlist</li>
 		<li><a href="http://bugs.slimdevices.com/show_bug.cgi?id=4237">#4237</a> - in Help > Internet Radio there is a dead link to Settings</li>
+		<li><a href="http://bugs.slimdevices.com/show_bug.cgi?id=4245">#4245</a> - Pressing PLAY on top level browse pushes right</li>
 	<ul>
 </ul>
 

Modified: trunk/server/Slim/Buttons/Home.pm
URL: http://svn.slimdevices.com/trunk/server/Slim/Buttons/Home.pm?rev=10077&r1=10076&r2=10077&view=diff
==============================================================================
--- trunk/server/Slim/Buttons/Home.pm (original)
+++ trunk/server/Slim/Buttons/Home.pm Wed Sep 27 22:38:06 2006
@@ -115,13 +115,14 @@
 		'play' => sub  {
 			my $client = shift;
 
-			if ($client->curSelection($client->curDepth()) eq 'NOW_PLAYING') {
+			my $selection = $client->curSelection($client->curDepth());
+			if ($selection eq 'NOW_PLAYING') {
 
 				$client->execute(['play']);
 
 				Slim::Buttons::Common::pushModeLeft($client, 'playlist');
 
-			} elsif ($client->curSelection($client->curDepth) =~ /^(?:BROWSE_|SAVED_PLAYLISTS)/) {
+			} elsif ($selection =~ /^(?:BROWSE_|SAVED_PLAYLISTS)/ && $selection !~ /^(?:BROWSE_MUSIC$)/) {
 
 				# If we're in a Browse mode and the user
 				# presses play, just go right, per Dean