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.

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

Fixed do_shortcode error #133

Merged
merged 1 commit into from
Feb 8, 2014
Merged

Fixed do_shortcode error #133

merged 1 commit into from
Feb 8, 2014

Conversation

jtwiest
Copy link

@jtwiest jtwiest commented Jan 25, 2014

Using do_shortcode() to render a contact form was preventing it from submitting correctly.

This fix adds a copy of the shortcode in post meta to be called if the shortocode can't be found in the content or the widget.

#102

@ptasker
Copy link

ptasker commented Feb 3, 2014

+1 Have encountered same bug and this fix works.

@blobaugh
Copy link
Contributor

blobaugh commented Feb 3, 2014

Thanks @ptasker!

@blobaugh blobaugh added this to the 2.9 Freeze milestone Feb 6, 2014
blobaugh added a commit that referenced this pull request Feb 8, 2014
@blobaugh blobaugh merged commit f5929ce into Automattic:master Feb 8, 2014
@jacobischwartz
Copy link

This works in submitting the form and saving the comment, but doesn't send the email. Can the others who tested this confirm whether they are able to send the email?

@yannandco
Copy link

Tried to used do_shortcode.
Email is sent to author but doesn't care of the email or subject in shortcode.
Ex: [contact-form to="[email protected]" subject="MY CONTACT FORM" ], it doesn't send to [email protected] but to the author of the post (idem for subject )

@jeherve
Copy link
Member

jeherve commented Apr 15, 2014

@yannandco Would you mind opening a new issue for this issue, including the code you used to insert do_shortcode in your template file?

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Contact Form [Type] Bug When a feature is broken and / or not performing as intended
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants