asofold/simplyvanish

simplyvanish's Issues

DynMap hook buggy

If you login vanished, or maybe just vanishing all together, Dynmap will essentially be broken from ever reshowing on the map until restart

The DynMap author says that SV's hook code is not correct:

https://github.com/asofold/SimplyVanish/blob/master/SimplyVanish/src/me/asofold/bpl/simplyvanish/api/hooks/impl/DynmapHook.java#L53

@mikeprimm assertPlayerInvisibility is right, but setPlayerVisibility is wrong
@mikeprimm first one is for the temporary thing, and should be 'true' for invisible and 'false' for default (whether visible or not)
@mikeprimm while the second call is adjusting the show/hide configuration - and setting it to 'false' means hide
@mikeprimm In any case, the assert is the right thing - just kill the other line, and it should be fine
@mikeprimm Right - assert is a plugin-keyed 'vote' on the visiblity or invisibility - if anyone is voting for it to be invisibible, it is: else, if anyone votes for it to be visible, it is; else the show/hide setting applies (which is what setPlayerVisibility controls)

Doors/Chests

Is there a way that will allow vanished players to pass through/open doors and chests that won't show what they are really doing? When follow griefers, they often close the doors behind them, forcing vanished staff to reopen them. The player can then see that the door open and closed again but nobody passed through. And while small servers who frequently lag can pass it off as lag, larger servers cannot. The same happens for chests.

Can this be changed, or is it one of Minecrafts issues?