When adding new WorkflowStates to a Workflow, you must set the machine_name yourself. This should be done like we are used to from other entities.
Also, when adding a Workflow State with diacritical characters, an error appears.

Comments

johnv created an issue. See original summary.

  • johnv committed e629048 on 8.x-1.x
    Issue #2571093: D8-port: machine_name of WorkflowState is generated...
johnv’s picture

The machine_name is now automatically created in code. but a proper widget is preferrable.
Also a prefix with the workflow_name is needed: now, when adding a new Workflow, the '(creation)' state of older Workflows is removed. :-(

johnv’s picture

Title: D8-port: the machine_name of WorkflowState needs attention » [D8-port task] The machine_name of WorkflowState needs attention
Parent issue: » #2351299: Port Workflow module to Drupal 8
johnv’s picture

From several issues, the option to add a machine_name manually, seems to give problems, e.g., when adding a second workflow with a state that already exists. The machine_name must be unique system wide.
Hence, the following patch greyes the machine_name out in the state list.

  • johnv committed ef89b26 on 8.x-1.x
    Issue #2571093: [D8-port task] The machine_name of WorkflowState cannot...

  • johnv committed b0686a9 on 8.x-1.x
    Issue #2571093: The machine_name of WorkflowState CREATION is now...
johnv’s picture

Above commit fixes a problem with generation of the machine_name, related to #2658116-5: Validation of unique machine_name of WorkflowState fails

johnv’s picture

Version: 8.x-1.x-dev » 8.x-1.0-beta5
Issue summary: View changes
Status: Active » Fixed
Related issues: +#2571093: [D8-port task] The machine_name of WorkflowState needs attention

Status: Fixed » Closed (fixed)

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