View Issue Details

IDProjectCategoryView StatusLast Update
0000217MMW v4Conversion/Levelingpublic2005-09-18 15:27
Reporterrusty Assigned To 
PriorityurgentSeverityminorReproducibilityalways
Status closedResolutionfixed 
Fixed in Version2.5 
Summary0000217: Conversion terminology is incorrect / unclear how to actually convert files in-place
DescriptionThe term 'convert' implies a change to the currently selected item. The actual implementation of the conversion functionality is to generate a converted copy of the currently selected item.

Moreover, with the current conversion dialog it is unclear how to actually convert the currently selected item (does one have to delete the mask entirely? does one have to look up the path to the currently selected file and generate a mask based on that path?)

A minor change to the conversion dialog would correct this:
Title: Choose encoding destination --> Convert Tracks

(o) Convert track(s) to new destination
( ) Convert track(s), replacing the originals

The Mask dropdown should only be active if the first option is enabled. Other aspects of the UI remain pretty much as is.

Other points:
1) If "( ) Convert track(s), replacing the originals" is chosen, then references to the original file (e.g. playlists, VCD) should now be updated to refer to the converted file.

2) Tracks that are converted from an external source (CD) should appear in the 'Added to library' node. Otherwise, tracks should not be added to this node.
Additional Informationhttp://www.songs-db.com/forum/viewtopic.php?t=1938
http://www.songs-db.com/forum/viewtopic.php?t=2077
http://www.mediamonkey.com/forum/viewtopic.php?t=3888
TagsNo tags attached.
Fixed in build891

Activities

jiri

2003-04-03 09:12

administrator   ~0000739

As I wrote elsewhere, I am afraid that it could bring some problems, like accidentally overwriting files with their encoded versions, ... Wouldn't it be enough to properly label the mask edit box? (e.g. as you proposed 'Create converted files in specified location').

rusty

2003-04-03 16:01

administrator   ~0000741

We could do as you propose, however, then by default, the path in the box would have to be the path to the original file, since if the path to the original file isn't there, it becomes difficult for the user to figure out how to replace the original file (since they have to look up the path information somewhere else).

Perhaps a good compromise would be to use the minor change to the dialog that I proposed, but change the default to:
(o) Created converted files in specified location

rusty

2003-04-25 18:50

administrator   ~0000938

This item is now covered in bug #312. Closing.

rusty

2003-05-15 20:17

administrator   ~0001192

Re-opening, since this was never actually fixed as part of bug #312.

rusty

2003-08-11 02:02

administrator   ~0002109

Per user suggestion: if the user chooses to 'Replace selected files with converted files' then all references to the original file should be modified to reference the converted file (e.g. playlists should be updated). I'd consider this to be a less urgent portion of the feature.

This was suggested in: http://www.songs-db.com/forum/viewtopic.php?t=456

rusty

2004-03-19 13:23

administrator   ~0003471

Another related issue is that whenever the converted tracks are added to the library, they should appear in the recently added tracks playlists (currently they don't).

Bug was pointed out by Octopod at: http://www.songs-db.com/forum/viewtopic.php?t=1512

rusty

2004-07-09 14:38

administrator   ~0004375

This issue has come up on several occasions. I've updated the bug to reflect all comments to-date, and raised the priority to Immediate on the assumption that it's easy.

rusty

2004-07-14 02:07

administrator   ~0004381

Another example of confusion & bugginess with the current functionality... (i.e. there's confusion about _how_ to replace the current file, and even when the user does follow a path that should replace the current file, MM fails).
http://www.songs-db.com/forum/viewtopic.php?t=2117

A half-fix would be to fix the bug so that overwrites don't fail. A full fix would be to implement the solution described in the bug description.

rusty

2005-07-21 03:26

administrator   ~0005694

Assigned to Ludek.

rusty

2005-09-18 15:27

administrator   ~0005846

Verified 895.