Block API: Fallback to all attributes when checking for unmodified block

Mamaduka · Mamaduka · commit 7067c1400a8f · 2025-12-10T08:19:18.000+04:00
diff --git a/packages/blocks/src/api/test/utils.js b/packages/blocks/src/api/test/utils.js
@@ -495,6 +495,14 @@ describe( 'isUnmodifiedBlock', () => {
 		expect( isUnmodifiedBlock( block, 'non-existent-role' ) ).toBe( true );
 	} );
 
+	it( 'should return false if no attributes exist for the role and some are modified', () => {
+		const block = createBlock( 'core/test-block', {
+			align: 'center',
+			content: 'Updated content',
+		} );
+		expect( isUnmodifiedBlock( block, 'non-existent-role' ) ).toBe( false );
+	} );
+
 	it( 'should return true if metadata attributes is not modified for role content', () => {
 		const block = createBlock( 'core/test-block' );
 		expect( isUnmodifiedBlock( block, 'content' ) ).toBe( true );
diff --git a/packages/blocks/src/api/utils.js b/packages/blocks/src/api/utils.js
@@ -43,20 +43,27 @@ export function isUnmodifiedBlock( block, role ) {
 	const blockAttributes = getBlockType( block.name )?.attributes ?? {};
 
 	// Filter attributes by role if a role is provided.
-	const attributesToCheck = role
+	const attributesByRole = role
 		? Object.entries( blockAttributes ).filter( ( [ key, definition ] ) => {
 				// A special case for the metadata attribute.
 				// It can include block bindings that serve as a source of content,
 				// without directly modifying content attributes.
 				if ( role === 'content' && key === 'metadata' ) {
-					return true;
+					return (
+						Object.keys( block.attributes[ key ]?.bindings ?? {} )
+							.length > 0
+					);
 				}
 
 				return (
 					definition.role === role ||
 					definition.__experimentalRole === role
 				);
 		  } )
+		: [];
+	// Fallback to all attributes if no attributes match the role.
+	const attributesToCheck = !! attributesByRole.length
+		? attributesByRole
 		: Object.entries( blockAttributes );
 
 	return attributesToCheck.every( ( [ key, definition ] ) => {
