Fix dry run option with batch and always

dbr / tvnamer

Automatic TV episode file renamer, uses data from thetvdb.com via tvdb_api

https://pypi.python.org/pypi/tvnamer/

The Unlicense

907 stars 115 forks source link

Fix dry run option with batch and always #173

Closed Zorvalt closed 4 years ago

Zorvalt commented 4 years ago

I think this PR fixes the issue #164.

I am quite new to this tool (and to contributing) so I may have not seen some corner cases.

dbr commented 4 years ago

Thanks! Looks good to me

Probably needs some refactoring in the future to remove the duplication of logic (the dry run path should probably be just before the actual "move file" call, rather than a separate if branch as it was previously), but this is change is much better than it was before!