- Issue created by @joachim
- Status changed to Fixed
11 months ago 9:56pm 14 March 2024 Automatically closed - issue fixed for 2 weeks with no activity.
Changes to an entity should be validated.
This is going to require some refactoring.
First, does this belong in checkReachable() or checkOperability()?
- checkOperability doesn't concern itself with direction, so probably not
- checkReachable doesn't call through to the state action plugin, but it does call getNextStateName() on the plugin
Maybe this should be handled in getNextStateName() -- the new value is calculated there anyway, so in addition:
1. set it on the entity
2. try to validate
If validation fails, then in fact the attempted value is NOT a valid next state. Therefore return NULL, and that gets back to checkReachable().
Fixed
1.0
Code
Automatically closed - issue fixed for 2 weeks with no activity.