BUG REPORT & FIX « Gravity Support Forums

Gravity Support Forums » Plugin Support » Gravity Forms

Kevin Miller
Member
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 13 years ago on Tuesday December 20, 2011 | Permalink
David Smith
Member

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 13 years ago on Tuesday December 20, 2011 | Permalink
Kevin Miller
Member

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 13 years ago on Wednesday December 21, 2011 | Permalink
David Smith
Member

My pleasure. Thanks for taking the initiative to share a solution. :)

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

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