RedProtect Anti-Grief

RedProtect Anti-Grief

73.3k Downloads

[BUG] Spawning flag

Melvout opened this issue ยท 5 comments

commented

Hi !

Here's a problem with the mob-spawning flag :

If I set to "true" only one value like this

image

Then I leave the GUI and come back on it, I can't set this value to "false" by cliking on it. If I do that and leave the GUI the value stay on "true"

The only way to reset the value is to use the "redstone block" button.

( Same problem for both passive and aggressive animals )


The issue seems to be on this file : https://github.com/FabioZumbi12/RedProtect/blob/master/RedProtect-Spigot/src/main/java/br/net/fabiozumbi12/RedProtect/Bukkit/guis/MobFlagGui.java#L133

On the condition at the line 134, if the string is empty you assign the old value.


It is not a huge problem but it seems not to work correctly.

Thanks for your help !

commented

This bug ia fixed allready. Could you close it?

commented

I am pretty sure that the bug is still relevant. Although if Fabio thinks that the bug is fixed he can close the issue by himself.

commented

Have you never tried with latest version from jenkins?

We use this on oir 1.16.3 papermc and there is problem does not exist anymore since the first fix of the new animals and Monster gui in the flags

commented

Yes we're using the latest version of RedProtect.

Do you correctly understand the issue that I'm talking about ? As I said it's not a big deal but it's not working the way it should be.

commented

I tried it the exact way u wrote and it works fine without any problem here.
The problem you wrote about can be recreated easiely with the first version of rp where the animal anf Monster gui was introduced