View Issue Details

IDProjectCategoryView StatusLast Update
0003021MMW v4Properties/Auto-Toolspublic2007-06-15 19:50
Reporterrusty Assigned To 
PriorityimmediateSeveritymajorReproducibilityalways
Status closedResolutionfixed 
Fixed in Version3.0 
Summary0003021: Small tool to fix flac tags broken by MM 2.5.5.996
DescriptionAs proposed by Peke, we need a small tool to help users who have damaged flac files from MM 2.5.5.996.

Ideally, the tool would be installed as a MediaMonkey script, and accessed via the scripts menu.

e.g.
1) User installs flacfix script
2) User selects all flac tracks and then clicks Scripts > flacfix
-->flacfix goes through the tracks one by one, detects any that are broken, and then fixes them.
Additional Informationhttp://www.mediamonkey.com/forum/viewtopic.php?t=16331
TagsNo tags attached.
Fixed in build1035

Activities

peke

2007-04-26 20:58

developer   ~0009066

Agreed on making script that will work within MM.
Jiri and I talked on IM and it should be Small commandline App that will fix the issue and MM script Will be Front End.

peke

2007-05-07 01:08

developer   ~0009114

SVN Updated as Jiri proposed Tool is located in Helpers\FlacFixTool

MM Script Included

jiri

2007-05-07 11:21

administrator   ~0009116

Ok, it looks good. Some things to be improved:
 1. Since there are several files distributed, we should make a simple installer based on http://www.mediamonkey.com/forum/viewtopic.php?t=4040. It should install it all to scripts folder and add scripts.ini files entries. Put the InnoSetup installer script to Helpers\FlacFixTool\Setup.
 2. FlacFixTool.vbs should be able to work with >1 file, i.e. for all selected files the .exe tool is called repeatedly and in the end the result for all of the selected files is shown (e.g. in an HTML control as a simple table).
 3. Currently the .exe tool is written to support ASCII only. I suppose that it won't cause trouble, but it could for some users with mutlilingual files. Then we would need to compile it with TNT controls and support Unicode.

peke

2007-05-13 01:41

developer   ~0009138

Fixed small File Lock Issue, changed script so that it works on file selection not just on one file added Instalation packages Files for MM 3.0 Made Icon file for util

Fix Work as AutoScript I'm not so sure that we need making complete installation package if both files are copied to scripts\Auto script will work

I added Statistic window on end.

rusty

2007-05-13 04:24

administrator   ~0009139

I don't think the installer is critical, but it's certainly preferable, considering the pain we've caused these users. But I'll let Jiri have the final word...

jiri

2007-05-13 21:36

administrator   ~0009141

Fixed and uploaded to forum for testing.

rusty

2007-05-14 21:22

administrator   ~0009142

Last edited: 2007-05-14 21:24

Peke asked for suggested UI changes to his front end. Re-opening on the assumption that we want to include the front end and package this into an installer:
Main change would be that the tool should indicate the status per track so that users know which tracks had problems, and which were fixed:

--------------------------------------------------------------------

Summary Report:
x Uncorrupted tracks
y Corrupted tracks fixed
z Ignored tracks (cannot be analyzed)

Details:
Artist__Title__Path__Result
xx . . yy . . zz . . aa

In most cases, a fixed track will function correctly, however, if the track header had been damaged on more than one occasion, it may be impossible to completely eliminate audio artifacts and the track will need to be re-encoded.

[Play fixed tracks] [Close]

------------------------------------------------------------------------

note: the 'Play fixed Tracks' function is a 'nice to have' function, but not critical.

peke

2007-05-15 00:50

developer   ~0009144

Updated GUI Correction according to Rusty specs, buttons are positioned on left due to better look in case of form resize, default is Close.

Funcs.pas is updated as it includes function to read FLAC info and regenerate Stream header from it, but left unactivated apron agreement with Jiri over IM.

rusty

2007-06-15 19:50

administrator   ~0009442

Verified and posted faq entry.