Sick-Beard icon indicating copy to clipboard operation
Sick-Beard copied to clipboard

Published 20 hours ago • verified publisher icon midgetspy

Filename sanitization fix

Open webdestroya opened this issue 10 years ago • 3 comments

Fixed this issue:

Season.23/How.Its.Made.S23E28.Wood.Garage.Doors;.Salt.Spreaders;.Animatronic.Dinosaurs.mp4

which should be:

Season.23/How.Its.Made.S23E28.Wood.Garage.Doors.Salt.Spreaders.Animatronic.Dinosaurs.mp4

webdestroya avatar Jan 19 '15 18:01 webdestroya

Hmm, the purpose of that function is simply to make it into a valid non-hidden filename, not to make it more human-readable. I'll have to put a little more thought into whether we want that function to also do other things...

midgetspy avatar Feb 03 '15 02:02 midgetspy

personally i like the ; as its used to separate the ep segments. if you trip it... well it loses its meaning. although it is quite cosmetic in the end..

thezoggy avatar Feb 03 '15 06:02 thezoggy

I guess my preference was to prevent anything that requires escaping when accessing files in shell, which included ;, $, etc

webdestroya avatar Feb 03 '15 18:02 webdestroya