WorkflowConfigTransitionListBuilder doesn't actually output a list of entities, or do any of the work that its base class is meant for.

Instead, its render() method returning a form from one of two other classes. It seems to me it would be simpler to declare these forms for those routes, and then remove the list builder.

Comments

joachim created an issue. See original summary.

  • johnv committed 54ab48b on 8.x-1.x
    Issue #2697031: WorkflowConfigTransitionListBuilder serves no purpose
    
johnv’s picture

Version: 8.x-1.x-dev » 8.x-1.0-beta5
Status: Active » Fixed

Your advice has been applied. Thanks.

  • johnv committed 1d9e8db on 8.x-1.x
    Issue #2697031: WorkflowConfigTransitionListBuilder serves no purpose
    
johnv’s picture

Also the canonical link "/admin/config/workflow/workflow/{workflow_type}" is now removed.

Status: Fixed » Closed (fixed)

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