Dead code in media command class

[ernilambar]

Bug Report

• [✓] Yes, I reviewed the contribution guidelines.
• [✓] Yes, more specifically, I reviewed the guidelines on how to write clear bug reports.

Describe the current, buggy behavior

Line 627 : https://github.com/wp-cli/media-command/blob/main/src/Media_Command.php#L627

There is conditional of is_wp_error() for $metadata which is output of wp_generate_attachment_metadata(). Since this function always return an array, my VSCode is flagging as error. Not sure about the background of the code. So mentioning here as an issue.

Ref: https://developer.wordpress.org/reference/functions/wp_generate_attachment_metadata/

[danielbachhuber]

It looks like this code has been in here for quite a while:

media-command/php/commands/media.php

Lines 70 to 75 in 8717808

	$metadata = wp_generate_attachment_metadata( $image->ID, $fullsizepath );
	if ( is_wp_error( $metadata ) ) {
	WP_CLI::warning( $metadata->get_error_message() );
	return;
	}

I'd be open to removing it if we have sufficient test coverage around it (e.g. a test case for the empty attachment data scenario).