mirror of
https://github.com/SabreTools/MPF.git
synced 2026-02-04 05:35:52 +00:00
MPF no longer handles '.' in Output Filename correctly #374
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @cdifan on GitHub (Jan 1, 2022).
Originally assigned to: @mnadareski on GitHub.
Version
What version are you using?
Build
What runtime version are you using?
Describe the issue
When you type or paste a period '.' in the Output Filename field, MPF 2.2 no longer replaces it with an underscore '_', e.g. it does not change "XXX 1.0" to "XXX 1_0" (MPF 2.1 used to do this). As a result, after running DIC, MPF will complain about not finding output files because it will replace anything after the first period with the expected suffix, e.g. it will look for "XXX 1.cue" and "XXX 1_disc.txt" instead of the correct "XXX 1.0.cue" and "XXX 1.0_disc.txt" which are generated by DIC.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Either restore the previous behaviour (replace '.' with '_') in the filename) or (preferred solution) correct the DIC output file search to find the correct files.
Additional context
Everything works fine if you use e.g. "XXX 1_0" in the Output Filename. Also, there are no issues with period characters in the Output Directory field.
@mnadareski commented on GitHub (Jan 1, 2022):
I'd like screenshots of both the UI before dumping when this comes up as well as a copy of what the log says after. I have an idea of what could be causing this issue, but I need to see the input before I'm sure.