Skip to content

Use KSPUtil.SanitizeString() to sanitize vessel filenames - #2391

Merged
lamont-granquist merged 1 commit into
devfrom
lcg/sanitize-vessel-filenames
Oct 28, 2026
Merged

lamont-granquist merged 1 commit into
devfrom
lcg/sanitize-vessel-filenames

Conversation

@lamont-granquist

@lamont-granquist lamont-granquist commented Oct 28, 2026 •

Copy link
Copy Markdown
Collaborator

This means that existing crafts which have e.g. leading or trailing whitespace will have their configs reset to default once.

Makes MJ consistent with how the rest of KSP operates.

Makes filenames consistent across different O/S (copy GameData between operating systems and it should be consistent).

Fixes a bug in the factory reset function to clean up configs correctly and to not use the KSP.IO functions that throw on other bad DLLs because they walk the assembly tree.

closes #1909

This means that existing crafts which have e.g. spaces will have their
configs reset to default once.

Makes MJ consistent with how the rest of KSP operates.

Makes filenames consistent across different O/S (copy GameData between
operating systems and it should be consistent).

Fixes a bug in the factory reset function to clean up configs correctly
and to not use the KSP.IO functions that throw on other bad DLLs
because they walk the assembly tree.

closes #1909
@github-actions

Copy link
Copy Markdown
Contributor

✅ Build Artifact 🚀

MechJeb2-ecafee0.zip (built from 44d10c1)

@lamont-granquist
lamont-granquist merged commit 656ea5c into dev Oct 28, 2026
3 checks passed
@lamont-granquist
lamont-granquist deleted the lcg/sanitize-vessel-filenames branch October 28, 2026 23:51
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

Successfully merging this pull request may close these issues.

Double quotes (and possibly other illegal characters) in craft names are not removed in filenames for craft settings

1 participant