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 702258 - 7.3.113.1,7.3.121.1,7.3.133.1,2,3,5,9,7.3.133.7: Need object.container upnp:createClass
7.3.113.1,7.3.121.1,7.3.133.1,2,3,5,9,7.3.133.7: Need object.container upnp:c...
Status: RESOLVED OBSOLETE
Product: rygel
Classification: Applications
Component: MediaExport plugin
git master
Other Linux
: Normal normal
: ---
Assigned To: Jens Georg
rygel-maint
ivi
Depends on:
Blocks: 702083
 
 
Reported: 2013-06-14 12:07 UTC by Jens Georg
Modified: 2018-05-22 12:38 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
server,media-export: Use object.container (8.76 KB, patch)
2013-06-18 13:58 UTC, Jens Georg
committed Details | Review
media-export: Use DLNA-compliant createClass (2.27 KB, patch)
2013-06-18 14:07 UTC, Jens Georg
committed Details | Review

Description Jens Georg 2013-06-14 12:07:21 UTC
We conly provide object.container.storageFolder. Probably should think about using object.container everywhere and rewrite the UPnP class to .storageFolder in the XBox hacks.
Comment 1 Parthiban Balasubramanian 2013-06-18 13:30:16 UTC
@Jens : We analyzed this issue and looks more like a DLNA CTT issue. I will be creating a bug with them to check if they accept it.

Based on ContentDirectory-v3-Service under Annex C.2.2.9 storageFolder:container: "This class is derived from the container class and inherits the properties defined by that class. Additionally,
the following REQUIRED properties are defined for this class:"

It looks like DLNA CTT is not considering storageFolder
Comment 2 Jens Georg 2013-06-18 13:58:06 UTC
Created attachment 247148 [details] [review]
server,media-export: Use object.container

We used to use object.container.storageFolder for XBox 360 compatibility but
DLNA doesn't like that.
Comment 3 Jens Georg 2013-06-18 14:07:58 UTC
Created attachment 247150 [details] [review]
media-export: Use DLNA-compliant createClass

DLNA does not like upnp:createClass other than object.container for
containers. So we use that and use the possibility to "upgrade" the
object's class upon creation.
Comment 4 Jens Georg 2013-06-18 14:28:35 UTC
(In reply to comment #1)
> @Jens : We analyzed this issue and looks more like a DLNA CTT issue. I will be
> creating a bug with them to check if they accept it.
> 
> Based on ContentDirectory-v3-Service under Annex C.2.2.9
> storageFolder:container: "This class is derived from the container class and
> inherits the properties defined by that class. Additionally,
> the following REQUIRED properties are defined for this class:"
> 
> It looks like DLNA CTT is not considering storageFolder

Please check. It was ok to use though when create-folder caps wasn't involved. But having said that, we only use it for compatibility with the XBox 360's DMP and this can be handled otherwise (patch in comment 2)
Comment 5 Jens Georg 2013-06-25 11:43:35 UTC
Leaving this bug open for now

Attachment 247148 [details] pushed as 56845ce - server,media-export: Use object.container
Attachment 247150 [details] pushed as 92885a0 - media-export: Use DLNA-compliant createClass
Comment 6 Parthiban Balasubramanian 2013-06-26 16:05:05 UTC
@Jens : FYI,I have raised this issue with DLNA. https://dlnadev.org/bugzilla/show_bug.cgi?id=1812

Will let you know what they come up with.
Comment 7 Parthiban Balasubramanian 2013-06-26 16:21:50 UTC
Also https://dlnadev.org/bugzilla/show_bug.cgi?id=1813
Comment 8 Jens Georg 2013-06-27 10:58:55 UTC
sorry, can't access that.
Comment 9 GNOME Infrastructure Team 2018-05-22 12:38:35 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to GNOME's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.gnome.org/GNOME/rygel/issues/39.