View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0013481 | MMA | General | public | 2016-08-26 11:18 | 2016-12-21 09:23 |
| Reporter | marek | Assigned To | |||
| Priority | immediate | Severity | major | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Product Version | 1.2.0 | ||||
| Target Version | 1.2.0 | Fixed in Version | 1.3.0 | ||
| Summary | 0013481: Scanned folders might be lost | ||||
| Description | There are specific situations where scanned folders are removed. This is caused by bug in conversion of path to document URI | ||||
| Steps To Reproduce | replicate 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/] | ||||
| Tags | No tags attached. | ||||
| Fixed in build | 650 | ||||
|
|
Fixed in build 613 |
|
|
Reopen If new SDCard is selected that do not contain those folder they are not Automatically created. |
|
|
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,...) |
|
|
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 |
|
|
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 |
|
|
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. |
|
|
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). |
|
|
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. |
|
|
Verified 654 Alcatel Idol 3, Nexus 7 2013, asus memopad 7 hd 653 LG Verizon |
|
|
Verified in Extended test 657 |
