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 676379 - Obey *-upload-folder settings
Obey *-upload-folder settings
Status: RESOLVED OBSOLETE
Product: rygel
Classification: Applications
Component: MediaExport plugin
git master
Other Linux
: Normal enhancement
: ---
Assigned To: Jens Georg
rygel-maint
ivi
Depends on:
Blocks:
 
 
Reported: 2012-05-19 18:10 UTC by Jens Georg
Modified: 2018-05-22 12:34 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
media-export: Use upload folder settings (10.70 KB, patch)
2013-06-10 15:03 UTC, Jens Georg
none Details | Review
media-export: Use upload folder settings (12.37 KB, patch)
2013-06-11 11:56 UTC, Jens Georg
none Details | Review

Description Jens Georg 2012-05-19 18:10:29 UTC
And while doing that, make those folders
  a) shared if they aren't
  b) make those the _only_ folders we upload to instead of every folder
Comment 1 Mark Ryan 2013-02-20 08:38:56 UTC
I've noticed that any media I upload to the AnyContainer using the MediaExport plugin always ends up in ~/Pictures, even if I'm uploading an audio file.

Here's an example of the CreateObject requests that I'm sending.

<DIDL-Lite xmlns:dc="http://purl.org/dc/elements/1.1/" xmlns:upnp="urn:schemas-upnp-org:metadata-1-0/upnp/" xmlns:dlna="urn:schemas-dlna-org:metadata-1-0/" xmlns="urn:schemas-upnp-org:metadata-1-0/DIDL-Lite/">
  <item id="" parentID="DLNA.ORG_AnyContainer" restricted="0">
    <dc:title>O-MP3-8.mp3</dc:title>
    <upnp:class>object.item.audioItem</upnp:class>
    <res protocolInfo="*:*:audio/mpeg:DLNA.ORG_PN=MP3"/>
  </item>
</DIDL-Lite>

Is this the same bug?  If not, I'll enter a new one.
Comment 2 Jens Georg 2013-02-20 14:11:28 UTC
yes, that's the same bug, part b)
Comment 3 Jens Georg 2013-02-26 10:28:24 UTC
b) actually should only apply for AnyContainer uploads, otherwise a true sync is not possible.
Comment 4 Mark Ryan 2013-02-26 13:53:42 UTC
Actually, I have another question about this.

Do we need a new upload folder for playlists items?
Comment 5 Jens Georg 2013-02-27 09:13:35 UTC
Good point.
Comment 6 Jens Georg 2013-05-17 09:06:04 UTC
Thought a bit about this, one thing we need to make sure is that we don't accidentaly share stuff that isn't supposed to be shared, the upload folders point to the standard XDG folders by default.
Comment 7 Jens Georg 2013-06-10 15:03:58 UTC
Created attachment 246410 [details] [review]
media-export: Use upload folder settings

When a client tries to upload a file, try to use the configured upload
folders instead of just picking some random folder to upload to.
Comment 8 Jens Georg 2013-06-10 15:05:59 UTC
First draft, not fixing all of the problems, but getting there.

One thing I'm not particularly sure about is whether to just return the upload folder or manipulating the results list and putting the upload folder on first place.
Comment 9 Jens Georg 2013-06-11 11:56:54 UTC
Created attachment 246502 [details] [review]
media-export: Use upload folder settings

When a client tries to upload a file, try to use the configured upload
folders instead of just picking some random folder to upload to.
Comment 10 Jens Georg 2013-06-18 07:31:37 UTC
Missing: set createClass of these folders to the relevant ones only
Comment 11 GNOME Infrastructure Team 2018-05-22 12:34:32 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/19.