search

EternalCodeTeam / EternalCombat

⚔ Combat Logging system for Minecraft!
https://eternalcode.pl
Apache License 2.0
16 stars 6 forks source link

GH-100 Add blacklist option to damage causes #100

Closed CitralFlo closed 1 year ago

CitralFlo commented 1 year ago

This is fix for issue reported on EternalCode discord.

image

CitralFlo commented 1 year ago

Myśle że to powinno być zrobione na zasadzie podobnej do komend, załączam link: https://github.com/EternalCodeTeam/EternalCombat/blob/master/src/main/java/com/eternalcode/combat/config/implementation/PluginConfig.java#L57-L70

Moim zdaniem trochę overkill, to tylko zamiana z obecnej whitelisty na blackliste, wykorzystywana tylko w jednym miejscu, plus zostało to skończone już.

Rollczi commented 1 year ago

Myśle że to powinno być zrobione na zasadzie podobnej do komend, załączam link: https://github.com/EternalCodeTeam/EternalCombat/blob/master/src/main/java/com/eternalcode/combat/config/implementation/PluginConfig.java#L57-L70

Moim zdaniem trochę overkill, to tylko zamiana z obecnej whitelisty na blackliste, wykorzystywana tylko w jednym miejscu, plus zostało to skończone już.

Można zrenamować tego enuma na "ListMode" z wartościami BLACKLIST i WHITELIST. Nie będzie trzeba dodawać kolejnego pliku poprostu użyje się tego do obsługi tych dwóch przypadków.