Changeset 13717 for branches/12.0/src/bp-blogs/bp-blogs-blocks.php

Timestamp:

01/28/2024 06:43:17 PM (2 years 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.

Fixes #9074 (branch 12.0)

File:

• branches/12.0/src/bp-blogs/bp-blogs-blocks.php (modified) (1 diff)

branches/12.0/src/bp-blogs/bp-blogs-blocks.php

@@ -27 +27 @@
         $attributes,
         array(
-            'title'     => __( 'Recent Networkwide Posts', 'buddypress' ),
+            'title'     => '',
             'maxPosts'  => 10,
             'linkTitle' => false,
         )
     );
+
+    if ( ! $block_args['title'] ) {
+        $block_args['title'] = __( 'Recent Networkwide Posts', 'buddypress' );
+    }
 
     $classnames           = 'widget_bp_blogs_widget buddypress widget';