From 94a4051d292cab19b0ebc323bde309948f52446b Mon Sep 17 00:00:00 2001 From: Luck Date: Mon, 30 Oct 2017 20:03:59 +0000 Subject: [PATCH] Add extra catch for null primary group in powerfulperms migration --- .../bukkit/migration/MigrationPowerfulPerms.java | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/bukkit/src/main/java/me/lucko/luckperms/bukkit/migration/MigrationPowerfulPerms.java b/bukkit/src/main/java/me/lucko/luckperms/bukkit/migration/MigrationPowerfulPerms.java index 6691cbff5..ec112bdde 100644 --- a/bukkit/src/main/java/me/lucko/luckperms/bukkit/migration/MigrationPowerfulPerms.java +++ b/bukkit/src/main/java/me/lucko/luckperms/bukkit/migration/MigrationPowerfulPerms.java @@ -264,10 +264,13 @@ public class MigrationPowerfulPerms extends SubCommand { user.setPermission(NodeFactory.makeSuffixNode(maxWeight, suffix).build()); } - String primary = joinFuture(pm.getPlayerPrimaryGroup(uuid)).getName().toLowerCase(); - if (!primary.equals("default")) { - user.setPermission(NodeFactory.make("group." + primary)); - user.getPrimaryGroup().setStoredValue(primary); + Group primaryGroup = joinFuture(pm.getPlayerPrimaryGroup(uuid)); + if (primaryGroup != null && primaryGroup.getName() != null) { + String primary = primaryGroup.getName().toLowerCase(); + if (!primary.equals("default")) { + user.setPermission(NodeFactory.make("group." + primary)); + user.getPrimaryGroup().setStoredValue(primary); + } } plugin.getUserManager().cleanup(user);