Skip to content

Fix broken editor when passing attributes as undefined or null and add warning for block authors #73149

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
Nov 20, 2025
Merged

Fix broken editor when passing attributes as undefined or null and add warning for block authors #73149

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
82e6464
Ensure attributes is always an object, never null or undefined
jeryj Nov 10, 2025
43a1493
Revert setting empty attributes object in favor of warning within reg…
jeryj Nov 13, 2025
1c1fee6
Revert "Revert setting empty attributes object in favor of warning wi…
jeryj Nov 20, 2025
e67d081
Add warning when passing attributes as undefined and inform that this…
jeryj Nov 20, 2025
d7c5041
Update warning message
jeryj Nov 20, 2025
2a06573
Add unit tests for block registration with undefined, null, or no att…
jeryj Nov 20, 2025
99c3dbb
Use it to avoid duplication in unit test
jeryj Nov 20, 2025
ac10381
Remove temporary messaging about the fix being removed
jeryj Nov 20, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Use it to avoid duplication in unit test 
Co-authored-by: Marin Atanasov <8436925+tyxla@users.noreply.github.com>
  • Loading branch information
@jeryj @tyxla
jeryj and tyxla authored Nov 20, 2025
commit 99c3dbb00243a290691a2c6db67989e8ecc06380
52 changes: 21 additions & 31 deletions packages/blocks/src/api/test/registration.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -320,39 +320,29 @@ describe( 'blocks', () => {
expect( blockType.attributes ).toEqual( {} );
} );

it( 'should warn and default to empty object when attributes is undefined', () => {
registerBlockType( 'core/test-block-undefined-attributes', {
title: 'block title',
category: 'text',
save: noop,
attributes: undefined,
} );

expect( console ).toHaveWarnedWith(
'The block "core/test-block-undefined-attributes" must not register attributes as `null` or `undefined`. Use an empty object (`attributes: {}`) or exclude the `attributes` key. In the next Gutenberg release, passing `null` or `undefined` for `attributes` will cause the block editor to crash.'
);

const blockType = getBlockType(
'core/test-block-undefined-attributes'
);
expect( blockType.attributes ).toEqual( {} );
} );

it( 'should warn and default to empty object when attributes is null', () => {
registerBlockType( 'core/test-block-null-attributes', {
title: 'block title',
category: 'text',
save: noop,
attributes: null,
} );
it.each( [
[ 'undefined', undefined ],
[ 'null', null ],
] )(
'should warn and default to empty object when attributes is %s',
( _label, value ) => {
registerBlockType( 'core/test-block-null-attributes', {
title: 'block title',
category: 'text',
save: noop,
attributes: value,
} );

expect( console ).toHaveWarnedWith(
'The block "core/test-block-null-attributes" must not register attributes as `null` or `undefined`. Use an empty object (`attributes: {}`) or exclude the `attributes` key. In the next Gutenberg release, passing `null` or `undefined` for `attributes` will cause the block editor to crash.'
);
expect( console ).toHaveWarnedWith(
'The block "core/test-block-null-attributes" must not register attributes as `null` or `undefined`. Use an empty object (`attributes: {}`) or exclude the `attributes` key. In the next Gutenberg release, passing `null` or `undefined` for `attributes` will cause the block editor to crash.'
);

const blockType = getBlockType( 'core/test-block-null-attributes' );
expect( blockType.attributes ).toEqual( {} );
} );
const blockType = getBlockType(
'core/test-block-null-attributes'
);
expect( blockType.attributes ).toEqual( {} );
}
);

it( 'should default to browser-initialized global attributes', () => {
const attributes = { ok: { type: 'boolean' } };
Expand Down
Loading