[Slim-Checkins] r33668 - /7.7/trunk/server/Slim/Plugin/ImageBrowser/Plugin.pm

[email protected]
Newsgroups gmane.music.equipment.slimdevices.cvs
Message-ID <[email protected]>
Author: mherger
Date: Mon Nov  7 23:06:52 2011
New Revision: 33668

URL: http://svn.slimdevices.com/slim?rev=33668&view=rev
Log:
Fixed Bug: 17737
Description: don't add albums as screensavers. We need a design, allowing the user to selectively provide screensavers.

Modified:
    7.7/trunk/server/Slim/Plugin/ImageBrowser/Plugin.pm

Modified: 7.7/trunk/server/Slim/Plugin/ImageBrowser/Plugin.pm
URL: http://svn.slimdevices.com/slim/7.7/trunk/server/Slim/Plugin/ImageBrowser/Plugin.pm?rev=33668&r1=33667&r2=33668&view=diff
==============================================================================
--- 7.7/trunk/server/Slim/Plugin/ImageBrowser/Plugin.pm (original)
+++ 7.7/trunk/server/Slim/Plugin/ImageBrowser/Plugin.pm Mon Nov  7 23:06:52 2011
@@ -74,21 +74,22 @@
 	
 	return if !$menu;
 	
-	my $data = _parseOPML('/il');
-	my $screensavers = [];
-	
-	my $albums = $data->{container};
-	
-	foreach my $item ( @$albums ) {
-		if ( $item->{'upnp:class'} =~ /^object.container/ ) {
-			push @$screensavers, {
-				cmd    => [ $args{tag}, 'items', 'id:' . $item->{id}, 'type:slideshow', 'slideshowId:' . $item->{id} ],
-				text => $item->{'dc:title'},
-			}
-		}
-	}
-	
-	$menu->[0]->{screensavers} = $screensavers;
+	# bug 17737 - we need a better design, make screensavers optional, selectable etc.
+#	my $data = _parseOPML('/il');
+#	my $screensavers = [];
+#	
+#	my $albums = $data->{container};
+#	
+#	foreach my $item ( @$albums ) {
+#		if ( $item->{'upnp:class'} =~ /^object.container/ ) {
+#			push @$screensavers, {
+#				cmd    => [ $args{tag}, 'items', 'id:' . $item->{id}, 'type:slideshow', 'slideshowId:' . $item->{id} ],
+#				text => $item->{'dc:title'},
+#			}
+#		}
+#	}
+#	
+#	$menu->[0]->{screensavers} = $screensavers;
 	
 	return $menu;
 }
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.