[Slim-Checkins] r11847 - in /trunk/server: Changelog6.html HTML/EN/cmdwrappers Slim/Display/Lib/Fonts.pm Slim/Music/Info.pm Slim/Utils/Misc.pm Slim/Utils/Text.pm

[kdf at svn.slimdevices.com]

Author: kdf
Date: Fri Apr 27 17:35:53 2007
New Revision: 11847

URL: http://svn.slimdevices.com?rev=11847&view=rev
Log:
Bug: 4940
Description: allow 0 for strings and titles, merged from 11846 from 6.5.2

Modified:
    trunk/server/Changelog6.html
    trunk/server/HTML/EN/cmdwrappers
    trunk/server/Slim/Display/Lib/Fonts.pm
    trunk/server/Slim/Music/Info.pm
    trunk/server/Slim/Utils/Misc.pm
    trunk/server/Slim/Utils/Text.pm

Modified: trunk/server/Changelog6.html
URL: http://svn.slimdevices.com/trunk/server/Changelog6.html?rev=11847&r1=11846&r2=11847&view=diff
==============================================================================
--- trunk/server/Changelog6.html (original)
+++ trunk/server/Changelog6.html Fri Apr 27 17:35:53 2007
@@ -53,6 +53,7 @@
 		<li><a href="http://bugs.slimdevices.com/show_bug.cgi?id=4910">#4910</a> - Fishbone skin:  Some texts are cut off.</li>
 		<li><a href="http://bugs.slimdevices.com/show_bug.cgi?id=4924">#4924</a> - Nice to have: Up and Down buttons for PlayList on Dark skin</li>
 		<li><a href="http://bugs.slimdevices.com/show_bug.cgi?id=4926">#4926</a> - Mac Installer & Preference Pane still shows 6.5.1 should be 6.5.2</li>
+		<li><a href="http://bugs.slimdevices.com/show_bug.cgi?id=4940">#4940</a> - Error! url:[0] is empty or a track could not be read</li>
 	</ul>
 	<br />
 	<li>Other:

Modified: trunk/server/HTML/EN/cmdwrappers
URL: http://svn.slimdevices.com/trunk/server/HTML/EN/cmdwrappers?rev=11847&r1=11846&r2=11847&view=diff
==============================================================================
--- trunk/server/HTML/EN/cmdwrappers (original)
+++ trunk/server/HTML/EN/cmdwrappers Fri Apr 27 17:35:53 2007
@@ -129,7 +129,7 @@
 
 [%# this is an individual bread crumb %]
 [% BLOCK crumblistitem -%]
-[%- IF item.title -%]
+[%- IF item.exists('title') -%]
 <a [% PROCESS "${item.hreftype}ItemHRef" %]>[% item.title | html %]</a>
 [%- ELSE -%]
 [% item | replace('^\s*/','') %]

Modified: trunk/server/Slim/Display/Lib/Fonts.pm
URL: http://svn.slimdevices.com/trunk/server/Slim/Display/Lib/Fonts.pm?rev=11847&r1=11846&r2=11847&view=diff
==============================================================================
--- trunk/server/Slim/Display/Lib/Fonts.pm (original)
+++ trunk/server/Slim/Display/Lib/Fonts.pm Fri Apr 27 17:35:53 2007
@@ -239,7 +239,11 @@
 
 sub string {
 	my $defaultFontname = shift || return (0, '');
-	my $string = shift || return (0, '');
+	my $string = shift;
+	
+	if (!defined $string) {
+		return (0, '');
+	}
 
 	my $defaultFont = $fonts->{$defaultFontname} || do {
 

Modified: trunk/server/Slim/Music/Info.pm
URL: http://svn.slimdevices.com/trunk/server/Slim/Music/Info.pm?rev=11847&r1=11846&r2=11847&view=diff
==============================================================================
--- trunk/server/Slim/Music/Info.pm (original)
+++ trunk/server/Slim/Music/Info.pm Fri Apr 27 17:35:53 2007
@@ -671,7 +671,7 @@
 
 		$j = Slim::Utils::Misc::pathFromFileURL($j);
 
-		if ($j && (splitdir($j))[-1]) {
+		if (defined $j && defined ((splitdir($j))[-1])) {
 			$j = (splitdir($j))[-1];
 		}
 

Modified: trunk/server/Slim/Utils/Misc.pm
URL: http://svn.slimdevices.com/trunk/server/Slim/Utils/Misc.pm?rev=11847&r1=11846&r2=11847&view=diff
==============================================================================
--- trunk/server/Slim/Utils/Misc.pm (original)
+++ trunk/server/Slim/Utils/Misc.pm Fri Apr 27 17:35:53 2007
@@ -597,8 +597,12 @@
 
 # there's not really a better way to do this..
 sub fixPath {
-	my $file = shift || return;
+	my $file = shift;
 	my $base = shift;
+
+	if (!defined($file)) {
+		return;
+	}
 
 	my $fixed;
 

Modified: trunk/server/Slim/Utils/Text.pm
URL: http://svn.slimdevices.com/trunk/server/Slim/Utils/Text.pm?rev=11847&r1=11846&r2=11847&view=diff
==============================================================================
--- trunk/server/Slim/Utils/Text.pm (original)
+++ trunk/server/Slim/Utils/Text.pm Fri Apr 27 17:35:53 2007
@@ -39,7 +39,11 @@
 =cut
 
 sub ignorePunct {
-	my $s = shift || return undef;
+	my $s = shift;
+
+	if (!defined $s) {
+		return undef;
+	}
 
 	my $orig = $s;
 
@@ -62,7 +66,11 @@
 =cut
 
 sub matchCase {
-	my $s = shift || return undef;
+	my $s = shift;
+
+	if (!defined $s) {
+		return undef;
+	}
 
 	# Upper case and fold latin1 diacritical characters into their plain versions, surprisingly useful.
 	$s =~ tr{abcdefghijklmnopqrstuvwxyzÀÁÂÃÄÅßÞÇ¢ÐÈÉÊËÌÍÎÏÑÒÓÔÕÖØÙÚÛÜ×Ýàáâãäåþçèéêëìíîïñòóôõöøùúûüÿýð¡°}
@@ -90,7 +98,11 @@
 =cut
 
 sub ignoreArticles {
-	my $item = shift || return;
+	my $item = shift;
+
+	if (!defined $item) {
+		return undef;
+	}
 
 	if (!defined($ignoredArticles)) {
 
@@ -116,7 +128,11 @@
 =cut
 
 sub ignoreCaseArticles {
-	my $s = shift || return undef;
+	my $s = shift;
+
+	if (!defined $s) {
+		return undef;
+	}
 
 	# We don't handle references of any kind.
 	if (ref($s)) {