From dea71358aed120295ac7542092f6c2d7f110a5c9 Mon Sep 17 00:00:00 2001 From: BONNe Date: Sun, 11 Aug 2019 18:12:56 +0300 Subject: [PATCH] Update Utils.java Avoid null-pointer issue if given item-list contains null elements. The issue is still that list contains these elements. --- src/main/java/world/bentobox/challenges/utils/Utils.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/main/java/world/bentobox/challenges/utils/Utils.java b/src/main/java/world/bentobox/challenges/utils/Utils.java index ecc4ecd..802c822 100644 --- a/src/main/java/world/bentobox/challenges/utils/Utils.java +++ b/src/main/java/world/bentobox/challenges/utils/Utils.java @@ -29,7 +29,7 @@ public class Utils // items are in players inventory. for (ItemStack item : requiredItems) { - boolean isUnique = true; + boolean isUnique = item != null; int i = 0; final int requiredSize = returnItems.size(); @@ -49,7 +49,7 @@ public class Utils i++; } - if (isUnique) + if (isUnique && item != null) { // The same issue as in other places. Clone prevents from changing original item. returnItems.add(item.clone());