[Slim-Checkins] r11761 - in /trunk/server: Changelog6.html Slim/Buttons/BrowseTree.pm Slim/Music/Info.pm Slim/Web/Pages/BrowseTree.pm

[mherger at svn.slimdevices.com]

Author: mherger
Date: Fri Apr 13 00:06:30 2007
New Revision: 11761

URL: http://svn.slimdevices.com?rev=11761&view=rev
Log:
Bug: 4902
Description: 
- correctly display and follow Windows shortcuts pointing to another drive
- fix an issue with the BMF path display on the player

Modified:
    trunk/server/Changelog6.html
    trunk/server/Slim/Buttons/BrowseTree.pm
    trunk/server/Slim/Music/Info.pm
    trunk/server/Slim/Web/Pages/BrowseTree.pm

Modified: trunk/server/Changelog6.html
URL: http://svn.slimdevices.com/trunk/server/Changelog6.html?rev=11761&r1=11760&r2=11761&view=diff
==============================================================================
--- trunk/server/Changelog6.html (original)
+++ trunk/server/Changelog6.html Fri Apr 13 00:06:30 2007
@@ -45,6 +45,7 @@
 		<li><a href="http://bugs.slimdevices.com/show_bug.cgi?id=4852">#4852</a> - down arrow moves song to 2nd position in playlist</li>
 		<li><a href="http://bugs.slimdevices.com/show_bug.cgi?id=4872">#4872</a> - Installer: Error creating INI entry in Logitech.url</li>
 		<li><a href="http://bugs.slimdevices.com/show_bug.cgi?id=4891">#4891</a> - "Play this song" and "Add this song to playlist" links are not always visible.</li>
+		<li><a href="http://bugs.slimdevices.com/show_bug.cgi?id=4902">#4902</a> - BMF: correctly display and follow Windows shortcuts pointing to a drive; fix an issue with the path display on the player interface</li>
 	</ul>
 	<br />
 	<li>Other:

Modified: trunk/server/Slim/Buttons/BrowseTree.pm
URL: http://svn.slimdevices.com/trunk/server/Slim/Buttons/BrowseTree.pm?rev=11761&r1=11760&r2=11761&view=diff
==============================================================================
--- trunk/server/Slim/Buttons/BrowseTree.pm (original)
+++ trunk/server/Slim/Buttons/BrowseTree.pm Fri Apr 13 00:06:30 2007
@@ -393,12 +393,15 @@
 	} else {
 
 		# one level down we show the folder name, below that we show two levels
-		my $level = (scalar @levels > 2) ? $levels[-2] : $levels[-1];
-		my $obj   = Slim::Schema->find('Track', $level);
-
-		if (blessed($obj) && $obj->can('title')) {
-
-			push @headers, $obj->title;
+
+		for (my $x = (scalar @levels > 2 ? -2 : -1); $x <= -1; $x++) {
+
+			my $obj   = Slim::Schema->find('Track', $levels[$x]);
+
+			if (blessed($obj) && $obj->can('title')) {
+
+				push @headers, ($obj->title ? $obj->title : Slim::Music::Info::fileName($obj->url));
+			}
 		}
 	}
 	

Modified: trunk/server/Slim/Music/Info.pm
URL: http://svn.slimdevices.com/trunk/server/Slim/Music/Info.pm?rev=11761&r1=11760&r2=11761&view=diff
==============================================================================
--- trunk/server/Slim/Music/Info.pm (original)
+++ trunk/server/Slim/Music/Info.pm Fri Apr 13 00:06:30 2007
@@ -668,7 +668,7 @@
 
 		$j = Slim::Utils::Misc::pathFromFileURL($j);
 
-		if ($j) {
+		if ($j && (splitdir($j))[1]) {
 			$j = (splitdir($j))[-1];
 		}
 

Modified: trunk/server/Slim/Web/Pages/BrowseTree.pm
URL: http://svn.slimdevices.com/trunk/server/Slim/Web/Pages/BrowseTree.pm?rev=11761&r1=11760&r2=11761&view=diff
==============================================================================
--- trunk/server/Slim/Web/Pages/BrowseTree.pm (original)
+++ trunk/server/Slim/Web/Pages/BrowseTree.pm Fri Apr 13 00:06:30 2007
@@ -52,7 +52,8 @@
 
 			push @{$params->{'pwd_list'}}, {
 				'hreftype'     => 'browseTree',
-				'title'        => $i == 0 ? string('MUSIC') : $obj->title,
+				'title'        => $i == 0 ? string('MUSIC') :
+						($obj->title ? $obj->title : Slim::Music::Info::fileName($obj->url)),
 				'hierarchy'    => join('/', @levels[0..$i]),
 			};
 		}
@@ -76,7 +77,7 @@
 	if ($count) {
 		my %form = %$params;
 
-		$form{'hierarchy'}	  = undef;
+		$form{'hierarchy'}   = undef;
 		$form{'descend'}     = 1;
 		$form{'text'}        = string('ALL_SONGS');
 		$form{'itemobj'}     = $topLevelObj;