View Issue Details

IDProjectCategoryView StatusLast Update
0012962MMW v4Synchronizationpublic2016-02-02 10:38
Reporterrusty Assigned To 
PriorityimmediateSeveritymajorReproducibilitysometimes
Status closedResolutionfixed 
Product Version4.1.9 
Target Version4.1.10Fixed in Version4.1.10 
Summary0012962: MMA fails to update playlists synced via USB
DescriptionFollowing the resolution of 0012948 (failure of MMW to recognize Android devices that have MMA 1.1.3 installed), 2 users have indicated that even though USB synchronization is now working correctly, some metadata fails to appear in MMA:

Ratings don't appear: http://www.mediamonkey.com/forum/viewtopic.php?f=21&t=82921#p416398

Playlists fail to update: BGI-133-36226
Moreover, when playlists do partially update, playlist hierarchy is lost.
TagsNo tags attached.
Attached Files
S3_logs_bug_12962.zip (170,465 bytes)
Fixed in build1769

Relationships

related to 0012299 resolvedmarek MMA Improved external storage handling on Android L 
related to 0012919 resolvedmarek MMA Playlists are getting deleted on upgrade to 1.1.3 for some users 
related to 0012973 resolvedLudek MMW v4 Enforce sync mask causes artwork to be lost in MMA 
child of 0012848 closedmarek MMA Container playlists and their children don't sync (regression) 

Activities

rusty

2015-11-16 20:28

administrator   ~0043319

I just synced MMA build 502 (Samsung Tab 8/Android 4.4.2) vs MMW 4.1.10.1666, and following the USB sync operation, all playlists from the device except for 1 don't appear. Debug log: PHMSTUR8LH

Also, I've posted the debug log to this bug.

Ludek

2015-11-16 20:35

developer   ~0043320

Last edited: 2015-11-16 23:14

From MMW log I see that MMW uploaded the synced metadata to \MediaMonkey\files\mmstore.db.synced.0012 , it looks like MMA is not processing the synced metadata anymore for some reason?

rusty

2015-11-16 22:39

administrator   ~0043323

Note: failure of MMA to display playlists only occurs sometimes (i.e. resyncing via USB a second or third time resolves the problem).

Ludek

2015-11-16 22:57

developer   ~0043324

Last edited: 2015-11-16 23:17

I tested MMA 501 and MMW 1766 and had no issues with updating metadata and playlists hierarchy over USB.

User from ticket BGI-133-36226 indicated that he needs to re-install MMA to force it to pick up the updated metadata.

rusty

2015-11-17 03:56

administrator   ~0043325

Last edited: 2015-11-17 04:04

I've been able to replicate failure to sync playlists via USB on a Galaxy Tab 12 running Android 5.1 as well. The Sync list consists of the following playlists:
a) Ari's Bar Mitzvah
b) -Bar Mitzvah 1 - beginning
c) -Bar Mitzvah 2 - dance
d) -Bar Mitzvah 3 - Background
e) -Bar Mitzvah 4 - end
f) Billboard

On the initial USB Sync (-3280 in the debug log, and MMA Debug 3547G7Y08J): only b,c,e appear in MMA (and in MMW after refreshing the node).

Second USB sync (-20819 in the debug log, and MMA debug KXT6FQ6N9R): d,f appear in MMA (and in MMW after refreshing the device node).

Third USB sync (-25217 in the debug log, and MMA debug 3QMUS0V3RN): b,c,e appear in MMA (and in MMW after refreshing the device node).

Note: when I look at the device file hierarchy, I can see:
/Device Storage/Playlists: Billboard.pla , Bar Mitzvah -3 Background.pla
/SD Card/Playlists: Billboard.m3u, Bar Mitzvah - Background.m3u
Also, all music is stored to /SD Card/Music (which makes sense since this is a lollipop device)

Note:
- debug log is posted to the bug
- after numerous such failed sync operations a successful one occurred (nothing different was done) and MMA showed all of the playlists. When this occurred, the device file hierarchy was:
/Device Storage/Playlists: Bar Mitzvah - 4 end.pla, Bar Mitzvah - 2 Dance.pla, Bar Mitzvah - 1 Beginning.pla, Billboard.pla , Bar Mitzvah -3 Background.pla
/SD Card/Playlists: Bar Mitzvah - 4 end.mu3, Bar Mitzvah - 2 Dance.m3u, Bar Mitzvah - 1 Beginning.m3u
- also as discussed, usb syncs always fail to sync playlist hierarchy (in contrast to wi-fi sync)

rusty

2015-11-17 14:25

administrator   ~0043326

New user report of what sounds like the same issue:
VKS-126-75698

rusty

2015-11-17 20:32

administrator   ~0043327

I've replicated the issue again with a Galaxy S3 running Android 4.4.2 as follows:

1) Synced 6 playlists via Wi-Fi: YUOADO35SN
a) Ari's Bar Mitzvah
b) - Bar Mitzvah 1 - beginning
c) - Bar Mitzvah 2 - dance
d) - Bar Mitzvah 3 - Background
e) - Bar Mitzvah 4 - end
f) New Pop 2.5+
g) Test Playlists
h) - Bronski test 111
i) - Test formats
--> Content appears correctly in MMA

