View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0010970 | MMW v4 | Synchronization | public | 2013-06-07 21:24 | 2013-09-13 02:45 |
| Reporter | rusty | Assigned To | |||
| Priority | urgent | Severity | major | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Fixed in Version | 4.1 | ||||
| Summary | 0010970: Sync logic problems MMA 139 / MMW 1642 | ||||
| Description | When testing MMA 139 against MMW 1642, I founds several sync problems, though it's unclear to me where the bugs lie: 1 Some metadata syncs bidirectionally but other metadata doesn't when bi-di sync is disabled. e.g. - re-ordering tracks on an MMA playlist isn't synced - play history from MMA is synced - ratings changes in MMA are synced I'm not sure whether this makes sense or not... 2 Playback of tracks in MMW --> failed to sync the #Played to MMA (note: this was tested by playing a track in MMW and syncing to MMA, and then playing the same track in MMA, and resyncing, and observing that the #played only shows 1 rather than 2) 3 Syncing via Wi-Fi and USB yield different behaviors with respect to removal of playlists. -WiFi sync: playlists remain on the device even when 'Delete other files and playlists from the device' is enabled. -USB sync: confirmation dialog appears Is this intentional? | ||||
| Tags | No tags attached. | ||||
| Fixed in build | 1644 | ||||
|
|
Reminder sent to: Ludek Ludek, please have a look. I'm not sure if this should be on your plate or marek's. |
|
|
Re: 1) Yes, this has always been the case also for iPods/iPhones that rating/playcount have been always bi-di synced. As I wrote here 0010689:0035549 my opinion is that there should be a config in [Auto-Sync(-->Library)] tab in MMW and in MMA/MM8 options: [x] Auto-Sync file metadata from the device to the PC Re: 2) This should definetly work. Could you write down eaxct steps to reproduce? Did you synced over WiFi or USB? Are you sure that the track was fully played? Re: 3) - WiFi sync: Sounds like a bug, but I believe that it is tracked as another issue, Marek? - USB sync: yes, this is intentional once '[x] Confirm deletion (recommended)' is checked |
|
|
1) So you're suggestion that (this would eliminate confusion about why playlists/ratings change in MMW even when bi-di sync is disabled): '[ ] Auto-sync file metadata from the device to the PC' should apply to existing files on the MMW auto-sync list? '[ ] Auto-sync files and playlists from the device to the PC' should apply only to new files in MMA? And that both should be enabled by default? Seems reasonable, BUT, if we do this, we'd have to make this work for iPods/iPhones as well AND we'd have to modify the MMA/MM8 UIs as well. So perhaps it should be deferred. 2) Cannot replicate. possibly a refresh issue. 3) Better tracked at 0010689 |
|
|
1) I think that iPods/iPhones are quite different, the [Auto-Sync(-->Library)] tab is missing entirely for Apple's devices for several reasons: a) the folder structure on iPods/iPhones is hidden and in a non-human readable form, therefore the folder config is useless for iPhones b) iPods/iPhones bi-di sync only Playcounts/Rating c) only newer iOS5+ devices supports bi-di playlist sync Because there is a very limited set of metadata to bi-di sync then the tab is missing and bi-di sync is enabled by default for Apple's devices But for Android devices there is a different situation, we bi-di sync much more stuff (all file metadata like Album, new files on device, new/modified playlists) therefore I think that user should have a possibility to configure what he wants to bi-di sync and what he doesn't wish to sync back. I think that most users would like to bi-di sync file metadata like rating/playcount, but not new files or playlist modifications. Therefore I think that we should make it configurable. The question is how much of granularity we want to involve. |
|
|
1) OK. So we'll go with the approach suggested at 0010970:0036441 |
|
|
thinking about it further and from MMW perspective (the [Auto-Sync(-->Library)] tab) then I suppose that something like this would work better (modified current layout): [Auto-Sync(-->Library)] [ ] Copy new files from the device to the PC -------------------------------------------- |-[x] Music |--[x] Audiobooks |--[x] Classical |--[x] Podcasts |[x] Video |--[x] Music Video |--[x] Podcasts |--[x] TV --------------------------------------- Destination [c:\Users\<user>\Music\] [ ] Auto-sync playlists from the device to the PC [x] Auto-sync file metadata from the device to the PC i.e. there would be 3 checkboxes, the first for new files, the second for playlists, and the third for file metadata. The problem is where to place '[x] Confirm Updates' checkbox, because in case of file metadata (e.g. playcount) we currently doesn't show any confirmation. The question is whether we should show confirmation also in this case? Note that this layout seems to be more logical also for non-android devices (or android devices without MMA installed). For such a devices we would just hide the second and the third checkbox. |
|
|
I like the layout, but: - this would also imply some changes to the MMA/MMW UIs (i.e. currently there's only a single entry for both Files and Playlists. - I don't think there's a need to confirm metadata updates. For this reason, I would suggest that we implement fewer changes for now--e.g. something like: [Auto-Sync(-->Library)] [ ] Sync new files and playlists from the device to the PC ---------------------------------------------------------- |-[x] Music |--[x] Audiobooks |--[x] Classical |--[x] Podcasts |[x] Video |--[x] Music Video |--[x] Podcasts |--[x] TV --------------------------------------- Destination [c:\Users\<user>\Music\] . . [x] Confirm updates (recommended) [x] Auto-sync file metadata from the device to the PC On the other hand, if Marek/Petr think that it would be trivial to make the corresponding changes on the MMA/MM8 client side, then we could implement your proposal as follows: [Auto-Sync(-->Library)] [ ] Copy new files from the device to the PC -------------------------------------------- |-[x] Music |--[x] Audiobooks |--[x] Classical |--[x] Podcasts |[x] Video |--[x] Music Video |--[x] Podcasts |--[x] TV --------------------------------------- Destination [c:\Users\<user>\Music\] . . [x] Confirm updates [ ] Auto-sync playlists from the device to the PC . . [x] Confirm updates [x] Auto-sync file metadata from the device to the PC |
|
|
OK, so lets go with the "fewer changes" approach for now and we can split the configs later (in case of need). Note that there is also a good reason to keep the common config for new files and playlists, because if a playlist contains new files then it doesn't make much sense to copy the playlist without the new files. |
|
|
Added in build 1644. |
|
|
Verified 1657 |
