Fix issue Automattic#8546 - remove window.require and window.define f…

…rom global scope after using a require definition
adamziel · Jan 17, 2018 · a0966bb · a0966bb
1 parent ac952d5
commit a0966bb
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/modules/shortcodes/mailchimp.php b/modules/shortcodes/mailchimp.php
@@ -199,6 +199,6 @@ static function shortcode( $lcase_attrs ) {
 
 		$displayed_once = true;
 
-		return "\n\n" . '<script type="text/javascript" data-dojo-config="' . esc_attr( implode( ', ', $config_vars ) ) . '">jQuery.getScript( "//downloads.mailchimp.com/js/signup-forms/popup/embed.js", function( data, textStatus, jqxhr ) { require(["mojo/signup-forms/Loader"], function(L) { L.start(' . wp_json_encode( $js_vars ) . ') }); } );</script>' . "\n\n";
+		return "\n\n" . '<script type="text/javascript" data-dojo-config="' . esc_attr( implode( ', ', $config_vars ) ) . '">jQuery.getScript( "//downloads.mailchimp.com/js/signup-forms/popup/embed.js", function( data, textStatus, jqxhr ) { require(["mojo/signup-forms/Loader"], function(L) { L.start(' . wp_json_encode( $js_vars ) . ') }); window.require = window.define = undefined; } );</script>' . "\n\n";
 	}
 }