View Issue Details

IDProjectCategoryView StatusLast Update
0010989MMASynchronizationpublic2013-08-09 01:11
ReporterLudek Assigned To 
PriorityurgentSeverityminorReproducibilityhave not tried
Status closedResolutionfixed 
Product Version1.0.4 
Target Version1.0.4Fixed in Version1.0.4 
Summary0010989: Implement sync confirmations for bi-di sync
DescriptionWe have implemented confirmations for playlists and files in case of USB sync when following is enabled on device profile:

[x] Auto-sync files and playlists from the device to the PC
....
[x] Confirm Updates


Example confirmation of USB sync (build 1642):

[x] New file(s) and playlist(s) to copy from the device to the PC (2)
--[x] Playlist: PLST2
--[x] \Music\Air\10000 Hz Legend\05 - Radian.mp3
[x] Existing file(s) and playlist(s) to update from the device to the PC (1)
--[x] Playlist: PLST1


We should add similar confirmations in case of WiFi sync, more details here 0010689
TagsNo tags attached.
Fixed in build146

Relationships

related to 0009692 closedmartin MMA Improved delete confirmation dialog that allows for deletion of tracks from Playlists 
related to 0010990 closedmartin MMA Optimize/Simplify delete confirmation dialog 
related to 0010992 closedmarek MMA Empty playlists remain in MMA in cases where they should be deleted 
related to 0010993 closedmarek MMA MMA fails to update playlists in response to removal of a track 
related to 0011140 closedLudek MMW v4 Metadata fail to sync back for read-only files 
child of 0010689 closedLudek MMW v4 Auto-sync (-->Library) doesn't work as expected for playlists 

Activities

Ludek

2013-06-13 10:13

developer   ~0036478

Note that there are also some bugs that needs to be fixed/verified: 0010689:0036471

rusty

2013-06-13 18:29

administrator   ~0036491

We currently have the following strings:
Confirm deletion
Delete the selected item/s from the device?
....
....
[Yes] [No]

It would probably make sense to have a single sync confirmation dialog so that the user doesn't have to accept multiple dialogs. This could be something like:

Confirm updates & deletions
[ ] Delete Tracks/Videos (<#>)
--[x] <Artist> <Title>
--[x] <Artist> <Title>
[x] Copy new files and playlists from the device to the PC (<#>)
--[x] Playlist: PLST2
--[x] \Music\Air\10000 Hz Legend\05 - Radian.mp3
[x] Update existing files and playlists from the device to the PC (<#>)
--[x] Playlist: PLST1
--[x] \Music\Air\10000 Hz Legend\05 - Radian.mp3
[Yes] [No]

Note: it might make sense to split files and playlists

marek

2013-06-14 21:37

developer   ~0036518

This general dialog for confirmation of everything is imho not a good idea.

There are too many lines and information for small phone display. I know that you want to make it similar to MMW but this layout is to complicated for display. It is most probably unusable even for medium size screens. I think it should be more adjusted and optimized.

The main problem is that the display can sometimes display only 6 lines of list (my xperia x10 mini displays even only 3). When you scroll the list and you have like 50 tracks for each category, it is hard to know if you currently check item from second or third category...because the list is simply too long. It is hard to operate even with current, one category, implementation

I think that it would be probably better to use on of these approaches:
1. Three simple separate dialogs (I know that you don't want the user to confirm each dialog but it is clear for the user)

Confirm delete of tracks[select all button]
-------------------------------------------
[ ] <Artist> <Title>
[ ] <Artist> <Title>

[Yes] [No]
.. similarly for other dialogs

2. Use viewpager, i.e. three pages in one dialog, user can switch between these dialogs by swiping. And it will be clearly visible that there are multiple pages

Confirm delete [select all button]
-----------------------------------
        Delete tracks | Copy f
--------=============--------------
[ ] <Artist> <Title>
[ ] <Artist> <Title>

[Yes] [No]

The second line represent bar similar to bar in play store application.

rusty

2013-06-14 22:52

administrator   ~0036521

Last edited: 2013-06-17 14:19

So if we took the viewpager approach it could appear as:

Confirm Updates & Deletions
Deletions | Updates | New files

Descriptive text for each viewpager screen:
Deletions: Items to delete from the device
Updates: Items to update on the PC
New files & playlists: Items to copy to the PC

Note: I assume that there's a single Yes | No for all three items (rather than three separate yes/no prompts).

marek

2013-06-22 00:04

developer   ~0036625

in build 144:

- new dialog was implemented
- confirmation of media/playlists deletion, media updates implemented
- changed synchronization workflow to support common confirmation dialog

marek

2013-06-22 14:23

developer   ~0036634

in build 145:

- some style improvements and fixes of dialog

What is missing:
- confirmation of playlist updates and uploads

marek

2013-06-26 10:01

developer   ~0036661

Implemented in build 146 at MMA side:

But confirmation of uploads/updates to MMW is not sent from MMW to MMA in settings XML. I added it as "UpdateConfirm" element.

Ludek

2013-06-26 18:07

developer   ~0036663

Last edited: 2013-07-01 15:36

There's always been <BiDirConfirm> element for this, and newly (from build 1644) there is also new element <BiDirSyncMetadata> for file metadata (rating, playcount etc.)

marek

2013-06-26 21:49

developer   ~0036666

Ok, I didn't see it.

Fixed in build 146

peke

2013-06-28 19:53

developer   ~0036674

Verified 146, but I encounter one issues where Read only files are updated which initiate additional Confirmation for updating Tracks and sync can't continue unless it is clicked.

My best guess is that those files are updated in Library but not physically.

marek

2013-07-01 11:28

developer   ~0036696

Could you please specify the steps to reproduce it?

Update of tracks doesn't have any confirmation...this is implemented only for playlists. Only upload of media to MMW and deletion of track in MMA has to be confirmed (if configured).

peke

2013-08-09 01:10

developer   ~0037121

It is not MMA issue but rather MMW issue.

Added steps in 0011140

peke

2013-08-09 01:11

developer   ~0037122

Last edited: 2013-08-09 01:11

Verified 1652 and 156