refactor: return values if profile is null
This commit is contained in:
parent
3dfb6517a1
commit
ffb50e8391
1 changed files with 21 additions and 11 deletions
|
@ -39,20 +39,30 @@ public class NickoExpansion extends PlaceholderExpansion {
|
|||
|
||||
@Override
|
||||
public @Nullable String onPlaceholderRequest(Player player, @NotNull String params) {
|
||||
if(player == null) return null;
|
||||
Optional<NickoProfile> optionalProfile = instance.getDataStore().getData(player.getUniqueId());
|
||||
if (player == null) return null;
|
||||
|
||||
String name, skin, locale;
|
||||
boolean bungeecord;
|
||||
|
||||
final Optional<NickoProfile> optionalProfile = instance.getDataStore().getData(player.getUniqueId());
|
||||
if (optionalProfile.isPresent()) {
|
||||
final NickoProfile profile = optionalProfile.get();
|
||||
name = profile.getName();
|
||||
skin = profile.getSkin();
|
||||
locale = profile.getLocale().getName();
|
||||
bungeecord = profile.isBungeecordTransfer();
|
||||
} else {
|
||||
name = skin = player.getName();
|
||||
locale = "N/A";
|
||||
bungeecord = true;
|
||||
}
|
||||
|
||||
return switch (params) {
|
||||
case "name" -> profile.getName();
|
||||
case "skin" -> profile.getSkin();
|
||||
case "locale" -> profile.getLocale().getName();
|
||||
case "bungeecord" -> String.valueOf(profile.isBungeecordTransfer());
|
||||
case "name" -> name;
|
||||
case "skin" -> skin;
|
||||
case "locale" -> locale;
|
||||
case "bungeecord" -> String.valueOf(bungeecord);
|
||||
default -> null;
|
||||
};
|
||||
} else {
|
||||
instance.getLogger().severe("Couldn't satisfy request for placeholder " + params + ". This is a bug!");
|
||||
return null;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue