Theguyhere0 / villager-defense-minigame

A Minecraft plugin that adds a fun, customizable minigame where players defend villagers against waves of monsters
GNU General Public License v3.0
7 stars 4 forks source link

Challenge-wave achievements instantly unlocked by leaving before game start #180

Open datatags opened 8 months ago

datatags commented 8 months ago

Describe the bug Achievements that require the player to get "past at least wave X" with specific challenge(s) (e.g. "It Hurts So Good") can be instantly unlocked by leaving during the "game starting" phase.

To Reproduce Steps to reproduce the behavior:

  1. Ensure you don't have all challenge-wave achievements unlocked already
  2. Join a VD arena
  3. Before the countdown finishes, equip all challenges and then leave the game.
  4. Observe you now have all challenge-wave achievements unlocked.

Expected behavior No achievements should be unlocked since the game never actually started.

Screenshots Chat full of achievement notifications Achievements unlocked after reproducing the bug

Server and plugin versions (please complete the following information):

Additional context Seems to be caused by these lines missing a check for ArenaStatus.WAITING