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
Boss bars for Wither & Ender Dragon #642
Conversation
if (hurtSound != null && !isSilent()) { | ||
world.playSound(location, hurtSound, getSoundVolume(), getSoundPitch()); | ||
} | ||
Sound hurtSound = health > 0 ? getHurtSound() : getDeathSound(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the reason for this change? Death sound is already played here:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reverted.
public void setHealth(double health) { | ||
super.setHealth(health); | ||
if (health <= 0) { | ||
bar.removeAll(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this should be moved to the remove()
method instead. It is called when the entity dies, as well as if it is removed by other factors like despawning or plugins.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
I'm fine with merging this for this release because it isn't huge and doesn't have breaking changes. |
Merge #641 first. Should this wait for after 2018.0.0?