Lord-Akkrand / Skyrim-NX-Toolkit

A set of scripts to convert mods from Skyrim SE to Skyrim NX
MIT License
71 stars 1 forks source link

Robust, sanitised inputs #6

Closed Lord-Akkrand closed 6 years ago

Lord-Akkrand commented 6 years ago

Currently there's no checks to make sure the scripts are getting what they expect.

Some basic checks and sanitisation would allow each script to work that out for itself.

Similarly, there's no check to see if the origin/target folders passed in are sane. If you were to pass in: Origin=C:\blah\something\mod Target=C:\blah then ROBOCOPY would clobber your entire blah folder, and probably break all the things.

Lord-Akkrand commented 6 years ago

It's better now. All the conversion scripts take the same input, and there's no longer some scripts using base/Data/ whilst others use base/ Still need to give a warning or error out if your ROBOCOPY would clobber the origin.