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

NPE on ClientEvents#onOverlayRender #347

Open
DiegoRutolo opened this issue Apr 5, 2024 · 1 comment
Open

NPE on ClientEvents#onOverlayRender #347

DiegoRutolo opened this issue Apr 5, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@DiegoRutolo
Copy link

var width = Mth.ceil(container.getStoredAura() / (float) container.getMaxAura() * 80);

Sometimes, container can be null, crashing the client. I wasn't able to reproduce it at will, but a simple null check should do the trick.

@Ellpeck
Copy link
Owner

Ellpeck commented Apr 5, 2024

Hmm, yea that should probably have the null check just to be sure, but also: the aura cache shouldn't be able to be without that capability, so something else must be going wrong as well there. If you manage to reproduce this issue, please let me know!

@Ellpeck Ellpeck added the bug Something isn't working label Apr 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants