If the parse call throws an exception anywhere, the parse exception will propagate, so any potentially known commands with permission conditions won't get checked.
This means that the end user sees this:
instead of this:
It would likely be necessary to allow parse to return partial results to verify, or for parse to validate conditions while parsing (which is probably out-of-scope for a parser).
In
CommandManager#execute: https://github.com/EngineHub/Piston/blob/v0.3.0/core/src/main/java/org/enginehub/piston/CommandManager.java#L145-L146If the
parsecall throws an exception anywhere, the parse exception will propagate, so any potentially known commands with permission conditions won't get checked.This means that the end user sees this:
instead of this:
It would likely be necessary to allow parse to return partial results to verify, or for parse to validate conditions while parsing (which is probably out-of-scope for a parser).