core: Use correct error in CreateObject
authorJens Georg <mail@jensge.org>
Mon, 9 Jan 2012 18:05:38 +0000 (19:05 +0100)
committerJens Georg <mail@jensge.org>
Wed, 18 Jan 2012 17:00:15 +0000 (18:00 +0100)
If CreateObject is called with an id that describes an item instead of
a container ContentDirectory:2 spec says that the error should be 701
(No such object) instead of 713 (Restricted parent)

src/rygel/rygel-item-creator.vala

index d6c867e..51b9040 100644 (file)
@@ -283,11 +283,10 @@ internal class Rygel.ItemCreator: GLib.Object, Rygel.StateMachine {
                                         (this.container_id, this.cancellable);
         }
 
-        if (media_object == null) {
+        if (media_object == null || !(media_object is MediaContainer)) {
             throw new ContentDirectoryError.NO_SUCH_OBJECT
                                         (_("No such object"));
-        } else if (!(media_object is MediaContainer) ||
-                   !(OCMFlags.UPLOAD in media_object.ocm_flags)) {
+        } else if (!(OCMFlags.UPLOAD in media_object.ocm_flags)) {
             throw new ContentDirectoryError.RESTRICTED_PARENT
                                         (_("Object creation in %s not allowed"),
                                         media_object.id);