View Issue Details

IDProjectCategoryView StatusLast Update
0021187MMAUI (general)public2024-09-30 22:10
Reportermartin Assigned To 
PriorityurgentSeverityminorReproducibilityhave not tried
Status closedResolutionreopened 
PlatformAndroidOS-OS Versionall
Product Version2.0.2 
Target Version2.0.1Fixed in Version2.0.1 
Summary0021187: Scan options / Prevent MMA rescan option on device reboot/MMA re-install
DescriptionAdd a separate options screen to summarize all settings related to scanning/V3 scanning.
The link to this screen can be added as an icon(or button) in the header of the Library folders screen.
Also, it can be added under the "Library" section in "Library & Sync" options.

For MediaStore scanning, there can be options:
How often scan new/changed media:
[] Manual
[] Each <30 min> (by default)
If the manual option is selected, then the scan button should be in visible place.

For V3 sync it can be:
Scanning MM5 metadata switch[enabled/disabled] (if disabled a user is not asked when storage is mounted)
[Grant access to folders] - visible only if some is missing
[] automatically scan if new/changed media are detected (if enabled a user is not asked when storage is mounted and sync is performed automatically)


I will add a picture of the UI design later.






TagsNo tags attached.
Fixed in build1214

Relationships

related to 0020918 resolvedmartin Make MMA function without Video/Photo access 
related to 0021140 closedrusty MMA nags user to scan for updated content on each MM start (regression) 

Activities

rusty

2024-09-13 15:16

administrator   ~0076938

1) Should 'Grant access to external storage' --> 'Grant access to storage' (i.e. doesn't this apply to internal and external storage)?

2) Re. 'Extended scanning', why is any configuration required? i.e. it's not reasonable to expect users to understand the mechanics of MM synchronization; shouldn't MMA automatically use 'extended sync metadata' automatically whenever it exists assuming that it is the most recent metadata?

martin

2024-09-13 15:35

developer   ~0076942

1) yes, since Android 11 this is related to internal storage too, so it can be changed to 'Grant access to storage'
2) it's possible when we take "Automatically" as the only option.

rusty

2024-09-13 16:01

administrator   ~0076943

Last edited: 2024-09-13 16:11

2) As discussed offline, there aren't any cases where 'sync metadata' should be ignored.
 i.e. it should be used automatically / according to scan configuration and whatever case triggers a folder scan should also trigger an update of 'synced metadata' provided the synced metadata is more recent. Furthermore, whatever permissions are required to make it automatic should be requested (and that there should be a means of fixing the lack of such access if it hasn't been granted or was lost).

Test note: this should work for:
a) WiFi Sync
b) USB Sync
c) USB Sync to SD Card or OTG drive that's plugged into MMA
d) Mixed content scenarios e.g
1 User DLs tracks to Android containing lots of metadata (which MMA doesn't see)
2 User Syncs some of those tracks to MMW which sees all of the metadata
3 MMW includes those tracks on the sync list to MMA
--> User would expect to see the metadata from MMW in MMA


Note: once MMA includes native tag support, we may want to rethink the logic slightly

martin

2024-09-18 18:11

developer   ~0077031

No Scan options screen in 1213.
Added "Grant access to storage" button to "Library folders" only if some access permission is not granted.

A user is asked to grant folder access on each mount storage event until he grants access or uses the "skip" button.
V3 scan is launched automatically if fresh data are available and all needed permissions are granted.
Fixed in build 2.0.1.1213

rusty

2024-09-18 21:43

administrator   ~0077037

Tested 1213 and it seems to be working correctly. A couple of questions:
i) If the user clicks the 'Skip' button on startup when folder access is requested, and they _ever_ prompted again to grant access? I'm asking because I would have expected that:
- if MMW is configured with a Library folder that has not been granted access and MMW is configured to sync to that folder, that MMW would at least indicate on the Library Folders screen that there's an access problem, and provide a means of resolving it. Currently there's a vague message :
Grant access to storage
MediaMonkey requires write permission to some folders

But:
- It's not highlighted (i.e. it's easy to overlook)
- it's vague in the sense that it doesn't highlight which of the folders are problematic

ii) 'Grant access to storage' appears in Options > Library and Sync as well, but again, it just appears to be a regular configuration item, when in fact it's a warning. Perhaps the text should be highlighted in some manner and the string changed to 'MediaMonkey requires write permission to these folders to function optimally!' (this change works for the above case as well).

martin

2024-09-19 10:39

developer   ~0077058

i) The user is asked again before the first launch of wifi sync (or after the second step in the sync wizard). The folders in the Access dialog are updated by folders specified in MM5 and yes it would be nice to inform the user on the MM5 side. If the user uses the skip button then he is never asked again until some change happens(new storage, new setting in MM5).


ii)
A) highlight
1) At "Library folders" it's not a problem. We can change it to a warning color and we can also highlight folders.
2) At "Library and Sync" there is an issue. This is the preference screen so there are required only preference items.
We can specify a custom layout for the option, but the general layout has slight differences depending on the Android version.
I will verify the possibilities.

B) text
I believe the description should be shared between the option in "Library and Sync" and "Library scan", so there should not be "these folders", I suggest:
MediaMonkey requires write permission to some folders to function optimally!'

martin

2024-09-20 14:42

developer   ~0077092

What do you think? I believe it's not too aggressive, warning enough though.
Screenshot_20240920-154050.png (98,310 bytes)   
Screenshot_20240920-154050.png (98,310 bytes)   
Screenshot_20240920-154104.png (211,067 bytes)   
Screenshot_20240920-154104.png (211,067 bytes)   

rusty

2024-09-23 20:36

administrator   ~0077109

Looks good, though I would change the text as follows:

MediaMonkey requires write permissions for backup & USB sync. (no need to mention 'MediaMonkey folder' since it already appears in the Title)

MediaMonkey requires folder write permissions to function optimally. (this is more concise and applies to both cases).

martin

2024-09-25 16:24

developer   ~0077117

Fixed in build 2.0.1.1214

peke

2024-09-30 22:10

developer   ~0077152

Verified 1214