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

sanitise input and output #16

Open
pascalo7 opened this issue Jul 10, 2019 · 2 comments
Open

sanitise input and output #16

pascalo7 opened this issue Jul 10, 2019 · 2 comments

Comments

@pascalo7
Copy link

When the script reads a file with [1080p-FLAC] in the title it will crash out with the error
Invalid [] range "p-F" in regex; marked by <-- HERE in m/[FFF] Golden Time - 01 [BD][1080p-F <-- HERE LAC][0A94EDB8].mkv/ at /adbren/adbren.pl line 187, <$log> chunk 1419.

When a show such as To Love-Ru: Trouble - Darkness OAD is processed the script wont escape / in the episode titles so when it processes the file it will create sub folders every time there is a / so the processed file ends up looking like

├── To Love-Ru: Trouble - Darkness OAD - 2 - Nostalgia: Back Then at That Place 
│   └──  The Changing Heart: Honest Feelings 
│       └──  Flower: Budding Feelings.mkv

@pascalo7
Copy link
Author

i forgot i was using --noclean in my script, but this should still be escaped so this does not happen regardless of what flags are being used

@hergonoway
Copy link

hergonoway commented Aug 6, 2019

I think I've got the same issue :

Invalid [] range "l-A" in regex; marked by <-- HERE in m/_corrupt_[AnimeRG] My Hero Academia - Two Heroes (2018 Movie) Boku no Hero Academia - Futari no Hero [1080p] [Dual-A <-- HERE udio] [5.1 AAC] [Multi-Sub] [HEVC] [x265] [pseudo].mkv/ at /usr/local/bin/adbren.pl line 187, <$log> chunk 291.

The issue seems to be isolated to line 187 in the grep call when the regex bump into non alphabetical order argument like l-A or p-F, my workaround is to add --noskip to pass this block of code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants