Twenty Twenty-One: twenty_twenty_one_get_attachment_image_attributes() missing wp_get_attachment_metadata() check

[justlevine]

Im getting a PHP notice from inc\template-functions.php::twenty_twenty_one_get_attachment_image_attributes() about accessing an array value on a boolean.

Issue stems from:

<?php
$meta = wp_get_attachment_metadata( $attachment->ID );
if ( $meta['width'] && $meta['height'] ) {
  $width  = (int) $meta['width'];
  $height = (int) $meta['height'];
}

since wp_get_attachment_meta() can return false when no attachment id exists.

Trac stresses me out, but here's the fix:

<?php
$meta = wp_get_attachment_metadata( $attachment->ID );
if( false === $meta ) {
  return $attr;
}
if ( $meta['width'] && $meta['height'] ) {
  $width  = (int) $meta['width'];
  $height = (int) $meta['height'];
}

[SergeyBiryukov]

Hi there, welcome back to WordPress Trac!

Thanks for the report, this is already fixed for WordPress 5.9 in [52217] / #54464.