Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
With the committed PR https://github.com/fago/rules/pull/138 this has moved far already. Imo, we should do the same for component state definitions and then we are good with the general interfaces and DX already.
I've added new default implemenations of execute() and evalute() that foward calls to doExecute() and doEvalute(). That way we can use type-hinting and save getContextValue() methods. See EntityFetchByField as example.
Comments
Comment #1
dasjoComment #2
dasjodiscussion started, first interfaces done
Comment #3
fagoWith the committed PR https://github.com/fago/rules/pull/138 this has moved far already. Imo, we should do the same for component state definitions and then we are good with the general interfaces and DX already.
Comment #4
fagoI've added new default implemenations of execute() and evalute() that foward calls to doExecute() and doEvalute(). That way we can use type-hinting and save getContextValue() methods. See EntityFetchByField as example.