View Issue Details

IDProjectCategoryView StatusLast Update
0010650MMAUPnP / Castingpublic2013-04-19 03:22
Reporterrusty Assigned To 
PriorityurgentSeverityminorReproducibilityalways
Status closedResolutionfixed 
Product Version1.0.4 
Target Version1.0.4 
Summary0010650: UPnP download manager doesn't offer choice of save location
DescriptionAs of build 113, when the user downloads content via UPnP, it's saved to the devices internal memory. This is kind of annoying, especially when the user has already configured MM to save synced content to the external memory.

For the short term we should allow the user to choose a folder for downloaded content (preferably via some sort of directory browser since most users won't know paths).

e.g. above the DEVICE LOGS section:
UPnP
Download location
Save files to: <path>

Clicking the entry would open some sort of directory browser that lets the user:
a) browse to the internal / external location
b) browse to the desired directory
c) create a new directory [nice to have]
TagsNo tags attached.
Attached Files
Fixed in build122

Activities

jiri

2013-03-14 20:28

administrator   ~0035374

Note that per IM discussion, we will also continue to store different content (Music, Video, ...) to different locations, but only relative to the manually selected path described in this issue.

marek

2013-03-15 23:56

developer   ~0035402

Implemented in build 114.

Create new folder option is missing...

marek

2013-03-19 09:52

developer   ~0035424

Fixed in build 115

rusty

2013-03-19 22:46

administrator   ~0035431

Tested build 115 and it's working for basic usecases, but there are still a few issues:
1) Default Configuration should be to save to an external location (rather than internal)

2) Configuration panel uses a '+' in the top Action bar (whereas MMA has used the convention of only showing Global commands in the Action bar, and shown context-specific commands in the menu bar at the bottom). Would it make sense to structure this screen similarly to the 'Send to Playlist' screen i.e. either:
a) leave as is (whole screen), but have 3 buttons at the bottom: Cancel, New folder, OK
b) use a modal 'Choose folder' dialog similar to plalists (or to the attached image) with 3 buttons: Cancel, New, OK

3) When tracks are downloaded they seem to be saved to 2 directories (Video and Music). I assume that MM can't detect whether a track is an Audiobook or Music track. So it might be preferable to save such tracks to /Audio as it is more generic than /Music.

4) The paths seem to use a standard mask (I think it's <Artist>-<Album>/<Artist>-<Title>), however, if metadata is missing (e.g. if Album="") then the path shows up as "The Beatles-null/The Beatles-She Loves You". If part of the path is unknown, then that portion should appear blank instead of 'null'.

5) Downloaded Audiobooks or Podcasts don't appear in their proper locations under Audiobooks or Podcasts (i.e. scanning rules which place them in the appropriate category based on Genre aren't applied).

6) Podcasts appear to download, but aren't saved to the configured path (in fact I don't see them anywhere in the MMA library)

marek

2013-04-09 18:22

developer   ~0035590

2) remade in build 121 to dialog

3) if upnp server publishes tracks as audiobooks, MMA can distinguish between them. Here are default directories:

            public static final String MUSIC = "/Music/";
            public static final String VIDEO = "/Video/";
            public static final String AUDIOBOOK = "/Music/Audiobooks/";
            public static final String PODCAST = "/Music/Podcasts/";

marek

2013-04-09 18:30

developer   ~0035591

5) it seems that MMW doesn't set type of upnp track properly and everything is set as music track. I've contacted Ludek.

marek

2013-04-10 19:53

developer   ~0035596

1) fixed in build 122

4) fixed in build 122

5) build 122 will contain parsing from genres and parsing of our MM specific type parameter. This is not implemented in MMW. Assigning to Ludek.

6) I found some problems when path contained invalid characters. This is fixed in build 122

Ludek

2013-04-10 20:51

developer   ~0035597

5) is fixed in MMW build 1631

marek

2013-04-10 20:53

developer   ~0035598

Fixed in build 122

peke

2013-04-19 03:22

developer   ~0035675

Verified 123