From e0a0be7befb9d32859f334647a944c086346a412 Mon Sep 17 00:00:00 2001 From: Adam Date: Thu, 12 Mar 2020 20:53:29 -0400 Subject: [PATCH] Move null check to the correct location (#746) Closes #744, bug introduced in #736 --- .../GriefPrevention/FindUnusedClaimsTask.java | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/src/main/java/me/ryanhamshire/GriefPrevention/FindUnusedClaimsTask.java b/src/main/java/me/ryanhamshire/GriefPrevention/FindUnusedClaimsTask.java index 572f306..bcb4773 100644 --- a/src/main/java/me/ryanhamshire/GriefPrevention/FindUnusedClaimsTask.java +++ b/src/main/java/me/ryanhamshire/GriefPrevention/FindUnusedClaimsTask.java @@ -18,12 +18,10 @@ package me.ryanhamshire.GriefPrevention; -import java.util.ArrayList; import java.util.Collections; import java.util.Iterator; import java.util.List; import java.util.Objects; -import java.util.Set; import java.util.UUID; import java.util.stream.Collectors; @@ -63,10 +61,8 @@ class FindUnusedClaimsTask implements Runnable public void refreshUUIDs() { // Fetch owner UUIDs from list of claims - Set uuidSet = GriefPrevention.instance.dataStore.claims.stream().filter(Objects::nonNull) - .distinct().map(claim -> claim.ownerID).collect(Collectors.toSet()); - - claimOwnerUUIDs = new ArrayList<>(uuidSet); + claimOwnerUUIDs = GriefPrevention.instance.dataStore.claims.stream().map(claim -> claim.ownerID) + .distinct().filter(Objects::nonNull).collect(Collectors.toList()); if (!claimOwnerUUIDs.isEmpty()) { // Randomize order