@automattic/newspack-blocks 4.29.0 → 4.29.1

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+## @automattic/newspack-blocks [4.29.1](https://github.com/Automattic/newspack-workspace/compare/newspack-blocks@4.29.0...newspack-blocks@4.29.1) (2026-08-04)
+
+
+### Bug Fixes
+
+* **author-profile:** hide default avatars when the toggle is on ([#570](https://github.com/Automattic/newspack-workspace/issues/570)) ([4aba9cd](https://github.com/Automattic/newspack-workspace/commit/4aba9cd016d8b5645358640dd27f9ff9070239b1))
+
 # @automattic/newspack-blocks [4.29.0](https://github.com/Automattic/newspack-workspace/compare/newspack-blocks@4.28.0...newspack-blocks@4.29.0) (2026-08-03)
 
 

package/newspack-blocks.php

@@ -7,7 +7,7 @@
  * Author URI:      https://newspack.com/
  * Text Domain:     newspack-blocks
  * Domain Path:     /languages
- * Version:         4.29.0
+ * Version:         4.29.1
  *
  * @package         Newspack_Blocks
  */
@@ -15,7 +15,7 @@
 define( 'NEWSPACK_BLOCKS__PLUGIN_FILE', __FILE__ );
 define( 'NEWSPACK_BLOCKS__BLOCKS_DIRECTORY', 'dist/' );
 define( 'NEWSPACK_BLOCKS__PLUGIN_DIR', plugin_dir_path( NEWSPACK_BLOCKS__PLUGIN_FILE ) );
-define( 'NEWSPACK_BLOCKS__VERSION', '4.29.0' );
+define( 'NEWSPACK_BLOCKS__VERSION', '4.29.1' );
 
 require_once NEWSPACK_BLOCKS__PLUGIN_DIR . 'includes/class-newspack-blocks.php';
 require_once NEWSPACK_BLOCKS__PLUGIN_DIR . 'includes/class-newspack-blocks-api.php';

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@automattic/newspack-blocks",
-	"version": "4.29.0",
+	"version": "4.29.1",
 	"author": "Automattic",
 	"description": "=== Newspack Blocks === Contributors: (this should be a list of wordpress.org userid's) Donate link: https://example.com/ Tags: comments, spam Requires at least: 4.5 Tested up to: 5.1.1 Stable tag: 0.1.0 License: GPLv2 or later License URI: https://www.gnu.org/licenses/gpl-2.0.html",
 	"repository": {

package/src/blocks/author-profile/class-wp-rest-newspack-authors-controller.php

@@ -210,9 +210,9 @@ class WP_REST_Newspack_Authors_Controller extends WP_REST_Controller {
 							$guest_author = ( new CoAuthors_Guest_Authors() )->get_guest_author_by( 'id', $guest_author->ID );
 
 							if ( in_array( 'avatar', $fields, true ) && function_exists( 'coauthors_get_avatar' ) ) {
-								$avatar = coauthors_get_avatar( $guest_author, 256 );
+								$avatar = coauthors_get_avatar( $guest_author, 256, $avatar_hide_default ? 'blank' : '' );
 
-								if ( $avatar && ( false === strpos( $avatar, 'avatar-default' ) || ! $avatar_hide_default ) ) {
+								if ( \Newspack_Blocks\is_avatar_displayable( $avatar, $avatar_hide_default ) ) {
 									$guest_author_data['avatar'] = $avatar;
 								}
 							}
@@ -238,9 +238,9 @@ class WP_REST_Newspack_Authors_Controller extends WP_REST_Controller {
 						];
 
 						if ( in_array( 'avatar', $fields, true ) ) {
-							$avatar = get_avatar( $user->data->ID, 256 );
+							$avatar = get_avatar( $user->data->ID, 256, $avatar_hide_default ? 'blank' : '' );
 
-							if ( $avatar && ( false === strpos( $avatar, 'avatar-default' ) || ! $avatar_hide_default ) ) {
+							if ( \Newspack_Blocks\is_avatar_displayable( $avatar, $avatar_hide_default ) ) {
 								$user_data['avatar'] = $avatar;
 							}
 						}
@@ -501,8 +501,8 @@ class WP_REST_Newspack_Authors_Controller extends WP_REST_Controller {
 				];
 
 				if ( in_array( 'avatar', $fields, true ) ) {
-					$avatar = get_avatar( $user->data->ID, 256 );
-					if ( $avatar && ( false === strpos( $avatar, 'avatar-default' ) || ! $avatar_hide_default ) ) {
+					$avatar = get_avatar( $user->data->ID, 256, $avatar_hide_default ? 'blank' : '' );
+					if ( \Newspack_Blocks\is_avatar_displayable( $avatar, $avatar_hide_default ) ) {
 						$user_data['avatar'] = $avatar;
 					}
 				}
@@ -551,8 +551,8 @@ class WP_REST_Newspack_Authors_Controller extends WP_REST_Controller {
 			];
 
 			if ( in_array( 'avatar', $fields, true ) && function_exists( 'coauthors_get_avatar' ) ) {
-				$avatar = coauthors_get_avatar( $guest_author, 256 );
-				if ( $avatar && ( false === strpos( $avatar, 'avatar-default' ) || ! $avatar_hide_default ) ) {
+				$avatar = coauthors_get_avatar( $guest_author, 256, $avatar_hide_default ? 'blank' : '' );
+				if ( \Newspack_Blocks\is_avatar_displayable( $avatar, $avatar_hide_default ) ) {
 					$author_data['avatar'] = $avatar;
 				}
 			}
@@ -588,8 +588,8 @@ class WP_REST_Newspack_Authors_Controller extends WP_REST_Controller {
 				];
 
 				if ( in_array( 'avatar', $fields, true ) && function_exists( 'coauthors_get_avatar' ) ) {
-					$avatar = coauthors_get_avatar( $guest_author, 256 );
-					if ( $avatar && ( false === strpos( $avatar, 'avatar-default' ) || ! $avatar_hide_default ) ) {
+					$avatar = coauthors_get_avatar( $guest_author, 256, $avatar_hide_default ? 'blank' : '' );
+					if ( \Newspack_Blocks\is_avatar_displayable( $avatar, $avatar_hide_default ) ) {
 						$author_data['avatar'] = $avatar;
 					}
 				}
@@ -607,8 +607,8 @@ class WP_REST_Newspack_Authors_Controller extends WP_REST_Controller {
 		];
 
 		if ( in_array( 'avatar', $fields, true ) ) {
-			$avatar = get_avatar( $user->data->ID, 256 );
-			if ( $avatar && ( false === strpos( $avatar, 'avatar-default' ) || ! $avatar_hide_default ) ) {
+			$avatar = get_avatar( $user->data->ID, 256, $avatar_hide_default ? 'blank' : '' );
+			if ( \Newspack_Blocks\is_avatar_displayable( $avatar, $avatar_hide_default ) ) {
 				$user_data['avatar'] = $avatar;
 			}
 		}

package/src/blocks/author-profile/view.php

@@ -150,7 +150,7 @@ function newspack_blocks_get_author_or_guest_author( $author_id, $avatar_size =
 		}
 
 		$author = ( new CoAuthors_Guest_Authors() )->get_guest_author_by( 'id', $author_id );
-		$avatar = function_exists( 'coauthors_get_avatar' ) ? coauthors_get_avatar( $author, $avatar_size ) : false;
+		$avatar = function_exists( 'coauthors_get_avatar' ) ? coauthors_get_avatar( $author, $avatar_size, $hide_default ? 'blank' : '' ) : false;
 
 		// Format CAP guest author object to return to the render function.
 		if ( $author && isset( $author->ID ) ) {
@@ -161,7 +161,7 @@ function newspack_blocks_get_author_or_guest_author( $author_id, $avatar_size =
 				$author
 			);
 
-			if ( $avatar && ( false === strpos( $avatar, 'avatar-default' ) || ! $hide_default ) ) {
+			if ( \Newspack_Blocks\is_avatar_displayable( $avatar, $hide_default ) ) {
 				$author['avatar'] = $avatar;
 			}
 		}
@@ -176,8 +176,8 @@ function newspack_blocks_get_author_or_guest_author( $author_id, $avatar_size =
 			$wp_user
 		);
 
-		$avatar = get_avatar( $author_id, $avatar_size );
-		if ( $avatar && ( false === strpos( $avatar, 'avatar-default' ) || ! $hide_default ) ) {
+		$avatar = get_avatar( $author_id, $avatar_size, $hide_default ? 'blank' : '' );
+		if ( \Newspack_Blocks\is_avatar_displayable( $avatar, $hide_default ) ) {
 			$author['avatar'] = $avatar;
 		}
 	}

package/src/shared/authors.php

@@ -61,3 +61,29 @@ function get_authors_roles_slugs() {
 		get_authors_roles()
 	);
 }
+
+/**
+ * Whether an avatar should be displayed, honoring a "hide default avatar" preference.
+ *
+ * Gravatar-served images never carry core's `avatar-default` class (core only
+ * emits it when no avatar was found, and the Gravatar branch always reports
+ * found), so a generated fallback like `d=mm` is indistinguishable from a real
+ * avatar by class alone. When hiding defaults, callers must fetch the avatar
+ * with Gravatar's `blank` fallback so it can be detected via the `d=blank`
+ * param — mirroring the Author List block's long-standing behavior.
+ *
+ * @param string|false $avatar       Avatar HTML as returned by get_avatar()/coauthors_get_avatar().
+ * @param bool         $hide_default Whether default (fallback) avatars should be hidden.
+ *
+ * @return bool True if the avatar should be displayed.
+ */
+function is_avatar_displayable( $avatar, $hide_default ) {
+	if ( ! $avatar ) {
+		return false;
+	}
+	if ( ! $hide_default ) {
+		return true;
+	}
+	$is_default = false !== strpos( $avatar, 'avatar-default' ) || false !== strpos( $avatar, 'd=blank' );
+	return ! $is_default;
+}

package/vendor/composer/installed.php

@@ -3,7 +3,7 @@
         'name' => 'automattic/newspack-blocks',
         'pretty_version' => 'dev-main',
         'version' => 'dev-main',
-        'reference' => '8a5fd59b2162a2b3cce706342177c7fc65f9f215',
+        'reference' => '4aba9cd016d8b5645358640dd27f9ff9070239b1',
         'type' => 'wordpress-plugin',
         'install_path' => __DIR__ . '/../../',
         'aliases' => array(),
@@ -13,7 +13,7 @@
         'automattic/newspack-blocks' => array(
             'pretty_version' => 'dev-main',
             'version' => 'dev-main',
-            'reference' => '8a5fd59b2162a2b3cce706342177c7fc65f9f215',
+            'reference' => '4aba9cd016d8b5645358640dd27f9ff9070239b1',
             'type' => 'wordpress-plugin',
             'install_path' => __DIR__ . '/../../',
             'aliases' => array(),