Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#42562 closed defect (bug) (invalid)

media: wp_get_attachment_metadata() can return WP_Error object which leads to Fatal error

Reported by: indigo74's profile indigo74 Owned by:
Milestone: Priority: normal
Severity: normal Version: 4.8.3
Component: Media Keywords:
Focuses: Cc:

Description

Dear all,

I searched for an existing ticket, but I didn't found one. Sorry if I missed something.

Lately, when opening the Media library, I was just seeing a spinning circle and nothing shown. It was a call to admin-ajax.php which returns Error 500.

After activating WP_DEBUG, I saw this error:

PHP Fatal error:  Uncaught Error: Cannot use object of type WP_Error as array in /home/clients/ef4f22c1fd864ad322af8c3169dede6f/web/next/wp-includes/media.php:3145
Stack trace:
#0 [internal function]: wp_prepare_attachment_for_js(Object(WP_Post))
#1 /home/clients/ef4f22c1fd864ad322af8c3169dede6f/web/next/wp-admin/includes/ajax-actions.php(2479): array_map('wp_prepare_atta...', Array)
#2 /home/clients/ef4f22c1fd864ad322af8c3169dede6f/web/next/wp-includes/class-wp-hook.php(298): wp_ajax_query_attachments('')
#3 /home/clients/ef4f22c1fd864ad322af8c3169dede6f/web/next/wp-includes/class-wp-hook.php(323): WP_Hook->apply_filters('', Array)
#4 /home/clients/ef4f22c1fd864ad322af8c3169dede6f/web/next/wp-includes/plugin.php(453): WP_Hook->do_action(Array)
#5 /home/clients/ef4f22c1fd864ad322af8c3169dede6f/web/next/wp-admin/admin-ajax.php(91): do_action('wp_ajax_query-a...')
#6 {main}
  thrown in /home/clients/ef4f22c1fd864ad322af8c3169dede6f/web/next/wp-includes/media.php on line 3145

Looking at media.php, function wp_prepare_attachment_for_js(), line 3145:
(https://core.trac.wordpress.org/browser/tags/4.8/src/wp-includes/media.php#L3141)

<?php
function wp_prepare_attachment_for_js( $attachment ) {
        if ( ! $attachment = get_post( $attachment ) )
                return;

        if ( 'attachment' != $attachment->post_type )
                return;

        $meta = wp_get_attachment_metadata( $attachment->ID ); /* <-- $meta is retrieved here */
        
        /* ... */

        if ( isset( $meta['filesize'] ) ) { /* <-- ERROR IS HERE, LINE 3145 */
                $bytes = $meta['filesize'];
        } elseif ( file_exists( $attached_file ) ) {
                $bytes = filesize( $attached_file );
        } else {
                $bytes = '';
        }

        /* ... */
}

In this case, $meta is set to WP_Error object.

If I print_r($meta) before the issue, I get:

WP_Error Object
(
    [errors] => Array
        (
            [skip_svg_generate] => Array
                (
                    [0] => Skipping SVG file.
                )

        )

    [error_data] => Array
        (
        )

)

The reason is because I have an old SVG file. It was uploaded using the SAFE SVG plugin. This plugin was removed, but the SVG file stayed.

Anyway, I believe a check is missing on the $meta variable to gracefully handle this kind of case.

This is the first time I open a ticket in WordPress, I hope I explained it well.

Regards
Indigo.

Change History (2)

#1 @indigo74
7 years ago

I corrected the issue by doing this:

<?php
function wp_prepare_attachment_for_js( $attachment ) {
        if ( ! $attachment = get_post( $attachment ) )
                return;

        if ( 'attachment' != $attachment->post_type )
                return;

        $meta = wp_get_attachment_metadata( $attachment->ID );
        
        /* ... */
        
        // Line added below
        if (is_wp_error($meta)) return apply_filters( 'wp_prepare_attachment_for_js', $response, $attachment, $meta );
        // End of modification

        if ( isset( $meta['filesize'] ) ) { 
                $bytes = $meta['filesize'];
        } elseif ( file_exists( $attached_file ) ) {
                $bytes = filesize( $attached_file );
        } else {
                $bytes = '';
        }

        /* ... */
}

I don't know if this is the proper way to handle this case... I'll let someone more versed in WordPress to fix this :)

#2 @desrosj
7 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to invalid
  • Status changed from new to closed

Hi @indigo74! Welcome to Trac!

This appears to be a bug in the Safe SVG plugin. The latest version (1.5.3) looks to have a fix for the issue you are experiencing. Since this is not a bug with Core, I am going to close this out.

If you continue to experience the issue, a support request on the plugin's forums would be the best place to get assistance resolving this bug.

Note: See TracTickets for help on using tickets.