Merge pull request #221 from BoldGrid/issue-220

avonville · web-flow · commit 385a1e2000e3 · 2024-03-13T12:29:11.000-06:00
Check if a post is actually a form before trying to render it
diff --git a/includes/class-frontend-form.php b/includes/class-frontend-form.php
@@ -39,7 +39,7 @@ public function render_shortcode( $atts, $contents = '' ) {
 
         $form = weforms()->form->get( $id );
 
-        if ( !$form->id ) {
+        if ( !$form->id || 'wpuf_contact_form' !== $form->data->post_type ) {
             return $this->show_error( __( 'The form couldn\'t be found.', 'weforms' ) );
         }
 

Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ public function render_shortcode( $atts, $contents = '' ) {`
`39`	`39`
`40`	`40`	`$form = weforms()->form->get( $id );`
`41`	`41`
`42`		`- if ( !$form->id ) {`
	`42`	`+ if ( !$form->id \|\| 'wpuf_contact_form' !== $form->data->post_type ) {`
`43`	`43`	`return $this->show_error( __( 'The form couldn\'t be found.', 'weforms' ) );`
`44`	`44`	`}`
`45`	`45`