core,external: No need for special album art class
authorZeeshan Ali (Khattak) <zeeshanak@gnome.org>
Thu, 23 Dec 2010 14:57:42 +0000 (16:57 +0200)
committerZeeshan Ali (Khattak) <zeeshanak@gnome.org>
Thu, 23 Dec 2010 14:57:42 +0000 (16:57 +0200)
Thumbnail class is enough, no need for a special subclass for album art.

src/plugins/external/rygel-external-album-art-factory.vala
src/rygel/Makefile.am
src/rygel/rygel-album-art.vala [deleted file]
src/rygel/rygel-media-art-store.vala
src/rygel/rygel-music-item.vala

index b9665f4..60917cd 100644 (file)
@@ -44,7 +44,7 @@ public class Rygel.External.AlbumArtFactory {
 
     private Thumbnail create_from_props (HashTable<string,Variant> props,
                                          string                    host_ip) {
-        var thumbnail = new AlbumArt ();
+        var thumbnail = new Thumbnail ();
 
         var value = props.lookup ("MIMEType");
         thumbnail.mime_type = (string) value;
index 182673e..c779988 100644 (file)
@@ -92,7 +92,6 @@ VAPI_SOURCE_FILES = rygel-configuration.vala \
                    rygel-photo-item.vala \
                    rygel-thumbnail.vala \
                    rygel-thumbnailer.vala \
-                   rygel-album-art.vala \
                    rygel-media-art-store.vala \
                    rygel-subtitle.vala \
                    rygel-subtitle-manager.vala \
diff --git a/src/rygel/rygel-album-art.vala b/src/rygel/rygel-album-art.vala
deleted file mode 100644 (file)
index 39023c9..0000000
+++ /dev/null
@@ -1,39 +0,0 @@
-/*
- * Copyright (C) 2010 Jens Georg <mail@jensge.org>.
- *
- * Author: Jens Georg <mail@jensge.org>
- *
- * This file is part of Rygel.
- *
- * Rygel is free software; you can redistribute it and/or modify
- * it under the terms of the GNU Lesser General Public License as published by
- * the Free Software Foundation; either version 2 of the License, or
- * (at your option) any later version.
- *
- * Rygel is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public License
- * along with this program; if not, write to the Free Software Foundation,
- * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
- */
-
-using GUPnP;
-
-public class Rygel.AlbumArt : Thumbnail {
-    public AlbumArt() {
-        base ();
-    }
-
-    /* This doesn't really add a resource. Blame UPnP/DLNA for treating
-     * thumbnails and album-art differently for no apparent reason.
-     */
-    internal override DIDLLiteResource? add_resource (DIDLLiteItem didl_item,
-                                                      string       protocol) {
-        didl_item.album_art = this.uri;
-
-        return null;
-    }
-}
index 4c8a024..295f758 100644 (file)
@@ -80,7 +80,7 @@ public class Rygel.MediaArtStore : GLib.Object {
             return null;
         }
 
-        var thumb = new AlbumArt ();
+        var thumb = new Thumbnail ();
         thumb.uri = file.get_uri ();
         thumb.size = (int64) info.get_size ();
 
index 84bbd65..6742f60 100644 (file)
@@ -67,7 +67,7 @@ public class Rygel.MusicItem : AudioItem {
             var protocol = this.get_protocol_for_uri (this.album_art.uri);
 
             if (allow_internal || protocol != "internal") {
-                album_art.add_resource (didl_item, protocol);
+                didl_item.album_art = this.album_art.uri;
             }
         }
     }