search

clip9 / adbren

adbren - Rename and organize anime using this AniDB API client written in perl
39 stars 10 forks source link

sanitise input and output #16

Open pascalo7 opened 5 years ago

pascalo7 commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

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