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

BR Height Error #2593

Closed
2 tasks done
Bramsies opened this issue Feb 27, 2024 · 1 comment · Fixed by #2632
Closed
2 tasks done

BR Height Error #2593

Bramsies opened this issue Feb 27, 2024 · 1 comment · Fixed by #2632
Labels
Requires Testing This is a new issue which needs to be approved before labeled with "bug"

Comments

@Bramsies
Copy link

Server Implementation

Paper

Server Version

1.20

Describe the bug

When binding a height brush, an error follows.
The error is only thrown if an imagemap is provided.

Providing additional parameters does not change anything (size and rotation parameters for example)

This bug persists in the latest version (FastAsyncWorldEdit-Bukkit-2.9.1-SNAPSHOT-661)

To Reproduce

  1. Install Paper-196 (MC 1.20.1)
  2. Install FAWE 2.9.0
  3. Hold any tool
  4. Run /br height https://i.imgur.com/HSPfNbu.png

Expected behaviour

The tool is supposed to bind a heightmap brush

Screenshots / Videos

image
image

Error log (if applicable)

https://paste.gg/p/anonymous/51ed33343ba848e2a6523bcee8894dec

Fawe Debugpaste

https://athion.net/ISPaster/paste/view/115d50ff387a4648bddac4047df8ab9e

Fawe Version

FastAsyncWorldEdit version 2.9.0

Checklist

Anything else?

No response

@Bramsies Bramsies added the Requires Testing This is a new issue which needs to be approved before labeled with "bug" label Feb 27, 2024
@PierreSchwang
Copy link
Member

I'm not sure if that's due to the age of the Issue - but that image link results in a 404 Not Found. That'd explain your error, as no image decoder may read that file (as it's no image but an HTML error page). We should presumably / most certainly handle null values there better either way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Requires Testing This is a new issue which needs to be approved before labeled with "bug"
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants