Changeset 10605 for sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-plugins/functions.php

Timestamp:

01/20/2021 04:37:47 PM (5 years ago)

Author:

ryelle

Message:

Plugin Directory: Remove unused React code.

As part of the Plugin Directory refresh #1719, the UI was rewritten into a React app, but currently only the screenshot slider uses React. This removes all the React JS except for the screenshots, and flattens the Sass into less complex files. Additionally it updates the packages and JS build process to use @wordpress/scripts.

Fixes #5431.

File:

• sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-plugins/functions.php (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-plugins/functions.php

@@ -116 +116 @@
     // React is currently only used on detail pages.
     if ( is_single() ) {
-        wp_enqueue_script( 'wporg-plugins-client', get_stylesheet_directory_uri() . '/js/theme.js', array(), '20200527', true );
-        wp_localize_script( 'wporg-plugins-client', 'pluginDirectory', array(
-            'endpoint' => untrailingslashit( rest_url() ), // 'https://wordpress.org/plugins-wp/wp-json',
-            'nonce'    => wp_create_nonce( 'wp_rest' ),
-            'base'     => get_blog_details()->path,
-            'userId'   => get_current_user_id(),
-        ) );
-        wp_localize_script( 'wporg-plugins-client', 'localeData', array(
-            ''            => array(
-                'Plural-Forms' => _x( 'nplurals=2; plural=n != 1;', 'plural forms', 'wporg-plugins' ),
-                'Language'     => _x( 'en', 'language (fr, fr_CA)', 'wporg-plugins' ),
-                'localeSlug'   => _x( 'en', 'locale slug', 'wporg-plugins' ),
-            ),
-            'screenshots' => __( 'Screenshots', 'wporg-plugins' ),
-        ) );
+        $assets_path = dirname( __FILE__ ) . '/js/build/theme.asset.php';
+        if ( file_exists( $assets_path ) ) {
+            $script_info = require( $assets_path );
+            wp_enqueue_script(
+                'wporg-plugins-client',
+                get_stylesheet_directory_uri() . '/js/build/theme.js',
+                $script_info['dependencies'],
+                $script_info['version'],
+                true
+            );
+            wp_localize_script(
+                'wporg-plugins-client',
+                'localeData',
+                array(
+                    '' => array(
+                        'Plural-Forms' => _x( 'nplurals=2; plural=n != 1;', 'plural forms', 'wporg-plugins' ),
+                        'Language'     => _x( 'en', 'language (fr, fr_CA)', 'wporg-plugins' ),
+                        'localeSlug'   => _x( 'en', 'locale slug', 'wporg-plugins' ),
+                    ),
+                    'screenshots' => __( 'Screenshots', 'wporg-plugins' ),
+                )
+            );
+        }
     }