Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Error form_display.php on line 2250 #27

Closed
jcesak opened this issue Oct 12, 2015 · 3 comments
Closed

Error form_display.php on line 2250 #27

jcesak opened this issue Oct 12, 2015 · 3 comments

Comments

@jcesak
Copy link

jcesak commented Oct 12, 2015

Hi Brady,
Your plugin worked like a charm embedding a GravityForm into an iframe on my site at http://www.isoho.org/. But, an error now appears in the iframe at the top of the form, "Warning: Invalid argument supplied for foreach() in /home/mcwr2015/public_html/wp-content/plugins/gravityforms/form_display.php on line 2250".

There is no error/warning when using the same form on a regular page (NOT embedded in an iframe). See http://www.isoho.org/nominationform/.

Rob Harrell at gravityforms was unable to duplicate the issue on his server. He suggested that I contact you to see if you have some suggestions to address this problem.

Gravity Forms was just updated to Version 1.9.14 and WordPress is running on the latest Version 4.3.1.

Can you help? Thank you! Janet
bradyvercher added a commit that referenced this issue Oct 12, 2015
@bradyvercher
Pass the full form object into print_form_scripts() in the template. …
7e7e52f 
…See #27

GFFormDisplay::print_form_scripts() appears to expect the full form object
rather than just a form ID. This should prevent PHP warnings and may clear up
other issues.
@bradyvercher
Copy link
Member

Hi Janet, I was able to reproduce this locally and it appears to be a long-standing issue in the template file. I'm surprised it hasn't been brought up before.

I'm hoping the commit I just pushed up should fix it. You can test by downloading the development branch here.

Thanks for the report!

@bradyvercher bradyvercher mentioned this issue Oct 12, 2015
Closed
@jcesak
Copy link
Author

jcesak commented Oct 12, 2015

Thank you, Brady!!!!

Perfect.

Janet

From: Brady Vercher [mailto:notifications@github.com]
Sent: Monday, October 12, 2015 5:02 PM
To: bradyvercher/gravity-forms-iframe
Cc: Janet Cesak
Subject: Re: [gravity-forms-iframe] Error form_display.php on line 2250 (#27)

Hi Janet, I was able to reproduce this locally and it appears to be a long-standing issue in the template file. I'm surprised it hasn't been brought up before.

I'm hoping the commit I just pushed up should fix it. You can test by downloading the development branchhttps://github.com/bradyvercher/gravity-forms-iframe/archive/develop.zip here.

Thanks for the report!

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/27#issuecomment-147533715.

@bradyvercher bradyvercher mentioned this issue Oct 12, 2015
Closed
@bradyvercher
Copy link
Member

Thanks for the update and glad to hear that took care of it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@bradyvercher @jcesak