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

Timestamp:

04/25/2023 10:17:33 PM (20 months 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',
-        ),
     );