There's an explicit call to drupal_redirect_form in workbench_moderation_moderate_form_submit that prevents additional submit handlers from getting added and called. Specifically, I would like to add submit handler after moderate_form_submit to capture additional information related to workbench_moderation_node_history.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

walkah’s picture

Status: Active » Needs review
FileSize
389 bytes

simple 1-line patch.

hass’s picture

teknic’s picture

Is this going to be rolled in to a future release?

sunnydeveloper’s picture

Any update?

hefox’s picture

Status: Needs review » Reviewed & tested by the community

Please review patches you want to see in future releases of a module, so that they get in

teknic’s picture

Issue summary: View changes

I've reviewed this patch and have it running on two production sites currently with no negative impacts on the functionality of workbench moderation.

Johnny vd Laar’s picture

Works for me!

colan’s picture

Status: Reviewed & tested by the community » Fixed

Thanks!

  • colan committed 15d29d6 on 7.x-1.x authored by walkah
    Issue #1515110 by walkah: Removed call to drupal_redirect_form.
    
colan’s picture

Version: 7.x-1.x-dev » 7.x-2.x-dev
Status: Fixed » Patch (to be ported)

Might need to be forward ported.

  • colan committed 15d29d6 on 7.x-3.x authored by walkah
    Issue #1515110 by walkah: Removed call to drupal_redirect_form.