Braekpo1nt / MCTManager

The Paper plugin for Minecraft Tournament (MCT)
Other
1 stars 1 forks source link

UI-related bugs shouldn't cause application failure #566

Closed Braekpo1nt closed 3 months ago

Braekpo1nt commented 3 months ago

Description

Sidebar and Topbar related bugs (i.e. adding players who already exist and removing players who don't) should not cause game-breaking Exceptions to be thrown. They should instead fail gracefully, log the error in an obvious way, and allow the execution flow to continue.

Reason for Redesign

Too many times a UI bug has completely broken the game, when the game could have continued without the UI and allowed events to go on and players to continue enjoying themselves. This issue is to end this cycle.

Proposed Redesign

Make a custom "error handling and reporting" method in Sidebar and Topbar which reports errors to the console (and maybe any admins) but allows execution to flow naturally beyond this so that UI-breaking bugs don't become game-breaking bugs.

Definition of Done

Related Issues