Altitude/PlayerUtils
5
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix NPE in key configuration, fixed tab completion

Browse Source 
Added missing key mapping to prevent NullPointerException when no UUIDs are present in KeyStorage. Also adjusted tab completion logic in Key.java to handle cases with more than one argument.
This commit is contained in:
Teriuihi 2024-10-04 21:24:49 +02:00
parent d43f269c22
commit 5a7b9859cd
2 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/main/java/com/alttd/playerutils/commands/playerutils_subcommands/Key.java
View File

@ -74,7 +74,7 @@ public class Key extends SubCommand {
@Override @Override
public List<String> getTabComplete(CommandSender commandSender, String[] args) { public List<String> getTabComplete(CommandSender commandSender, String[] args) {
if (args.length > 1) { if (args.length > 2) {
return List.of(); return List.of();
} }
return Config.KEY.CRATES.keySet().stream().toList(); return Config.KEY.CRATES.keySet().stream().toList();

1
src/main/java/com/alttd/playerutils/config/KeyStorage.java
View File

@ -44,6 +44,7 @@ public class KeyStorage extends AbstractConfig {
ConfigurationSection configurationSection = config.getConfigurationSection(prefix + crate); ConfigurationSection configurationSection = config.getConfigurationSection(prefix + crate);
if (configurationSection == null) { if (configurationSection == null) {
config.logger.info(String.format("No section yet for crate %s", crate)); config.logger.info(String.format("No section yet for crate %s", crate));
KEYS.put(crate, count);
continue; continue;
} }
List<UUID> uuids = configurationSection.getKeys(false).stream().map(UUID::fromString).toList(); List<UUID> uuids = configurationSection.getKeys(false).stream().map(UUID::fromString).toList();