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

fix MC-53850 #61

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

MicrocontrollersDev
Copy link

Description

!! this changes vanilla behavior

mojira: https://bugs.mojang.com/browse/MC-53850
forge: MinecraftForge/MinecraftForge#3807

this is fixed in forge 1.11+ so i dont think itd be a problem in 1.8 even though its not a fully client sided fix. it was a very simple backport though so thought id pr it regardless

Related Issue(s)

Fixes #

How to test

item command for 1.8.9: /summon Item ~2 ~ ~ {Item:{id:stone,Count:1},Invulnerable:1}
xp command for 1.8.9: /summon Item ~2 ~ ~ {Item:{id:stone,Count:1},Invulnerable:1}

you can tell its not vanilla as without the fix, setting the xp orb on fire will make the armor stand stop in place until you are close enough to collect it, meanwhile with the fix the armor stand will continue to slowly follow you around even while on/after the fire. since its fixed in forge i dont think its an issue here

Release Notes

Fix vanilla bug where damaged invulnerable entities stop rendering

Documentation

@Wyvest
Copy link
Member

Wyvest commented Jul 21, 2024

just remove the other check other than remote

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

Successfully merging this pull request may close these issues.

2 participants