Currently a Rules event is being invoked at the end of webform_workflow_transition(). While this is great for those who use Rules, I don't see any way for other code to run on state transition.

I'd like to see a hook invoked there for custom modules to implement.

CommentFileSizeAuthor
#5 webform_workflow-invoke_transition_hook-2775815-5.patch1.48 KBlambic
#3 webform_workflow-invoke_transition_hook-2775815-2.patch1.2 KBdtarc
#2 webform_workflow-invoke_transition_hook-2775815-1.patch1.3 KBdtarc
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dtarc created an issue. See original summary.

dtarc’s picture

dtarc CreditAttribution: dtarc at Affinity Bridge commented
Status: Active » Needs review
FileSize
webform_workflow-invoke_transition_hook-2775815-1.patch1.3 KB

Here's a patch.

dtarc’s picture

dtarc CreditAttribution: dtarc at Affinity Bridge commented
FileSize
webform_workflow-invoke_transition_hook-2775815-2.patch1.2 KB

Here's a better patch

pjcdawkins’s picture

pjcdawkins CreditAttribution: pjcdawkins as a volunteer commented
Assigned: dtarc » Unassigned
Status: Needs review » Reviewed & tested by the community
lambic’s picture

lambic CreditAttribution: lambic commented
FileSize
webform_workflow-invoke_transition_hook-2775815-5.patch1.48 KB

This hook was giving us problems when anything triggers token replacement, as that triggers a submission render, which tries to build the state form and ends up doing a redirect that kills everything. This patch addresses that by not rendering the form if the format is 'text'.