Fixed bug preventing file rename in different cases #10
Conversation
…fied and a whole folder was converted the files were not renamed according to the database. Use OutputFolder /OUT instead of outputpath from /F when converting an entire directory
| @@ -46,7 +46,7 @@ static void Main(string[] args) | |||
|
|
|||
|
|
|||
| if (decryptorOptions.UsageMode == Mode.Folder) | |||
| decryptor.DecryptAll(decryptorOptions.InputPath, decryptorOptions.OutputPath); | |||
| decryptor.DecryptAll(decryptorOptions.InputPath, decryptorOptions.OutputFolder); | |||
There was a problem hiding this comment.
good catch! seems i missed to change this one...
There was a problem hiding this comment.
Enjoy!
Also check the other change in Decryptor.cs. In case you don't give an output directory (using a /OUT parameter) and you give a database (using a /DB) it will go, search through the database, get the correct video name but will throw an error "Could not retrive media information from database...Falling back to default behaviour" because the outputDirectory is null/not set to an instance of an object.
I set it to the same directory as the input one by default if none is received, it works just fine like this and it creates all the folder and nice video file names structure in the same directory as the input files. Maybe another check and fix would also be nice here, if the /OUT parameter does not give a legal file path do the same thing and create the nice chapter folder structure in the input folder.
There was a problem hiding this comment.
@mdomnita Can you please give full command that I can use to convert whole folder while renaming files according to database.
|
Merged this PR with some minor changes... thanks @mdomnita for pointing out. |
Fixed bug: when no output directory was specified, database was specified and a whole folder was converted the files were not renamed according to the database.
Use OutputFolder /OUT instead of outputpath from /F when converting an entire directory