Changeset 13464 for trunk/src/bp-members/bp-members-cssjs.php

Timestamp:

04/25/2023 10:17:33 PM (3 years ago)

Author:

imath

Message:

Update all BP Blocks & Modern JS PHP registration code

• Use the metadata argument of the bp_register_block() function to load all BP Blocks JSON files.
• Get BP Blocks dependencies using the @wordpress/scripts index.asset.php generated files.
• Although the BP Blocks category has been replaced by the BP Blocks collection, leave the no more used bp_block_category() function in place in case some third party plugins are using it for their custom blocks.
• PHPUnit tests: make sure to unregister all BP Blocks when using the BP_UnitTestCase->go_to() method.

See #8842
Fixes #8457
Closes ​https://github.com/buddypress/buddypress/pull/92

File:

• trunk/src/bp-members/bp-members-cssjs.php (modified) (2 diffs)

trunk/src/bp-members/bp-members-cssjs.php

@@ -15 +15 @@
  *
  * @since 9.0.0
+ * @since 12.0.0 Uses the `@wordpress/scripts` `index.asset.php` generated file to get dependencies.
  *
  * @param array $scripts Data about the scripts to register.
@@ -20 +21 @@
  */
 function bp_members_register_scripts( $scripts = array() ) {
+        if ( ! bp_support_blocks() ) {
+                return $scripts;
+        }
+
+        $asset      = array(
+                'dependencies' => array(),
+                'version'      => ''
+        );
+        $asset_path = trailingslashit( dirname( __FILE__ ) ) . 'blocks/dynamic-widget/index.asset.php';
+
+        if ( file_exists( $asset_path ) ) {
+                $asset = require $asset_path;
+        }
+
         $scripts['bp-dynamic-members-script'] = array(
+                'file'         => plugins_url( 'blocks/dynamic-widget/index.js', __FILE__ ),
+                'dependencies' => $asset['dependencies'],
                 'footer'       => true,
-                'file'         => plugins_url( 'js/dynamic-members.js', __FILE__ ),
-                'dependencies' => array(
-                        'bp-dynamic-widget-block-script',
-                        'wp-i18n',
-                ),
         );