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

Suspicious sand NBT default state #3044

Open
2 tasks done
RobijnenTaart opened this issue Dec 23, 2024 · 1 comment · May be fixed by #3053
Open
2 tasks done

Suspicious sand NBT default state #3044

RobijnenTaart opened this issue Dec 23, 2024 · 1 comment · May be fixed by #3053
Labels
Requires Testing This is a new issue which needs to be approved before labeled with "bug"

Comments

@RobijnenTaart
Copy link

Server Implementation

Paper

Server Version

1.21.3

Describe the bug

When running:
//rep yellow sand,suspicious_sand,smooth_sandstone
Error being displayed:
Invalid value for <to> (You are not allowed to use nbt. Lacking permission: worldedit.anyblock.nbt), acceptable values are any pattern.
Solution for problem (FAWE staff suggested):
Make "default" NBT is handled separately.

To Reproduce

  1. Make a selection.
  2. //rep yellow sand,suspicious_sand,smooth_sandstone

Expected behaviour

Replace yellow_wool to sand,suspicious_sand,smooth_sandstone

Screenshots / Videos

No response

Error log (if applicable)

No response

Fawe Debugpaste

https://athion.net/ISPaster/paste/view/a66ea62682964555802864f3fbfc6e81

Fawe Version

FastAsyncWorldEdit-Bukkit-2.12.3-SNAPSHOT-1002.jar

Checklist

Anything else?

No response

@RobijnenTaart RobijnenTaart added the Requires Testing This is a new issue which needs to be approved before labeled with "bug" label Dec 23, 2024
@dordsor21
Copy link
Member

We could do a check on if the NBT is equal to the default nbt, but I do not know if a simple reference comparison on the default NBT would be enough, and also whether a #equals check would necessarily work either.

dordsor21 added a commit that referenced this issue Dec 31, 2024
@dordsor21 dordsor21 linked a pull request Dec 31, 2024 that will close this issue
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