Comments

dasjo’s picture

Priority: Critical » Normal
Issue summary: View changes
dasjo’s picture

Status: Active » Needs work

discussion started, first interfaces done

fago’s picture

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.

fago’s picture

Status: Needs work » Fixed

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.

  • 137c91c committed on 8.x-3.x
    Issue #2245031: Improve DX by invoking doExecute() and doEvaluate()...
  • e7b8a19 committed on 8.x-3.x
    Merge pull request #174 from fago/improve-dx
    
    Issue #2245031: Improve DX...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.