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

Entities rendering incorrectly due to DataEntry value not updating #3940

Closed
XXMA16 opened this issue Feb 1, 2024 · 0 comments
Closed

Entities rendering incorrectly due to DataEntry value not updating #3940

XXMA16 opened this issue Feb 1, 2024 · 0 comments
Labels
status: needs triage This label is automatically applied to new issues and pull requests to indicate they require triage type: bug Something isn't working

Comments

@XXMA16
Copy link
Contributor

XXMA16 commented Feb 1, 2024

Affected Product(s)

SpongeForge, SpongeVanilla

Version

spongevanilla-1.16.5-8.2.0

Operating System

Windows

Java Version

jbrsdk jcef 17.0.9

Plugins/Mods

No additional mods or plugins

Describe the bug

Any mob whose rendering calls entity.isAggressive() (i.e. Skeletons and Drowned with Tridents) will have an incorrect pose while attacking due to said method call returning false (when it should be true). This happens because of an @Overwrite here. Without any plugins the value will always be retrieved from converter.get().getValueFromEvent(currentValue, event.endResult()) which will return currentValue, thus discarding the incomingValue and telling the client the mob is not aggressive while attacking.

Although I have not done extensive testing, this bug seems to affect the following versions (and almost certainly any version between them, both Forge and Vanilla):

  • spongevanilla-1.16.5-8.2.0
  • spongeforge-1.16.5-36.2.5-8.2.0
  • spongevanilla-1.19.4-10.0.0-RC1469
  • spongevanilla-1.20.2-11.0.0-RC1471

spongevanilla-1.12.2-7.4.8-RC427 is not affected

Link to logs

No response

@XXMA16 XXMA16 added status: needs triage This label is automatically applied to new issues and pull requests to indicate they require triage type: bug Something isn't working labels Feb 1, 2024
@aromaa aromaa closed this as completed in 3027be5 Feb 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: needs triage This label is automatically applied to new issues and pull requests to indicate they require triage type: bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant