WordPress.org

Make WordPress Core

Changeset 49546


Ignore:
Timestamp:
11/09/2020 03:33:51 PM (3 months ago)
Author:
SergeyBiryukov
Message:

Coding Standards: Use consistent formatting for error messages in WP_Image_Editor_Imagick::write_image() and ::strip_meta().

See #50767.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-includes/class-wp-image-editor-imagick.php

    r49542 r49546  
    736736             */
    737737            if ( file_put_contents( $filename, $image->getImageBlob() ) === false ) {
    738                 /* translators: %s: PHP function name. */
    739                 return new WP_Error( 'image_save_error', sprintf( __( '%s failed while writing image to stream.' ), '<code>file_put_contents()</code>' ), $filename );
     738                return new WP_Error(
     739                    'image_save_error',
     740                    sprintf(
     741                        /* translators: %s: PHP function name. */
     742                        __( '%s failed while writing image to stream.' ),
     743                        '<code>file_put_contents()</code>'
     744                    ),
     745                    $filename
     746                );
    740747            } else {
    741748                return true;
    742749            }
    743750        } else {
    744             $dir_name   = dirname( $filename );
    745             $dir_exists = wp_mkdir_p( $dir_name );
    746 
    747             if ( ! $dir_exists ) {
     751            $dirname = dirname( $filename );
     752
     753            if ( ! wp_mkdir_p( $dirname ) ) {
    748754                return new WP_Error(
    749755                    'image_save_error',
     
    751757                        /* translators: %s: Directory path. */
    752758                        __( 'Unable to create directory %s. Is its parent directory writable by the server?' ),
    753                         esc_html( $dir_name )
     759                        esc_html( $dirname )
    754760                    )
    755761                );
     
    802808
    803809        if ( ! is_callable( array( $this->image, 'getImageProfiles' ) ) ) {
    804             /* translators: %s: ImageMagick method name. */
    805             return new WP_Error( 'image_strip_meta_error', sprintf( __( '%s is required to strip image meta.' ), '<code>Imagick::getImageProfiles()</code>' ) );
     810            return new WP_Error(
     811                'image_strip_meta_error',
     812                sprintf(
     813                    /* translators: %s: ImageMagick method name. */
     814                    __( '%s is required to strip image meta.' ),
     815                    '<code>Imagick::getImageProfiles()</code>'
     816                )
     817            );
    806818        }
    807819
    808820        if ( ! is_callable( array( $this->image, 'removeImageProfile' ) ) ) {
    809             /* translators: %s: ImageMagick method name. */
    810             return new WP_Error( 'image_strip_meta_error', sprintf( __( '%s is required to strip image meta.' ), '<code>Imagick::removeImageProfile()</code>' ) );
     821            return new WP_Error(
     822                'image_strip_meta_error',
     823                sprintf(
     824                    /* translators: %s: ImageMagick method name. */
     825                    __( '%s is required to strip image meta.' ),
     826                    '<code>Imagick::removeImageProfile()</code>'
     827                )
     828            );
    811829        }
    812830
Note: See TracChangeset for help on using the changeset viewer.