[EXPORTING] Add named, labeled transitions

+1 for transition labels. This is a must for workflow-in-core-in-5.0.

Most workflow systems use the transition labels as labels for buttons that execute the transition.

In thinking about a very basic default workflow and UI for Drupal, I like the idea of comments not enabled by default, and clicking on the button executes the transition. That's about as simple as it gets.

For advanced workflows we could enable commenting and the workflow tab.

i've already implemented this if anyone would like the code

Attaching a patch here would be nifty.

subscribing

I got MrTaco's code and made a patch, but didn't see this issue and put it at #227038: Add named transition states. Then I apparently made an even bigger mess by making http://drupal.org/project/workflow_named_transitions. There are some bugs in it and I am working through them, but is this something I should be working back into core workflow or is it better off an a separate module?

I'd like to see it in core workflow, even if it's not required by default (requiring named transitions makes configuration tedious for basic workflows).

If could have a 'next' or 'default' transition that would go to the next node if it hadn't been created, but could be overridden if that transition had been explicitly defined. The caveat being that I would want to be able to force the 'next' transition to return to the same state so that I could put my Error states after the completion states and not worry about a default transition moving into the error state.

Transition labels from POV is important as it should achieve the following :

1. By defining labels will define also what is possible transitions like "Draft->Reviewed", "Reviewed->Checked" and "Checked->Published" only.So transition like "Draft->Published" if not desirable without defining a label for it should be available.
2. Also the transition labels will be used in "Workflow Edit" rather than see table with rows and columns cout of workflow stats. As for example if the wok flow has 4 stats then a table with 25 cells ((4+1)^2) will be in edit screen and if workflow has 8 stats then a table with 81 CELLS!!! will be there.
   
   And more worst in each cell all roles will be there (really Something not desirable).
   But Using labels in workflow with 8 stats (normaly will be in it less than 15 transition) so this will result in table with 2 columns only with 15 rows (Simpler to read and work with).
3. The project workflow_named_transitions Should be built-in Workflow itself

I've released a module that builds on the lovely "Workflow Named Transitions" module (without being dependent on it). It takes the concept one step further to get rid of the radio buttons altogether, replacing them by single action submit buttons.
See Workflow extensions

This seems UI related and button related and generally a good idea once other issues are resolved. Saving this one in the queue for now.

Bumping this up to 7.x to be backported to 6 once finished in 7.

I'm in favor of leaving this in Workflow Extensions unless Rik wants to become a maintainer here and merge it.

Rik doesn't feel like becoming a maintainer of Workflow -- too much work!
Rik

Yeah. Please remind me, why did I become one?

Because you're doing a great job, Nancy!
Massive efforts, first by Bastlynn and now you.
We're all eagerly awaiting 7.x-1.1 becoming official...
Rik

Re-opening this:
- Workflow Extensions is not compatible anymore with 7.x-2.x
- Workflow Named Transitions is not compatible anymore with 7.x-2.x
See #867186: [EXPORTING] Add machine names to workflows and workflow states for adding machine names.

Wow, a five-digit issue can now be closed.

I've taken the code from workflow_named_transitions, and added it to this module.

It required:
- one extra field in the workflow_transitions table;
- one extra page in workflow_admin_ui;
- few lines in WorkflowState->getOptions;

The patch is rather big, but includes some re-organizing of files.

It does not include token support from workflow_extensions. See #2249211: Add token support for Transition labels.

Transition label UI form do not display data under 'from' state column except for first row.
Attached patch resolved for me. Could you please check if this is fine.

This was done on purpose for better UI. I set first column on change of from-state. Else you have a long list en get lost .
Do you think the data is wrong after this explanation?

condition $old_state == $new_state returns false as they are different states and print '' from second row on wards..
Attached screen shot with data issue.

Ah, you must save the transitions first. Until now the stayOnThisState was not saved. Now it is but you must resave them .
This needs an update hook or help text.
[EDIT] After our private conversation, it shows that update.php did not run, yet.

Agree with comment in #25 and patch in #24 is not required. Thanks.