fix portable settings.path not respected for prefs and language by avinxshKD · Pull Request #1439 · processing/processing4
processing / processing4 Public
-
-
Notifications
You must be signed in to change notification settings - Fork 170
Merged
Merged
Conversation
avinxshKD
commented
Feb 17, 2026
avinxshKD
commented
Contributor
fixed #1435
check Base.getSettingsOverride() before Settings.getFolder() in Preferences.kt and Locale.kt so portable settings.path works
avinxshKD
commented
Feb 17, 2026
avinxshKD commented
Contributor Author
@catilac @SableRaf @Stefterv
fixed Preferences.kt nd also found the same bug in Locale.kt (language.txt would also go to the wrong folder in portable mode). both were calling Settings.getFolder() directly instead of checking Base.getSettingsOverride() first
let me know if adjustments needed, thanks goodnight
catilac
commented
Feb 19, 2026
catilac commented
Collaborator
lgtm! thank you
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment