View Issue Details

IDProjectCategoryView StatusLast Update
0013481MMAGeneralpublic2016-12-21 09:23
Reportermarek Assigned To 
PriorityimmediateSeveritymajorReproducibilityalways
Status closedResolutionfixed 
Product Version1.2.0 
Target Version1.2.0Fixed in Version1.3.0 
Summary0013481: Scanned folders might be lost
DescriptionThere are specific situations where scanned folders are removed. This is caused by bug in conversion of path to document URI
Steps To Reproducereplicate the original issue by these steps:
1) Start MediaMonkey
2) Grant access to sdcard
3) Automatically selected library folders [/storage/external_SD/Video/, /storage/emulated/0/Music/, /storage/external_SD/Music/, /storage/emulated/0/Movies/]
5) Go to home screen (MMA to backgrand)
6) On go back to MMA foreground are selected only library folders
[/storage/emulated/0/Music/, /storage/emulated/0/Movies/]
TagsNo tags attached.
Fixed in build650

Relationships

related to 0013246 closedmarek Rework the way how the path to file is stored to DB and preferences 

Activities

marek

2016-08-26 11:18

developer   ~0045497

Fixed in build 613

peke

2016-08-26 14:11

developer   ~0045499

Reopen If new SDCard is selected that do not contain those folder they are not Automatically created.

peke

2016-08-26 14:15

developer   ~0045500

Last edited: 2016-08-26 14:19

Steps to reproduce:
1. Backup /MediaMonkey/ folder to keep same storageinfo
2. Format SD card
3. Restore /MediaMonkey/ Folder so that MMA use existing storage info
4. Install MMA
5. Grant SD card Access
6. Only root is available but specific default folders should be created and added to library (Music, Playlists, Videos,...)

marek

2016-08-26 16:08

developer   ~0045502

We do not create these folders, as opposite we detect that the folders are removed and we remove them.

Btw. you remove the folder settings by uninstalling MMA - tracked as 0013285

peke

2016-08-26 22:22

developer   ~0045504

As we do not crete them by default we should:

a) Create default folders on grant access and add them to library folders

b) Add them to library folders and create them on first sync

marek

2016-08-29 16:55

developer   ~0045512

a) Android creates the folders too. We do not know the exact folder names (Video or Movies) and I think that we can create them during sync

b) Existing folders that are set in sync profile are automatically added to library folders.

But there is one limitation now, here is an example:
1. User wants to sync to /Music and /Video on SD card(SAF storage), no folders are allowed for writing and only /Music folder exists
2. User starts wifi sync and it request writable access to /Music folder only. It won't request /Video folder because it doesn't exist and MMA cannot create it.
3. Wifi sync continues when /Music folder is allowed by user (not whole sd card!) and video tracks are synced to application specific folder and write request for /Video folder is not requested and /Video folder is not created at all.

To fix this, MMA should probably find out that /Video doesn't exist and request access to whole SD card to be able to create /Video folder.

One detail: what folders should be created by default?
 i) folders that are listed in MMW in Auto-Sync(-->Library) tab - probably YES
 ii) folders that are listed in MMW in Options->File Locations->Other media files - probably NO
 

Please note that this issue occurs on SAF storages only, writable storages are ok. The folder is created when track (e.g. "/Video/movie.mp4") is downloaded.

rusty

2016-08-29 18:09

administrator   ~0045513

I don't understand what the ambiguity is. i.e. MMA should just create folders when it needs to write to folders that don't yet exist. So if sync destination locations don't have access, then they should automatically be granted access to in the library. Which is pretty much what you described in points 1-3 at 0013481:0045512

Re. what folders mma writes to, they're:
a) Auto-sync --> Library (but since it's disabled by default, access only needs to be granted once it is enabled)
c) the folders that are configured in Sync Options > Options > File Locations > 'Sync files to:'

Or am I missing something?

Note: b) Sync Options > Options > File Locations > Other Media Files is just used to determine which files are scanned in MMW for display in the MMW UI (in cases where there are files in directories other than those that are synced by MMW --> device).

marek

2016-10-01 08:56

developer   ~0045803

This is now fixed in 1.2.1.650

There are following fixes:
1. MMA parses target paths at the start of wifi sync and requests write permission to it's parent folder to be able to create them
2. When no folder exists on storage, the root folder is set as scanned folder(as it was till now) BUT it is set as temporary and any newly added scanned folder will replace it. That means that syncing of sync profile will cause that the root is replaced by scanned folders from MMW profile and these folders are created.

peke

2016-12-01 15:13

developer   ~0046352

Last edited: 2016-12-01 15:23

Verified 654 Alcatel Idol 3, Nexus 7 2013, asus memopad 7 hd
653 LG Verizon

peke

2016-12-21 09:23

developer   ~0046665

Verified in Extended test 657