Skip to content

Bug/24286 create block error #24287

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
youknowriad merged 10 commits into from
Jul 30, 2020
Merged

Bug/24286 create block error #24287

Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
96b832f
Merge remote-tracking branch 'upstream/master'
Mar 19, 2020
e5a20f4
Merge branch 'master' of github.com:ryanwelcher/gutenberg
May 29, 2020
b5d9c57
Merge branch 'master' of github.com:ryanwelcher/gutenberg
Jun 12, 2020
f2caf37
Merge branch 'master' of github.com:ryanwelcher/gutenberg
Jul 30, 2020
4843ff7
Add warning to createBlock.
Jul 30, 2020
937de91
Add tests for new change.
Jul 30, 2020
b1dad82
Update tests to actually work.
Jul 30, 2020
594b9c4
Add changes to package-lock.json
Jul 30, 2020
d125251
Throw an exception instead of a warning.
Jul 30, 2020
1e68ce0
Update docs.
Jul 30, 2020
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
4 changes: 4 additions & 0 deletions packages/blocks/src/api/factory.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,10 @@ export function createBlock( name, attributes = {}, innerBlocks = [] ) {
// Get the type definition associated with a registered block.
const blockType = getBlockType( name );

if ( undefined === blockType ) {
Copy link
Contributor

@youknowriad youknowriad Jul 30, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not important but yoda conditions are not a guideline in JavaScript because we enforce strict equality.

throw new Error( `Block type '${ name }' is not registered.` );
}

// Ensure attributes contains only values defined by block type, and merge
// default values for missing attributes.
const sanitizedAttributes = reduce(
Expand Down