2) Initiated USB sync (no changes to the sync profile) --> MMW prompts to delete g) Test Playlists, and proceeds with sync. Ran MMA
--> Playlist node is empty! (but all content appears as expected)
/Local Storage/Playlists contains .pla files for all of the playlists, but they're all 0B
/SD Card/Playlists contains no files
MMA debug log: YY5NDMJX08

3) Initiated Wi-Fi sync
--> Playlists appear correctly even though storage of playlists is identical to above:
/Local Storage/Playlists contains .pla files for all of the playlists, but they're all 0B
/SD Card/Playlists contains no files
Debug log: 2PI3SMLYP1

Ludek

2015-11-18 10:25

developer   ~0043329

Last edited: 2015-11-19 12:32

From the "S3_logs_bug_12962.zip" I see that MMW uploaded the playlists to the MMA app specific folder like:
\Android\data\com.ventismedia.android.mediamonkey\files\Playlists\Bar Mitzvah - 2 Dance.m3u
because the SD card has limited write access according to
\Android\data\com.ventismedia.android.mediamonkey\files\files\storageInfo.xml

I see that all the mentioned 1a) .. 1i) were successfully uploaded with exception to 1g) "Test Playlists.m3u" that was deleted (0012966)

EDIT: As found over IM, MMA is deleting the playlists due to a regression caused by the change suggested in 0012848:0042945

marek

2015-11-18 17:43

developer   ~0043334

Fixed in build 504

rusty

2015-11-18 20:19

administrator   ~0043335

Last edited: 2015-11-18 20:45

Tested 504 and it still deletes playlists following a USB sync:
S3/kitkat: After initial scan, all playlists but 'New Pop 2.5+' are deleted: 2QJOXKX1E0
Tab 12/lollipop: After initial scan, all playlists but 'Billboard' are deleted: U4FYKJFXA3

rusty

2015-11-19 22:38

administrator   ~0043342

Test results with build 505b:
USB Sync to the Tab 8 --> all hierarchical playlists (parent containers and child playlists) are deleted from the device.
Debug log: 0C4P02LR0S

marek

2015-11-21 08:05

developer   ~0043359

This is fixed on MMA side in build 506 but there still remains one issue... There are following sync operations in DB:
1.delete the container playlist X
2.insert the container playlist X

After this, all subplaylists has invalid parent playlist because the new inserted playlist has new id. Why there isn't one update of playlist ?

Or it would be correct, if MMW updated also all childrens. I am not able to fix it in MMA DB because the mmstore.db.synced contains this inconsistence too.

See this screenshot of DB: aaa Hiearchic and aaa HierarchicX Empty are containers that were removed and inserted and has id 11 and 12 but parent ids are still 1 and 8. Only Subplaylist B was udated.

https://www.dropbox.com/s/yepm3gbdwijn0dp/Screenshot_2015-11-21-08-59-09.png?dl=0

rusty

2015-11-22 02:51

administrator   ~0043364

I'm not quite sure that I understand comment 0012962:0043359, but I tested build 506 and the test results seem to be the same as with build 502. i.e. Wi-Fi Sync generally works (see exception below), but USB sync fails for hierarchical playlists exactly as before.

S3:
1 Wi-Fi Sync --> Success. Debug log: 0IA773LI1V
2 USB Sync performed immediately after Wi-Fi sync
-->Failed--container playlists and their child playlists were deleted. The only one that remained was 'New Pop 2.5+'. Debug log: 8T10WBC6GZ
3 Delete 'New Pop 2.5+' in MMA and then perform another USB sync
-->Failed--Ari's Bar Mitzvah container (but not children) and 'New Pop 2.5+' synced, but nothing else did. Debug log: 4QH2QW152S

Tab 8.4:
1 Wi-Fi Sync:
-->Failed--for hierarchical playlists only the parent container synced--the child playlists don't appear (the containers are 'Ari's Bar Mitzvah' and 'Test Playlists'). Debug log: S7CI1BT8Q4
note: I'm not sure why this occurred, but see tests 2 and 4 below--for some reason, this failure didn't reccur.............
2 Delete the empty container playlists from Step 1 and initiate Wi-Fi sync again
--> This time Wi-Fi sync operation is successful. Debug log: SHCBZURMDK
3 Perform USB Sync
-->Failed. Container playlists and their child playlists were deleted. The only ones that remained are the non-hierarchical playlists 'Billboard' and 'Good stuff 4+'. Debug log: YQB9XLO61H.

marek

2015-11-22 17:13

developer   ~0043366

Your logs confirmed it:

In first attempt, where container is missing:

There are following sync operations for container playlists:
1.DELETE
2.INSERT
3.UPDATE

Because we take type only from the first sync operation, only DELETE is done.

It should be UPDATE only!

In second log, it is as described in 0012962:0043359 . Container remains there but is empty. This occurs when sync of multiple playlists is done, i.e. DELETE and INSERT of one playlist is separated by another operation :

