- Object:
TotalDeathMessages.playerKillStats
- Description:
- Players are not removed from the list, which might impact performance, e.g. in the
KillingspreeMessageTask
- Players are not removed from the list, which might impact performance, e.g. in the
- Solution:
- Possibly in
KillingspreeMessageTask
, as we iterate over all entries there
- Possibly in
- Impact:
- Performance (at high user count/very long server uptime)
- Priority: very low
- Object:
EntityDeathListener.onEntityDeath
- Description:
- Handler for Entity deaths
- Wayyyy too long to be understandable, giant nested if-else abomination
- Solution:
- Replace with some kind of plug-and-play Handler system, where each death type is called from a central place
- Impact:
- Code Readability
- Maintainability
- Priority: high
- Object: everywhere
- Description:
- BungeeCord chat system was deprecated in Paper and replaced by Adventure
- Solution:
- Replace all BungeeCord chat component usages with Adventure
- Impact:
- Maintainability
- Priority: medium
- Object: everywhere
- Description:
- Some messages are referencing the player as male
- Solution:
- Replace all mentions with they
- Impact:
- Inclusivity
- Priority: medium
- Object:
com.example.test
- Description: insert here
- Solution:
- insert
- Impact:
- insert
- Priority: insert here