-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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.
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
Add optional chain to sizes indexing of media details in edit-site #68995
Conversation
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.
To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
👋 Thanks for your first Pull Request and for helping build the future of Gutenberg and WordPress, @grgar! In case you missed it, we'd love to have you join us in our Slack community. If you want to learn more about WordPress development in general, check out the Core Handbook full of helpful information. |
This optional chaining is already present in 21 other lines in Gutenberg where `sizes` (in `media_details`) is indexed, but is missing on these two lines. For some reason, `media_details` is an empty object for some of my attachments on my site, so this throws a TypeError. This change resolves this issue. Co-authored-by: grgar <grgarside@git.wordpress.org>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @grgar!
The patch works as expected ✅
…ordPress#68995) This optional chaining is already present in 21 other lines in Gutenberg where `sizes` (in `media_details`) is indexed, but is missing on these two lines. For some reason, `media_details` is an empty object for some of my attachments on my site, so this throws a TypeError. This change resolves this issue. Co-authored-by: grgar <grgarside@git.wordpress.org> Co-authored-by: Mamaduka <mamaduka@git.wordpress.org>
What?
This optional chaining is already present in 21 other lines in Gutenberg where
sizes
(inmedia_details
) is indexed, but is missing on these two lines.Why?
For some reason,
media_details
is an empty object for some of my attachments on my site, so this throws a TypeError. This change resolves this issue.How?
?.
is optional chaining to short circuit before the TypeError.Testing Instructions
I don't actually know how some attachments don't have media_details on my site in the first place. I tried to reproduce on a playground with some hooks from the real site, e.g.
/wp-admin/site-editor.php?postType=page
to trigger the page thumbnails to be shown in the list, which triggers this code path and throws the TypeError without this change.n/a from PR template
Testing Instructions for Keyboard
n/a
Screenshots or screencast
n/a