1.DELETE Playlist A
2.DELETE Playlist B
3.INSERT Playlist A
4.INSERT Playlist B

Btw. the sub-playlists are not deleted, they are in DB but they only have invalid parent id so they are not visible in library. This is visible on screenshot in 0012962:0043359.

I will add some improvements to build 507. I think that these invisible playlists should be removed when they occurs in MMA and MediaStore. Now MMA deletes them but they are resynced from MediaStore. So users will have to just resync after update to 507 - not other actions will be needed.

rusty

2015-11-22 21:20

administrator   ~0043369

Last edited: 2015-11-22 23:56

Tested 507, and confirmed that now Wi-Fi sync will always sync playlists correctly--regardless of whether a USB sync was done beforehand or not. (In contrast, with build 502, Wi-Fi sync that is done subsequent to USB sync could fail to correct playlists that were incorrectly synced over usb.)

What remains to be fixed is for USB sync to correctly sync hierarchical playlists.

Note: more info from rklevy at http://www.mediamonkey.com/forum/viewtopic.php?f=21&t=82986&p=416620#p416551

Ludek

2015-11-23 12:13

developer   ~0043373

Last edited: 2015-11-23 12:18

I can confirm that there was also a regression on the MMW side caused by 0012853 , fixed in 4.1.10.1768

rusty

2015-11-23 14:01

administrator   ~0043378

Peke, when you regression test MMW, please ensure that there aren't any regressions related to failed ratings updates ( http://www.mediamonkey.com/forum/viewtopic.php?f=21&t=82921#p416398 ), which was initially raised in this bug, but never replicated.

rusty

2015-11-24 05:08

administrator   ~0043383

Tested MMA 507 vs MMW 4.1.10.1768, and there still seem to be some issues with USB sync.

S3/Android 4.4.2:
0 Delete playlists from MMA
1 Wi-Fi Sync Ari's Bar Mitzvah + 4 child playlists, New Pop 2.5+, 2 Playlists that are children of Test Playlists
--> Success. Debug log: LVMM8CF1UK
2 Perform USB sync (even though files already exist on the device)
-->Success (i.e. no playlists are deleted). Debug log: DKW9G3BQM5. DBGVIEW logs posted.
3 Delete playlists on device, and perform Wi-Fi sync.
-->Failure. For some reason, playlists take several minutes to sync. Once complete, playlists node is empty. Debug log: UG6IAAUWDC


Tab 8/Android 4.4.2:
0 Delete playlists from MMA
1 Wi-Fi Sync Ari's Bar Mitzvah + 4 child playlists, Billboard, New Pop 2.5+, 2 Playlists that are children of Test Playlists
--> Success. Debug log: DEG7S63U1Q
2 Perform USB sync (even though files are already on the device)
--> Success (i.e. no playlists are deleted), though it takes about 8 minutes to complete. Debug log: 2C40JDD6V8 . DBGVIEW logs posted.
3 Delete playlists on device, and perform Wi-Fi sync.
--> Failure. Wi-Fi Sync Ari's Bar Mitzvah + 4 child playlists, Billboard, New Pop 2.5+, synced, however, only 1 of the 2 Playlists that are children of Test Playlists synced (High bitrate mix), and it didn't display in the correct hierarchy (it displayed at the root playlist level rather than as a child of 'Test Playlists'. I suspect that the problem might be related to the fact that the child playlists are selected, but the parent isn't.
Debug log: AN7BQ5BZZR. DBGVIEW logs posted.

Alcatel Idol 3 / Android 5:
Test results same as for the Tab Pro 8. i.e. with USB sync not all playlists successfully synced: Ari's Bar Mitzvah + 4 subplaylists synced, Billboard synced, but Shlomo Katz, and 2 Playlists that were children of Test Playlists failed to sync.

Summary: it seems to be better than in build 1767, but playlists are still not syncing correctly.

Ludek

2015-11-24 09:03

developer   ~0043385

Last edited: 2015-11-24 09:05

As discussed with Marek offline, this slightly different use case (unchecked parent playlist, but checked child) caused that MMW did not fill the _data column (path) for the parent playlist and MMA has failed, because MMA always expect to have path filled up (even for playlists without M3U file) so that MMA knows where to create the M3U once a track is added to the playlist in MMA.

Ludek

2015-11-24 09:13

developer   ~0043387

To correct Rusty: In the steps 3 it was actually USB sync and not WiFi sync

Ludek

2015-11-24 11:13

developer   ~0043388

Last edited: 2015-11-24 11:29

I added syncing of unchecked parent for checked child in build 4.1.10.1769 and 5.0.0.2007 , nevertheless there still needs to be an issue in MMA.

MMA stopped syncing the hierarchy also for checked parents with tracks and M3U file for some reason although it worked fine previously and nothing changed by MMW.

Solving with Marek...

EDIT: Suddenly it started to work, the only change was restarting of MMA.
Resolving for testing as I can no longer reproduce and the issue observed by Rusty ( 0012962:0043383 ) is resolved.

rusty

2015-11-24 19:48

administrator   ~0043393

Verified 1769.