From 3113ee85414c0bf007e60f5e38708aef1364ce5a Mon Sep 17 00:00:00 2001 From: Krakenied Date: Sat, 6 Jul 2024 23:05:28 +0200 Subject: Get autostart from config only once --- .../quests/bukkit/questcontroller/NormalQuestController.java | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) (limited to 'bukkit/src/main/java/com') diff --git a/bukkit/src/main/java/com/leonardobishop/quests/bukkit/questcontroller/NormalQuestController.java b/bukkit/src/main/java/com/leonardobishop/quests/bukkit/questcontroller/NormalQuestController.java index d0fffa3d..e00f5417 100644 --- a/bukkit/src/main/java/com/leonardobishop/quests/bukkit/questcontroller/NormalQuestController.java +++ b/bukkit/src/main/java/com/leonardobishop/quests/bukkit/questcontroller/NormalQuestController.java @@ -23,6 +23,8 @@ import com.leonardobishop.quests.common.quest.Task; import com.leonardobishop.quests.common.questcontroller.QuestController; import org.bukkit.Bukkit; import org.bukkit.entity.Player; +import org.jetbrains.annotations.NotNull; +import org.jetbrains.annotations.Nullable; import java.util.ArrayList; import java.util.HashSet; @@ -187,10 +189,13 @@ public class NormalQuestController implements QuestController { return QuestStartResult.NO_PERMISSION_FOR_CATEGORY; } } - if (questProgress.isStarted() || quest.isAutoStartEnabled() || config.getBoolean("options.quest-autostart")) { + + boolean autostart = this.config.getBoolean("options.quest-autostart"); + if (questProgress.isStarted() || quest.isAutoStartEnabled() || autostart) { return QuestStartResult.QUEST_ALREADY_STARTED; } - if (!config.getBoolean("options.quest-autostart") && quest.doesCountTowardsLimit()) { + + if (quest.doesCountTowardsLimit()) { Set startedQuests = getStartedQuestsForPlayer(qPlayer); int questLimitCount = 0; for (Quest q : startedQuests) { @@ -202,6 +207,7 @@ public class NormalQuestController implements QuestController { return QuestStartResult.QUEST_LIMIT_REACHED; } } + return QuestStartResult.QUEST_SUCCESS; } -- cgit v1.2.3-70-g09d2