-
-
Notifications
You must be signed in to change notification settings - Fork 311
Open
Labels
Requires TestingThis is a new issue which needs to be approved before labeled with "bug"This is a new issue which needs to be approved before labeled with "bug"
Description
Server Implementation
Paper
Server Version
1.20.4
Describe the bug
Changes in #3052 now require commands using selections to include @Confirm(Confirm.Processor.REGION). Not having this gives a pretty unfriendly error:
Previously you would just get the regular informative error:
I have tested adding the annotation to one command and it doesn't appear to cause issues with regular WorldEdit, so the change doesn't cause any compatibility issues.
To Reproduce
- Create a command which takes a Selection
- Do not add the Confirm annotation
- Run the command ingame without an active selection
- IllegalStateException error
Expected behaviour
Commands expecting a selection which do not recieve one should error instructively as before:

Effectively, when no @Confirm(...) is present for the region, Confirm.Processor.REGION should be the default.
Screenshots / Videos
No response
Error log (if applicable)
No response
Fawe Debugpaste
https://athion.net/ISPaster/paste/view/d8e209464d4e4bcd85aec7766117481f
Fawe Version
FastAsyncWorldEdit-Bukkit-2.12.4-SNAPSHOT-1020
Checklist
- I have included a Fawe debugpaste.
- I am using the newest build from https://ci.athion.net/job/FastAsyncWorldEdit/ and the issue still persists.
Anything else?
No response
Metadata
Metadata
Assignees
Labels
Requires TestingThis is a new issue which needs to be approved before labeled with "bug"This is a new issue which needs to be approved before labeled with "bug"
