View Issue Details

IDProjectCategoryView StatusLast Update
0013122MMASynchronizationpublic2016-06-28 23:42
Reportermarek Assigned To 
PriorityimmediateSeveritymajorReproducibilityrandom
Status closedResolutionfixed 
Product Version1.1.3 
Target Version1.2.0Fixed in Version1.2.0 
Summary0013122: Size of artworks and tracks differs on some devices with Marshmallow
DescriptionWe have some logs that indicates that the size of downloaded artworks are different than original. Observed only on Android 6.0 devices.
TagsNo tags attached.
Fixed in build593

Relationships

related to 0013244 closedmarek File browser use deprecated java.io.File interface 
related to 0013246 closedmarek Rework the way how the path to file is stored to DB and preferences 

Activities

marek

2016-02-10 21:36

developer   ~0044119

Added logs to build 547 to verify the size

marek

2016-02-10 21:38

developer   ~0044120

https://rink.hockeyapp.net/manage/apps/64672/app_versions/347/crash_reasons/110376365

marek

2016-02-11 09:42

developer   ~0044123

Peke has Moto G 2014. So he might be able to reproduce it. The OS version should be 6.0.1.

Peke, please try wifi sync to download some tracks with artworks and then immediately send me the logs.

peke

2016-02-11 21:51

developer   ~0044139

Marek, my MotoG LTE is classified as 2013 Model so I'm stuck to 5.1 till I'll be able to install CM13

Test like you pointed is not reproducible on 5.1 device.

marek

2016-03-04 14:02

developer   ~0044243

I have reproduced it on nVidia shield tablet. It is caused by SAF API that works differently on Marshmallow ... again. Generally it take some time (approximately 10 seconds) to write the file to SD card and to update SAF DB. So when MMA downloads file with length 1000 B and then it overwrites it with new file with length 1500B, it takes 10 seconds to update the length. It is probably somewhere cached and the update is done asynchronously later.

This is quite bad because our workflow is not designed for this. There is no listener that we could use to for waiting till it is done. I would need to implement our workaround that will somehow determine that all files are written and that wifi sync can stop.

Btw. it can cause that tracks are redownloaded when playcount is updated even it is not needed. Only the size is incorrect.

marek

2016-03-04 16:23

developer   ~0044247

Added question to StackOverflow:

http://stackoverflow.com/questions/35800564/android-marshmallow-files-written-by-saf-are-not-immediatelly-written

marek

2016-04-05 09:05

developer   ~0044380

Fixed in build 589

Removed using of java.io.File class that caused delays in SAF api updates.

peke

2016-04-12 00:11

developer   ~0044429

Verified 589 no regressions on my devices.

marek

2016-04-19 19:56

developer   ~0044466

There are still many occurrences of File class that has to be replaced by new interface

marek

2016-04-19 20:09

developer   ~0044467

Last edited: 2016-04-19 20:13

Many fixes were made in build 591

A lot of things that were related to file processing had to be changed.
There are following things to test:
1. Library folders - adding and scanning library folders
2. Wifi sync - media files, artworks and playlists files should be properly stored and visible in library
3. Wifi sync - synchronization of bidirectional and scanned folders with MMW. I mean that folders that are added to library in MMA should be added to profile settings in MMW
4. Video thumbnail generation
5. storing of storageInfo.xml and .nomedia files to app specific folder or to /MediaMonkey folder (depends on storage type)
6. All device storages should be visible in Options, should be tested on as many devices as possible

All these tests should be done on different storage types:
a. Internal storage on any api
b. SD card on 4.4
c. SD card on 5.x
d. Internal storage on 6.x
e. SD card on 6.x

Please note that Folder browser is still not reworked. So it is not necessary to test it.

marek

2016-05-02 20:28

developer   ~0044532

Fixed in build 593

peke

2016-06-28 23:42

developer   ~0045065

Verified 607