Anvilcraft/Create
2
0
Fork 0
mirror of https://github.com/Creators-of-Create/Create.git synced 2024-09-19 16:08:54 +02:00
Code Issues Projects Releases Packages Wiki Activity

Fixed null pointer when resetting gameplay settings in config menu

Browse source 
- Added a null check to make sure a config value has a comment before trying to split the comment
- Fixes #2412
This commit is contained in:
Cael Warner 2021-12-11 20:58:23 -08:00
parent 51763382d3
commit 01866db72e
No known key found for this signature in database
GPG key ID: F70FD7559CCDFC48
1 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/main/java/com/simibubi/create/foundation/config/ui/SubMenuConfigScreen.java
View file

@ -150,7 +150,12 @@ public class SubMenuConfigScreen extends ConfigScreen {
} else if (obj instanceof ForgeConfigSpec.ConfigValue<?>) {
ForgeConfigSpec.ConfigValue configValue = (ForgeConfigSpec.ConfigValue<?>) obj;
ForgeConfigSpec.ValueSpec valueSpec = spec.getRaw((List<String>) configValue.getPath());
List<String> comments = new ArrayList<>(Arrays.asList(valueSpec.getComment().split("\n")));
List<String> comments = new ArrayList<>();
if (valueSpec.getComment() != null)
comments.addAll(Arrays.asList(valueSpec.getComment().split("\n")));
Pair<String, Map<String, String>> metadata = ConfigHelper.readMetadataFromComment(comments);
ConfigHelper.setValue(String.join(".", configValue.getPath()), configValue, valueSpec.getDefault(), metadata.getSecond());