View Issue Details

IDProjectCategoryView StatusLast Update
0001350MMW v4Synchronizationpublic2013-12-12 21:52
Reporterrusty Assigned To 
PriorityurgentSeverityfeatureReproducibilityalways
Status closedResolutionfixed 
Target Version4.1Fixed in Version4.1 
Summary0001350: Portable Device: Bidirectional Synchronization
DescriptionUsers sometimes want to be able to synch tracks that are new to their devices to the library, and/or to delete tracks from their library if they've been deleted from the device.

This can be implemented via a change to the bottom of the synchronization panel as follows:

(o) Synchronize to device uni-directionally
. . . [x] Delete Tracks on device if not in synchronization list
( ) Synchronize bidirectionally
. . . [x] Download Tracks from device if not in library
. . . . . Store tracks to: _______ [browse]
. . . [x] Update properties from device, if newer

Note: there may be a need to make this configurable via the Portable device combo bar. e.g. Synchronize Unidirectionally vs Synchronize Bi-directionally
Additional InformationOriginally raised and discussed in greater detail in #419 and 419.txt (attached to #419) (db issues + similarity to new vcd functionality).

Make sure to review JBSynch prior to implementation:
http://www.cflashsoft.com/jbsynch.htm
TagsNo tags attached.
Fixed in build1600

Relationships

related to 0008034 closedjiri MMW v4 d_WMDM plug-in doesn't allow synchronization to PC 
related to 0008804 closedLudek MMW v4 Sync Selected can be confusing 
related to 0008808 closedLudek MMW v4 Improved bi-di sync implementation 
related to 0009368 closedLudek MMW v4 Improved UI for bi-di sync 
has duplicate 0001662 closedrusty MMW v4 Bi-directional Synchronization should synchronize _all_ files from the device 
related to 0002479 resolvedrusty MMW v4 MTP Device Synchronization: doesn't support synching tracks from the device to the PC 
related to 0002520 closedjiri MMW v4 Regression: iPod metadata (ratings/playhistory) fails to synch bi-directionally for manually synched tracks 
related to 0002615 closedLudek MMW v4 MTP Synchronization is MUCH slower than it should be 
related to 0003210 closedLudek MMW v4 Audio files are deleted when they shouldn't be deleted 

Activities

rusty

2006-03-20 13:43

administrator   ~0007083

Note: User recently wrote in with a usecase where the lack of this functionality is problematic: The synch tracks to their device, and then delete tracks that they don't like. When they resynch, they want the deleted tracks to also be removed from their Library.

rusty

2006-06-08 18:10

administrator   ~0007333

Requested by Avanquest.

jiri

2006-12-11 15:50

administrator   ~0008262

Some notes:
 1. I think that
   [x] Delete Tracks on device if not in synchronization list
is needed for both options, i.e. also for Synchronize bidirectionally, because user might want to copy new tracks from device to PC, but also remove them from device then.

 2. [x] Update properties from device, if newer
probably isn't needed. E.g. due to specifics of iPod, this has to be done, otherwise the information is lost (files carrying the info have to be deleted). I.e. this should be done automatically.

 3. Re. Usercase where tracks should be deleted from PC if they were previously deleted from Device - I'm a little afraid of this, this could easily result in data loss, even if there is a confirmation dialog.

 4. After the changes I described above
[x] Download Tracks from device if not in library
probably isn't needed, it should happen always if bidirectional synch is selected, only the target path should be configurable. Note: No mask configuration seems to be necessary, tracks would go to a folder and user could configute to automatically organize tracks that appear in a folder on PC (and AutoOrganization should support an automatic organization of tracks in a specified folder only then?).

rusty

2006-12-12 19:47

administrator   ~0008286

1. agreed.
2. ok.
3. I think that we can still provide this as long as:
 a) when the user selects it they are given a warning that this can delete their entire library
 b) when the confirmation dialog appears, users are forced to
 . . i) select the tracks to be deleted (via a select all checkbox in the header)
 . . ii) default confirmation is 'no'
 . . iii) there's no 'don't show this again' option
4. Agreed.

In light of your feedback, I would suggest an alternate UI that I think is simpler and more compact. Note however, that we could just as easily use the current UI without any changes except for the addition of a couple of lines re. deletion of tracks from the PC. I much prefer this though:

Auto-Synchronization Settings Confirm
[x] Sync new/updated tracks on Auto-sync list to device
    [x] Delete tracks not on the Auto-sync list from the device [x]
    Except folders: _______________________________________
[ ] Sync new/updated tracks from the device to the PC
        Save tracks to: _______________________________________ [browse]
    [ ] Delete tracks not on the device from the PC [x]

jiri

2006-12-12 20:02

administrator   ~0008287

3. Ok, with such an increased security it would probably be possibly to implement. With lower priority though, I don't think it will be too often used feature...

5. Re. new UI:

I see the logic why you included:

[x] Sync new/updated tracks on Auto-sync list to device

however, I'm afraid that it would cause problems - if user unchecks this option, clicking Auto-synchronize wouldn't do anything, i.e. tracks wouldn't be uploaded to device. Seems to be unnecessarily confusing to me.

rusty

2006-12-12 20:16

administrator   ~0008289

Last edited: 2007-07-05 20:21

5. There's a scenario where this is useful: if the user is copying tracks from a friend's device to their PC.

If you disagree with this usecase, an easy fix would be to simply eliminate the checkbox to the left of "Sync new/updated tracks on Auto-sync list to device"

btw: see bug 0002615

rusty

2011-09-06 13:05

administrator   ~0027466

6. User pointed out the fact that MM assumes that the portable device only has a single directory for music, and that when Music store apps run on a smartphone, they'll often save to a specific directory (e.g. /AmazonMP3) other than /Music.

We'll need to support this scenario.

http://www.mediamonkey.com/forum/viewtopic.php?f=6&t=60236

jiri

2012-03-02 13:21

administrator   ~0030600

This seems to be ready for implementation, just:

5. I think that either way is ok, we probably could start without the checkbox in order to make it easier.

6. This doesn't mean any problem, since we plan to scan the whole device anyway, right?

Ludek

2012-03-04 23:29

developer   ~0030610

Last edited: 2012-03-04 23:33

Bi-directional synchronization has been implemented as part of issues #8677 and 0008808

The resulted UI is:

Auto-sync files from the PC to the device
 . Delete files and playlists not included in the Auto-Sync list from the device
 . . [x] Library files
 . . . [x] Confirm deletion
 . . [ ] Files not included in the Library
 . . . [x] Confirm deletion (recommended)

[x] Auto-sync files from the device to the PC
 . . 'Destination: ___________________ [Browse]'
 . . . [x] Confirm updates (recommended)

Exclude Folders: ________________________________________________

peke

2012-12-19 23:59

developer   ~0034038

Verified 1615 close if this is final UI.

Ludek

2013-12-12 21:52

developer   ~0038744

No, this was not the final UI, but it was solved as another issue 0008808 , closing this one