Altitude/AltitudeQuests
5
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Fixed being able to get reward without doing the quest

Browse Source
This commit is contained in:
Teriuihi 2023-03-10 21:26:22 +01:00
parent 7d9d37f17b
commit 4e2aa4258d
2 changed files with 8 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/main/java/com/alttd/altitudequests/commands/subcommands/CommandGetReward.java
View File

@ -35,10 +35,16 @@ public class CommandGetReward extends SubCommand {
return true; return true;
} }
if (!dailyQuest.isDone()) {
player.sendMiniMessage(MessagesConfig.NOT_FINISHED_QUEST, null);
return true;
}
if (dailyQuest.isRewardReceived()) { if (dailyQuest.isRewardReceived()) {
player.sendMiniMessage(MessagesConfig.REWARD_ALREADY_RECEIVED, null); player.sendMiniMessage(MessagesConfig.REWARD_ALREADY_RECEIVED, null);
return true; return true;
} }
for (String command : dailyQuest.getRewardCommand()) for (String command : dailyQuest.getRewardCommand())
Bukkit.getServer().dispatchCommand(Bukkit.getConsoleSender(), command.replaceAll("<player>", commandSender.getName())); Bukkit.getServer().dispatchCommand(Bukkit.getConsoleSender(), command.replaceAll("<player>", commandSender.getName()));
dailyQuest.setRewardReceived(true); dailyQuest.setRewardReceived(true);

2
src/main/java/com/alttd/altitudequests/config/MessagesConfig.java
View File

@ -37,6 +37,7 @@ public class MessagesConfig extends AbstractConfig{
public static String REWARD_ALREADY_RECEIVED = "<red>You already collected this reward</red>"; public static String REWARD_ALREADY_RECEIVED = "<red>You already collected this reward</red>";
public static String REWARD_SENT = "<white>[<gold>Mascot</gold>] <light_purple>Scruff</light_purple><gray>:</gray> <green>Thank you for completing the quest! Your reward has been sent!</green></white>"; public static String REWARD_SENT = "<white>[<gold>Mascot</gold>] <light_purple>Scruff</light_purple><gray>:</gray> <green>Thank you for completing the quest! Your reward has been sent!</green></white>";
public static String FINISHED_QUEST = "<white>[<gold>Mascot</gold>] <light_purple>Scruff</light_purple><gray>:</gray> <green>You finished your quest! Don't forget to collect your reward at the end of the quest book!</green></white>"; public static String FINISHED_QUEST = "<white>[<gold>Mascot</gold>] <light_purple>Scruff</light_purple><gray>:</gray> <green>You finished your quest! Don't forget to collect your reward at the end of the quest book!</green></white>";
public static String NOT_FINISHED_QUEST = "<red>You have not finished this quest yet.</red>";
private static void loadMessages() { private static void loadMessages() {
TOO_FAR_FROM_NPC = config.getString("messages.too-far-from-npc", TOO_FAR_FROM_NPC); TOO_FAR_FROM_NPC = config.getString("messages.too-far-from-npc", TOO_FAR_FROM_NPC);
DAILY_ALREADY_DONE = config.getString("messages.daily-already-done", DAILY_ALREADY_DONE); DAILY_ALREADY_DONE = config.getString("messages.daily-already-done", DAILY_ALREADY_DONE);
@ -44,6 +45,7 @@ public class MessagesConfig extends AbstractConfig{
REWARD_ALREADY_RECEIVED = config.getString("messages.reward-already-received", REWARD_ALREADY_RECEIVED); REWARD_ALREADY_RECEIVED = config.getString("messages.reward-already-received", REWARD_ALREADY_RECEIVED);
FINISHED_QUEST = config.getString("messages.finished-quest", FINISHED_QUEST); FINISHED_QUEST = config.getString("messages.finished-quest", FINISHED_QUEST);
REWARD_SENT = config.getString("messages.reward-send", REWARD_SENT); REWARD_SENT = config.getString("messages.reward-send", REWARD_SENT);
NOT_FINISHED_QUEST = config.getString("messages.not-finished-quest", NOT_FINISHED_QUEST);
} }
public static String NO_PERMISSION = "<red>You do not have permission to do that.</red>"; public static String NO_PERMISSION = "<red>You do not have permission to do that.</red>";