Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ERROR: The regular expression pattern ) is not valid. #68

Closed
alekdavis opened this issue May 10, 2020 · 3 comments · Fixed by #71 or #73
Closed

ERROR: The regular expression pattern ) is not valid. #68

alekdavis opened this issue May 10, 2020 · 3 comments · Fixed by #71 or #73
Assignees
Labels
Milestone

Comments

@alekdavis
Copy link

When the script tries to process a file, it generates this error:

05/10/20 14:09:23 Processing - \\MYNAS\DIR\PATH\Video (2012)\Video (2012).mkv
05/10/20 14:09:23 File 4 of 14 - Total queue 28.57%
05/10/20 14:09:24 Video: H264, Audio: AC3. Encoding audio to AAC
The regular expression pattern ) is not valid.
At C:\conv2mp4\files\func\ConvertFile.ps1:76 char:13
+             $title = $title -replace "$remove",''
+             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: ():String) [], RuntimeException
    + FullyQualifiedErrorId : InvalidRegularExpression

Using v4.1.2 of the script.

@BrianDMG
Copy link
Owner

Will push out a fix for this shortly. In the meantime, you can set use_set_metadata_title=false in the config

@BrianDMG BrianDMG self-assigned this Jun 20, 2020
@BrianDMG BrianDMG added the Bug label Jun 20, 2020
@BrianDMG BrianDMG added this to the 4.2 milestone Jun 20, 2020
BrianDMG added a commit that referenced this issue Jun 20, 2020
- Closes #67 
- Closes #68
- Intermediary file (target file) now takes on a non-media file extension (.conv2mp4) while encoding is underway in order to prevent it from being scanned/picked up by media scanners (Plex media scanner, etc.) before it is completed. The file extension is restored to .mp4 once encoding has completed.
@BrianDMG
Copy link
Owner

Just pushed a fix for this issue to develop branch. Give it a shot and see if that clears your issue up. If it does, I'll merge to master.

@alekdavis
Copy link
Author

Looking good.

@BrianDMG BrianDMG linked a pull request Aug 4, 2020 that will close this issue
BrianDMG added a commit that referenced this issue Aug 4, 2020
@BrianDMG BrianDMG mentioned this issue Aug 4, 2020
BrianDMG added a commit that referenced this issue Aug 4, 2020
- Improved metadata parsing
- Bug fixes
   - Closes #67
   - Closes #68
   - Closes #70
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants