PLEASE NOTE: These forums are no longer utilized and are provided as an archive for informational purposes only. All support issues will be handled via email using our support ticket system. For more detailed information on this change, please see this blog post.

BUG REPORT & FIX

  1. Hi-

    I found a bug in gravity forms that I thought had been fixed but is still in the latest release.

    form_display.php line 77

    if(is_array($confirmation) && isset($confirmation["redirect"])){
                    header("Location: {$confirmation["redirect"]}");
                    do_action("gform_post_submission", $lead, $form);
                    do_action("gform_post_submission_{$form["id"]}", $lead, $form);
    
                    exit;
                }

    The header function should be moved AFTER the action calls as they will not be called with this current ordering.

    Let me know if I can answer any questions or give more detail.

    Thanks,

    Kevin

    PS Is there another place for bug submissions?

    Posted 12 years ago on Tuesday December 20, 2011 | Permalink
  2. Hi Kevin,

    Thanks for sharing. We introduced a new hook gform_after_submission and have deprecated the use of the gform_post_submission hook due to this bug. We did not want to fix this for fear it would impact how other users currently had configured their code for this hook (it was one of the first GF hooks). Anywhere you are using gform_post_submission you should now use gform_after_submission (for version 1.6 and greater).

    Posted 12 years ago on Tuesday December 20, 2011 | Permalink
  3. Hi David-

    Makes sense to me :) Thank you so much for the quick reply and the pointer, I have updated my code and everything is solid.

    Thanks,

    Kevin

    Posted 12 years ago on Wednesday December 21, 2011 | Permalink
  4. My pleasure. Thanks for taking the initiative to share a solution. :)

    Posted 12 years ago on Wednesday December 21, 2011 | Permalink

This topic has been resolved and has been closed to new replies.