fix(wpforms): resolves #343, fix for jquery dependency

bmarshall511 · bmarshall511 · commit 7bc5a9daeec6 · 2022-11-30T10:06:15.000-06:00
diff --git a/modules/wpforms/class-wpforms.php b/modules/wpforms/class-wpforms.php
@@ -170,9 +170,9 @@ public function add_scripts() {
 		// Only add scripts to the appropriate pages.
 		if ( 'enabled' === \ZeroSpam\Core\Settings::get_settings( 'verify_wpforms' ) ) {
 			wp_enqueue_script( 'zerospam-davidwalsh' );
-			add_action( 'wp_footer', function() {
-				echo '<script type="text/javascript">jQuery(".wpforms-form").ZeroSpamDavidWalsh();</script>';
-			}, 999 );
+			add_action( 'wp_enqueue_scripts', function() {
+				wp_add_inline_script('zerospam-davidwalsh', '(function($) {$(function() { $(".wpforms-form").ZeroSpamDavidWalsh(); })})(jQuery)' );
+			} );
 		}
 	}
 
diff --git a/readme.txt b/readme.txt
@@ -109,6 +109,7 @@ If hosting with Pantheon, see their [known issues page](https://pantheon.io/docs
 
 = v5.4.4 =
 
+* fix(wpforms): resolves #343, fix for jquery dependency
 * fix(registration): resolves #342, fix for failed registration output
 * fix(php8): resolves #341, fix for php8+ compatibility issue
 

Original file line number	Diff line number	Diff line change
`@@ -170,9 +170,9 @@ public function add_scripts() {`
`170`	`170`	`// Only add scripts to the appropriate pages.`
`171`	`171`	`if ( 'enabled' === \ZeroSpam\Core\Settings::get_settings( 'verify_wpforms' ) ) {`
`172`	`172`	`wp_enqueue_script( 'zerospam-davidwalsh' );`
`173`		`- add_action( 'wp_footer', function() {`
`174`		`- echo '<script type="text/javascript">jQuery(".wpforms-form").ZeroSpamDavidWalsh();</script>';`
`175`		`- }, 999 );`
	`173`	`+ add_action( 'wp_enqueue_scripts', function() {`
	`174`	`+ wp_add_inline_script('zerospam-davidwalsh', '(function($) {$(function() { $(".wpforms-form").ZeroSpamDavidWalsh(); })})(jQuery)' );`
	`175`	`+ } );`
`176`	`176`	`}`
`177`	`177`	`}`
`178`	`178`