Changeset 13716 for trunk/src/bp-groups/bp-groups-blocks.php

Timestamp:

01/28/2024 06:19:59 PM (18 months ago)

Author:

imath

Message:

BP Blocks: make sure default block titles can be translated

Some of the BP Blocks are using a title attribute into their block.json. Unfortunately, block attributes do not seem to be include in the translatable strings of the WP Block Metadata API.

To fix this, all block.json title attributes now default to an empty string & the default title value is handled within the PHP callback to render the block.

For the specific case of the Latest Activities block, this commit also fixes some errors that were preventing it to be able to select the list of Activity types to include into the rendered activities.

See #9074 (trunk)
Closes ​https://github.com/buddypress/buddypress/pull/223

File:

• trunk/src/bp-groups/bp-groups-blocks.php (modified) (2 diffs)

trunk/src/bp-groups/bp-groups-blocks.php

@@ -371 +371 @@
         $attributes,
         array(
-            'title'        => __( 'Groups', 'buddypress' ),
+            'title'        => '',
             'maxGroups'    => 5,
             'groupDefault' => 'active',
@@ -377 +377 @@
         )
     );
+
+    if ( ! $block_args['title'] ) {
+        $block_args['title'] = __( 'Groups', 'buddypress' );
+    }
 
     $classnames         = 'widget_bp_groups_widget buddypress widget';