Skip to content

convert authorization exceptions - #2938

Merged
me4502 merged 5 commits into
EngineHub:version/7.4.xfrom
SamuraiWorf:fix/convert-auth-exceptions
Apr 7, 2026
Merged

convert authorization exceptions#2938
me4502 merged 5 commits into
EngineHub:version/7.4.xfrom
SamuraiWorf:fix/convert-auth-exceptions

Conversation

@SamuraiWorf

Copy link
Copy Markdown
Contributor

Pretty niche issue, but caused some more serious exceptions.

-> 1. Only have worldedit.undo.self permission. 2. Run //undo anything

@SamuraiWorf
SamuraiWorf requested a review from a team as a code owner April 6, 2026 17:45
@octylFractal

Copy link
Copy Markdown
Member

What happens currently? The existing WorldEditException handler should already be mapping this properly.

Maybe there's some null issues? I think we should just pass the proper Component in at call sites for now, and maybe deprecate having null for Component in WorldEditException in a separate PR.

@SamuraiWorf

Copy link
Copy Markdown
Contributor Author

Yeah exactly, its the null issues. Switched to fixing it at the call sites.

@me4502
me4502 merged commit 51ec8a8 into EngineHub:version/7.4.x Apr 7, 2026
2 checks passed
TheFather007 pushed a commit to TheFather007/WorldEdit that referenced this pull request Jun 29, 2026
… creation (EngineHub#2938)

* convert authorization exceptions (see undo.other)

* fix at callsites

* remove unused import

* make default constructor usable

* remove unused imports

(cherry picked from commit 51ec8a8)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants