diff --git a/src/com/massivecraft/massivecore/money/MoneyMixinVault.java b/src/com/massivecraft/massivecore/money/MoneyMixinVault.java index 872c14c6..4acfab32 100644 --- a/src/com/massivecraft/massivecore/money/MoneyMixinVault.java +++ b/src/com/massivecraft/massivecore/money/MoneyMixinVault.java @@ -252,6 +252,7 @@ public class MoneyMixinVault extends MoneyMixinAbstract public static Object getEconomyObject(String accountId) { + if (null == accountId) return null; // Because of Factions we have this crazy-workaround. // Where offlineplayers will be used when possible // but otherwise names will. diff --git a/src/com/massivecraft/massivecore/store/SenderColl.java b/src/com/massivecraft/massivecore/store/SenderColl.java index 952fe1af..6d6723c5 100644 --- a/src/com/massivecraft/massivecore/store/SenderColl.java +++ b/src/com/massivecraft/massivecore/store/SenderColl.java @@ -70,7 +70,6 @@ public class SenderColl> extends Coll implements Se @Override public String fixId(Object oid) { - // A null oid should always return null. if (oid == null) throw new NullPointerException("oid"); String ret = null; @@ -87,8 +86,10 @@ public class SenderColl> extends Coll implements Se if (ret == null) { // Always lower case. - return IdUtil.getId(oid); + ret = IdUtil.getId(oid); } + + if (ret == null) throw new NullPointerException("Could not fix: " + oid); if (this.isLowercasing()) {