[Slim-Checkins] r33857 - in /7.7/trunk/server: Changelog7.html Slim/Web/XMLBrowser.pm

[email protected] Wed, 22 Feb 2012 14:07:32 -0000
Newsgroups gmane.music.equipment.slimdevices.cvs
Message-ID <[email protected]>
Author: mherger
Date: Wed Feb 22 06:07:32 2012
New Revision: 33857

URL: http://svn.slimdevices.com/slim?rev=33857&view=rev
Log:
Fixed Bug: 17881
Description: don't set the track title when we're dealing with a local file - metadata is being extracted from the file itself

Modified:
    7.7/trunk/server/Changelog7.html
    7.7/trunk/server/Slim/Web/XMLBrowser.pm

Modified: 7.7/trunk/server/Changelog7.html
URL: http://svn.slimdevices.com/slim/7.7/trunk/server/Changelog7.html?rev=33857&r1=33856&r2=33857&view=diff
==============================================================================
--- 7.7/trunk/server/Changelog7.html (original)
+++ 7.7/trunk/server/Changelog7.html Wed Feb 22 06:07:32 2012
@@ -19,6 +19,7 @@
 		<li><a href="http://bugs.slimdevices.com/show_bug.cgi?id=17852">#17852</a> - Support for new ReadyNAS Duo and NV+ V2 devices</li>
 		<li><a href="http://bugs.slimdevices.com/show_bug.cgi?id=17855">#17855</a> - Only enable single output channel mode if actively synced</li>
 		<li><a href="http://bugs.slimdevices.com/show_bug.cgi?id=17877">#17877</a> - Resume after underrun broken for synchronized (ip3k) players</li>
+		<li><a href="http://bugs.slimdevices.com/show_bug.cgi?id=17881">#17881</a> - BMF scan corrupts dB replacing title with file name</li>
 		<li><a href="http://bugs.slimdevices.com/show_bug.cgi?id=17882">#17882</a> - Fixed wrong MP4 MIME type for DLNA (audio/m4a was used instead of audio/mp4)</li>
 	</ul>
 	<br />

Modified: 7.7/trunk/server/Slim/Web/XMLBrowser.pm
URL: http://svn.slimdevices.com/slim/7.7/trunk/server/Slim/Web/XMLBrowser.pm?rev=33857&r1=33856&r2=33857&view=diff
==============================================================================
--- 7.7/trunk/server/Slim/Web/XMLBrowser.pm (original)
+++ 7.7/trunk/server/Slim/Web/XMLBrowser.pm Wed Feb 22 06:07:32 2012
@@ -1300,6 +1300,10 @@
 			my $link = 'anyurl?p0=playlist&p1=' . $action . '&p2=' . Slim::Utils::Misc::escape($playUrl);
 	
 			my $title = $item->{'title'} || $item->{'name'};
+			
+			# bug 17881: don't set the track title when we're dealing with a local file, and the title is the filename (BMF)
+			$title = '' if $title && $item->{filename} && $item->{filename} eq $title;
+			
 			$link .= '&p3=' . Slim::Utils::Misc::escape($title) if $title;
 			 
 			return $link;