From 71c079f9dc52bd75454bb8b06f8687620bbb70c2 Mon Sep 17 00:00:00 2001 From: TechnicallyCoded Date: Mon, 9 Oct 2023 10:37:14 +0200 Subject: [PATCH] Change AlchemyBrewCheckTask to be run at the block location instead of the player --- .../gmail/nossr50/skills/alchemy/AlchemyPotionBrewer.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/main/java/com/gmail/nossr50/skills/alchemy/AlchemyPotionBrewer.java b/src/main/java/com/gmail/nossr50/skills/alchemy/AlchemyPotionBrewer.java index b753c668f..e67ae05d4 100644 --- a/src/main/java/com/gmail/nossr50/skills/alchemy/AlchemyPotionBrewer.java +++ b/src/main/java/com/gmail/nossr50/skills/alchemy/AlchemyPotionBrewer.java @@ -259,8 +259,9 @@ public final class AlchemyPotionBrewer { } public static void scheduleCheck(Player player, BrewingStand brewingStand) { - // TODO TECH CHECK - should be runAtLocation? We are using block data AND player data. Must resolve correctly - mcMMO.p.getFoliaLib().getImpl().runAtEntity(player, new AlchemyBrewCheckTask(player, brewingStand)); + // TODO FOLIA - Player thread safety is needed. Opted to use the brewing stand location since world data is + // probably more sensitive. + mcMMO.p.getFoliaLib().getImpl().runAtLocation(brewingStand.getLocation(), new AlchemyBrewCheckTask(player, brewingStand)); } public static void scheduleUpdate(Inventory inventory) {