After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 697793 - (DIDL_S) Empty DIDL_S in PlaylistContainer
(DIDL_S)
Empty DIDL_S in PlaylistContainer
Status: RESOLVED FIXED
Product: rygel
Classification: Applications
Component: MediaExport plugin
git master
Other Linux
: Normal enhancement
: ---
Assigned To: Jens Georg
rygel-maint
Depends on:
Blocks:
 
 
Reported: 2013-04-11 13:43 UTC by Ludovic Ferrandis
Modified: 2013-04-30 10:42 UTC
See Also:
GNOME target: ---
GNOME version: 2.31/2.32


Attachments
server: Don't add resource to empty containers (1.90 KB, patch)
2013-04-12 09:43 UTC, Jens Georg
none Details | Review
server: Don't add resource to empty containers (2.47 KB, patch)
2013-04-12 10:24 UTC, Jens Georg
committed Details | Review

Description Ludovic Ferrandis 2013-04-11 13:43:07 UTC
When getting PlaylistContainer properties, for an empty container, Rygel returns a res property with an empty DIDL_S file.

Rygel should not provide res+DIDL_S information when there is no media content.
Comment 1 Jens Georg 2013-04-12 09:43:50 UTC
Created attachment 241331 [details] [review]
server: Don't add resource to empty containers
Comment 2 Jens Georg 2013-04-12 09:46:00 UTC
Hm, no, that seems to leave a <res/> node in the XML...
Comment 3 Jens Georg 2013-04-12 10:24:46 UTC
Created attachment 241335 [details] [review]
server: Don't add resource to empty containers
Comment 4 Jens Georg 2013-04-12 10:52:26 UTC
Note that this still generates an empty DIDL_S file if the container only has container as its children, but better than nothing.
Comment 5 Jens Georg 2013-04-30 10:42:11 UTC
Attachment 241335 [details] pushed as 59ae651 - server: Don't add resource to empty containers