Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The mod isn't respecting any restrictions (Claims, server-properties PVP, /heal) #54

Open
TheFlash787 opened this issue Oct 7, 2019 · 4 comments

Comments

@TheFlash787
Copy link

Hey!

There's a major issue with FirstAid preventing it from being used on many/any servers. Firstly, when using /heal from a plugin, it doesn't actually take effect because FirstAid is overwriting it with it's own health data, without using the generic one. (See SpongePowered/SpongeForge#2998)

Secondly, any claim protection is being completely ignored because whilst players are not being visually hit, the health is still being taken which means that players can be spawn-killed even if PVP protection is enabled.

Lastly, and the most important one (since I understand that mod developers will ignore issues related to Sponge or plugins), when pvp is disabled in server.properties, it makes absolutely no effect whatsoever. Players are never visually hurt, but the distributed damage is taken and the player will still die.

@ichttt
Copy link
Owner

ichttt commented Oct 8, 2019

Heal should work if the watchSetHealth and allowOtherHealingItems config option is enabled, as it watches if the genic health one changed and adjusts the average health of body parts to match.
I just tested disabling pvp, at least in the 1.14.4 version it works without problems. Disabling pvp allows no damage to be done.

@MrFriendlyz
Copy link

MrFriendlyz commented Jun 5, 2020

was this problem solved im having same problems using worlddefender for sponge forge

@nicholasrobertm
Copy link

This isn't resolved, I'm guessing it may be fixed in a non 1.12.2 version but this still exists in 1.12.2 despite permissions and such.

@BoomerSchmirtz
Copy link

Still not resolved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants