php warning issue

I can find these two warnings when visiting the attachment page for an SVG because WordPress does not set their (full-size) dimensions in metadata.

Unless you would prefer disabling all image attachment pages, you could try adding a filter to set both SVG dimensions to 0:

function svg_attachment_metadata_dimensions( $data ) {
	if (
		( ! isset( $data['width'] ) || ! isset( $data['height'] ) )
		&& isset( $data['sizes']['thumbnail']['mime-type'] )
		&& $data['sizes']['thumbnail']['mime-type'] === 'image/svg+xml'
	) {
		$data['width'] = 0;
		$data['height'] = 0;
	}
	return $data;
}
add_filter( 'wp_get_attachment_metadata', 'svg_attachment_metadata_dimensions', 10, 1 );

If you have a child theme, you have multiple options. The simplest might be wrapping the printf in a condition like if ( isset( $metadata['width'] ) && isset( $metadata['height'] ) ). That would skip adding the entry meta if the image does not define width and/or height, so you might also want an else fallback.

Steps:

1. Add Twenty Eleven’s image.php template in your child theme (if you have not done that already).
2. Edit the printf code that begins after $metadata = wp_get_attachment_metadata(); with one of the code snippets below (or something else, as you like).
3. Save changes to the file, and upload it if you are not editing within the admin.
4. Visit the attachment pages for images, especially any pages you know had the error messages.

Snippet 1: Checks for width and height before outputting entry meta, and shows only the date if it does not find both dimensions.

<?php
	$metadata = wp_get_attachment_metadata();
if ( isset( $metadata['width'] ) && isset( $metadata['height'] ) ) :
	printf(
		/* translators: 1: Time, 2: Date, 3: Image permalink, 4: Image width, 5: Image height, 6: Parent permalink, 7: Parent post title, 8: Parent post title. */
		__( '<span class="meta-prep meta-prep-entry-date">Published </span> <span class="entry-date"><abbr class="published" title="%1$s">%2$s</abbr></span> at <a href="%3$s" title="Link to full-size image">%4$s &times; %5$s</a> in <a href="%6$s" title="Go to %7$s" rel="gallery">%8$s</a>', 'twentyeleven' ),
		esc_attr( get_the_time() ),
		get_the_date(),
		esc_url( wp_get_attachment_url() ),
		$metadata['width'],
		$metadata['height'],
		esc_url( get_permalink( $post->post_parent ) ),
		esc_attr( strip_tags( get_the_title( $post->post_parent ) ) ),
		get_the_title( $post->post_parent )
	);
else:
	printf(
		'<span class="entry-date"><abbr class="published" title="%1$s">%2$s</abbr></span>',
		esc_attr( get_the_time() ),
		get_the_date()
	);
endif;
?>

Snippet 2: Checks for width and height individually within the entry meta printf, and gives fallback text if the dimension is not found. (The fallback could be '0' or 'unknown', for example.)

<?php
	$metadata = wp_get_attachment_metadata();
	printf(
		/* translators: 1: Time, 2: Date, 3: Image permalink, 4: Image width, 5: Image height, 6: Parent permalink, 7: Parent post title, 8: Parent post title. */
		__( '<span class="meta-prep meta-prep-entry-date">Published </span> <span class="entry-date"><abbr class="published" title="%1$s">%2$s</abbr></span> at <a href="%3$s" title="Link to full-size image">%4$s &times; %5$s</a> in <a href="%6$s" title="Go to %7$s" rel="gallery">%8$s</a>', 'twentyeleven' ),
		esc_attr( get_the_time() ),
		get_the_date(),
		esc_url( wp_get_attachment_url() ),
		isset( $metadata['width'] ) ? $metadata['width'] : '0',
		isset( $metadata['height'] ) ? $metadata['height'] : '0',
		esc_url( get_permalink( $post->post_parent ) ),
		esc_attr( strip_tags( get_the_title( $post->post_parent ) ) ),
		get_the_title( $post->post_parent )
	);
?>

• This reply was modified 1 year, 1 month ago by Stephen Bernhardt.