Merge pull request #444 from BeAPI/feat/register_script_strategy

Modification du paramètre `(bool) $in_footer` => `(array) $strategy` pour `wp_register_script`

Author: firestar300

inc/Framework.php

@@ -5,12 +5,10 @@
 use BEA\Theme\Framework\Services\Acf;
 use BEA\Theme\Framework\Services\Assets;
 use BEA\Theme\Framework\Services\Performance;
-use BEA\Theme\Framework\Services\Assets_JS_Async;
 use BEA\Theme\Framework\Services\Editor;
 use BEA\Theme\Framework\Services\Editor_Patterns;
 use BEA\Theme\Framework\Services\Gravity_Forms;
 use BEA\Theme\Framework\Services\Menu;
-use BEA\Theme\Framework\Services\Sidebar;
 use BEA\Theme\Framework\Services\Svg;
 use BEA\Theme\Framework\Services\Theme;
 use BEA\Theme\Framework\Tools\Body_Class;
@@ -35,7 +33,6 @@ class Framework {
 		Theme::class,
 		Assets::class,
 		Performance::class,
-		Assets_JS_Async::class,
 		Editor::class,
 		Editor_Patterns::class,
 		Gravity_Forms::class,

inc/Services/Assets.php

@@ -68,7 +68,7 @@ public function register_assets(): void {
 			'dist/' . $file,
 			array_merge( [ 'jquery' ], $asset_data['dependencies'] ), // ensure jQuery dependency is set even if not declared explicitly in the JS
 			$asset_data['version'],
-			true
+			[ 'strategy' => 'defer' ]
 		);
 
 		wp_add_inline_script(

inc/Services/Assets_JS_Async.php

@@ -55,7 +55,6 @@ private function add_theme_supports(): void {
 		add_theme_support( 'post-thumbnails' );
 		add_theme_support( 'html5', [ 'comment-list', 'comment-form', 'search-form', 'gallery', 'caption', 'script', 'style' ] );
 		add_theme_support( 'title-tag' );
-		add_theme_support( 'async-js' );
 		add_theme_support( 'yoast-seo-breadcrumbs' );
 	}
 

inc/Services/Theme.php

@@ -17,12 +17,12 @@ class Assets {
 	 * @param $src : Have to be a relative filename
 	 * @param array $deps
 	 * @param mixed $ver
-	 * @param bool $in_footer
+	 * @param array $strategy
 	 *
 	 * @return bool
 	 */
-	public function register_script( string $handle, string $src, array $deps = [], $ver = false, bool $in_footer = false ): bool {
-		return \wp_register_script( $handle, \get_theme_file_uri( $src ), $deps, $ver, $in_footer );
+	public function register_script( string $handle, string $src, array $deps = [], $ver = false, array $strategy = [] ): bool {
+		return \wp_register_script( $handle, \get_theme_file_uri( $src ), $deps, $ver, $strategy );
 	}
 
 	/**

inc/Tools/Assets.php

@@ -6,4 +6,5 @@ Version: 1.0.0
 Author: BeAPI
 Author URI: http://www.beapi.fr
 Text Domain: beapi-frontend-framework
+Requires at least: 6.3
 */