SPIGOT-7399: Revert null check in CraftMetaItem#safelyAdd

By: DerFrZocker <derrieple@gmail.com>
This commit is contained in:
CraftBukkit/Spigot 2023-06-17 11:41:38 +02:00
parent 20acc420d8
commit f88d039091

View File

@ -1329,7 +1329,12 @@ class CraftMetaItem implements ItemMeta, Damageable, Repairable, BlockDataMeta {
for (Object object : addFrom) {
if (!(object instanceof String)) {
Preconditions.checkArgument(object == null, "%s cannot contain non-string %s", addFrom, object.getClass().getName());
if (object != null) {
// SPIGOT-7399: Null check via if is important,
// otherwise object.getClass().getName() could throw an error for a valid argument -> when it is null which is valid,
// when using Preconditions
throw new IllegalArgumentException(addFrom + " cannot contain non-string " + object.getClass().getName());
}
addTo.add(CraftChatMessage.toJSON(IChatBaseComponent.empty()));
} else {