WordPress.org
Get WordPress

Make WordPress Core


Ignore:
Timestamp:
11/30/2017 11:09:33 PM (7 years ago)
Author:
pento
Message:

Code is Poetry.
WordPress' code just... wasn't.
This is now dealt with.

Props jrf, pento, netweb, GaryJ, jdgrimes, westonruter, Greg Sherwood from PHPCS, and everyone who's ever contributed to WPCS and PHPCS.
Fixes #41057.

File:
1 edited

Legend:

Unmodified
Added
Removed

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

    r41288 r42343  
    4646
    4747        // First, test Imagick's extension and classes.
    48         if ( ! extension_loaded( 'imagick' ) || ! class_exists( 'Imagick', false ) || ! class_exists( 'ImagickPixel', false ) )
     48        if ( ! extension_loaded( 'imagick' ) || ! class_exists( 'Imagick', false ) || ! class_exists( 'ImagickPixel', false ) ) {
    4949            return false;
    50 
    51         if ( version_compare( phpversion( 'imagick' ), '2.2.0', '<' ) )
     50        }
     51
     52        if ( version_compare( phpversion( 'imagick' ), '2.2.0', '<' ) ) {
    5253            return false;
     54        }
    5355
    5456        $required_methods = array(
    … …  
    7577
    7678        // Now, test for deep requirements within Imagick.
    77         if ( ! defined( 'imagick::COMPRESSION_JPEG' ) )
     79        if ( ! defined( 'imagick::COMPRESSION_JPEG' ) ) {
    7880            return false;
     81        }
    7982
    8083        $class_methods = array_map( 'strtolower', get_class_methods( 'Imagick' ) );
    … …  
    104107        $imagick_extension = strtoupper( self::get_extension( $mime_type ) );
    105108
    106         if ( ! $imagick_extension )
     109        if ( ! $imagick_extension ) {
    107110            return false;
     111        }
    108112
    109113        // setIteratorIndex is optional unless mime is an animated format.
    110114        // Here, we just say no if you are missing it and aren't loading a jpeg.
    111         if ( ! method_exists( 'Imagick', 'setIteratorIndex' ) && $mime_type != 'image/jpeg' )
     115        if ( ! method_exists( 'Imagick', 'setIteratorIndex' ) && $mime_type != 'image/jpeg' ) {
    112116                return false;
     117        }
    113118
    114119        try {
    115120            return ( (bool) @Imagick::queryFormats( $imagick_extension ) );
    116         }
    117         catch ( Exception $e ) {
     121        } catch ( Exception $e ) {
    118122            return false;
    119123        }
    … …  
    128132     */
    129133    public function load() {
    130         if ( $this->image instanceof Imagick )
     134        if ( $this->image instanceof Imagick ) {
    131135            return true;
    132 
    133         if ( ! is_file( $this->file ) && ! preg_match( '|^https?://|', $this->file ) )
    134             return new WP_Error( 'error_loading_image', __('File doesn&#8217;t exist?'), $this->file );
     136        }
     137
     138        if ( ! is_file( $this->file ) && ! preg_match( '|^https?://|', $this->file ) ) {
     139            return new WP_Error( 'error_loading_image', __( 'File doesn&#8217;t exist?' ), $this->file );
     140        }
    135141
    136142        /*
    … …  
    141147
    142148        try {
    143             $this->image = new Imagick();
     149            $this->image    = new Imagick();
    144150            $file_extension = strtolower( pathinfo( $this->file, PATHINFO_EXTENSION ) );
    145             $filename = $this->file;
     151            $filename       = $this->file;
    146152
    147153            if ( 'pdf' == $file_extension ) {
    … …  
    153159            $this->image->readImage( $filename );
    154160
    155             if ( ! $this->image->valid() )
    156                 return new WP_Error( 'invalid_image', __('File is not an image.'), $this->file);
     161            if ( ! $this->image->valid() ) {
     162                return new WP_Error( 'invalid_image', __( 'File is not an image.' ), $this->file );
     163            }
    157164
    158165            // Select the first frame to handle animated images properly
    159             if ( is_callable( array( $this->image, 'setIteratorIndex' ) ) )
    160                 $this->image->setIteratorIndex(0);
     166            if ( is_callable( array( $this->image, 'setIteratorIndex' ) ) ) {
     167                $this->image->setIteratorIndex( 0 );
     168            }
    161169
    162170            $this->mime_type = $this->get_mime_type( $this->image->getImageFormat() );
    163         }
    164         catch ( Exception $e ) {
     171        } catch ( Exception $e ) {
    165172            return new WP_Error( 'invalid_image', $e->getMessage(), $this->file );
    166173        }
    … …  
    194201                $this->image->setImageCompressionQuality( $quality );
    195202                $this->image->setImageCompression( imagick::COMPRESSION_JPEG );
    196             }
    197             else {
     203            } else {
    198204                $this->image->setImageCompressionQuality( $quality );
    199205            }
    200         }
    201         catch ( Exception $e ) {
     206        } catch ( Exception $e ) {
    202207            return new WP_Error( 'image_quality_error', $e->getMessage() );
    203208        }
    … …  
    218223    protected function update_size( $width = null, $height = null ) {
    219224        $size = null;
    220         if ( !$width || !$height ) {
     225        if ( ! $width || ! $height ) {
    221226            try {
    222227                $size = $this->image->getImageGeometry();
    223             }
    224             catch ( Exception $e ) {
     228            } catch ( Exception $e ) {
    225229                return new WP_Error( 'invalid_image', __( 'Could not read image size.' ), $this->file );
    226230            }
    227231        }
    228232
    229         if ( ! $width )
     233        if ( ! $width ) {
    230234            $width = $size['width'];
    231 
    232         if ( ! $height )
     235        }
     236
     237        if ( ! $height ) {
    233238            $height = $size['height'];
     239        }
    234240
    235241        return parent::update_size( $width, $height );
    … …  
    251257     */
    252258    public function resize( $max_w, $max_h, $crop = false ) {
    253         if ( ( $this->size['width'] == $max_w ) && ( $this->size['height'] == $max_h ) )
     259        if ( ( $this->size['width'] == $max_w ) && ( $this->size['height'] == $max_h ) ) {
    254260            return true;
     261        }
    255262
    256263        $dims = image_resize_dimensions( $this->size['width'], $this->size['height'], $max_w, $max_h, $crop );
    257         if ( ! $dims )
    258             return new WP_Error( 'error_getting_dimensions', __('Could not calculate resized image dimensions') );
     264        if ( ! $dims ) {
     265            return new WP_Error( 'error_getting_dimensions', __( 'Could not calculate resized image dimensions' ) );
     266        }
    259267        list( $dst_x, $dst_y, $src_x, $src_y, $dst_w, $dst_h, $src_w, $src_h ) = $dims;
    260268
    … …  
    336344             */
    337345            if ( is_callable( array( $this->image, 'sampleImage' ) ) ) {
    338                 $resize_ratio = ( $dst_w / $this->size['width'] ) * ( $dst_h / $this->size['height'] );
     346                $resize_ratio  = ( $dst_w / $this->size['width'] ) * ( $dst_h / $this->size['height'] );
    339347                $sample_factor = 5;
    340348
    … …  
    399407                $this->image->setInterlaceScheme( Imagick::INTERLACE_NO );
    400408            }
    401 
    402         }
    403         catch ( Exception $e ) {
     409        } catch ( Exception $e ) {
    404410            return new WP_Error( 'image_resize_error', $e->getMessage() );
    405411        }
    … …  
    429435     */
    430436    public function multi_resize( $sizes ) {
    431         $metadata = array();
    432         $orig_size = $this->size;
     437        $metadata   = array();
     438        $orig_size  = $this->size;
    433439        $orig_image = $this->image->getImage();
    434440
    435441        foreach ( $sizes as $size => $size_data ) {
    436             if ( ! $this->image )
     442            if ( ! $this->image ) {
    437443                $this->image = $orig_image->getImage();
     444            }
    438445
    439446            if ( ! isset( $size_data['width'] ) && ! isset( $size_data['height'] ) ) {
    … …  
    453460
    454461            $resize_result = $this->resize( $size_data['width'], $size_data['height'], $size_data['crop'] );
    455             $duplicate = ( ( $orig_size['width'] == $size_data['width'] ) && ( $orig_size['height'] == $size_data['height'] ) );
     462            $duplicate     = ( ( $orig_size['width'] == $size_data['width'] ) && ( $orig_size['height'] == $size_data['height'] ) );
    456463
    457464            if ( ! is_wp_error( $resize_result ) && ! $duplicate ) {
    … …  
    464471                if ( ! is_wp_error( $resized ) && $resized ) {
    465472                    unset( $resized['path'] );
    466                     $metadata[$size] = $resized;
     473                    $metadata[ $size ] = $resized;
    467474                }
    468475            }
    … …  
    498505        try {
    499506            $this->image->cropImage( $src_w, $src_h, $src_x, $src_y );
    500             $this->image->setImagePage( $src_w, $src_h, 0, 0);
     507            $this->image->setImagePage( $src_w, $src_h, 0, 0 );
    501508
    502509            if ( $dst_w || $dst_h ) {
    503510                // If destination width/height isn't specified, use same as
    504511                // width/height from source.
    505                 if ( ! $dst_w )
     512                if ( ! $dst_w ) {
    506513                    $dst_w = $src_w;
    507                 if ( ! $dst_h )
     514                }
     515                if ( ! $dst_h ) {
    508516                    $dst_h = $src_h;
     517                }
    509518
    510519                $thumb_result = $this->thumbnail_image( $dst_w, $dst_h );
    … …  
    515524                return $this->update_size();
    516525            }
    517         }
    518         catch ( Exception $e ) {
     526        } catch ( Exception $e ) {
    519527            return new WP_Error( 'image_crop_error', $e->getMessage() );
    520528        }
    … …  
    536544         */
    537545        try {
    538             $this->image->rotateImage( new ImagickPixel('none'), 360-$angle );
     546            $this->image->rotateImage( new ImagickPixel( 'none' ), 360 - $angle );
    539547
    540548            // Normalise Exif orientation data so that display is consistent across devices.
    … …  
    545553            // Since this changes the dimensions of the image, update the size.
    546554            $result = $this->update_size();
    547             if ( is_wp_error( $result ) )
     555            if ( is_wp_error( $result ) ) {
    548556                return $result;
     557            }
    549558
    550559            $this->image->setImagePage( $this->size['width'], $this->size['height'], 0, 0 );
    551         }
    552         catch ( Exception $e ) {
     560        } catch ( Exception $e ) {
    553561            return new WP_Error( 'image_rotate_error', $e->getMessage() );
    554562        }
    … …  
    567575    public function flip( $horz, $vert ) {
    568576        try {
    569             if ( $horz )
     577            if ( $horz ) {
    570578                $this->image->flipImage();
    571 
    572             if ( $vert )
     579            }
     580
     581            if ( $vert ) {
    573582                $this->image->flopImage();
    574         }
    575         catch ( Exception $e ) {
     583            }
     584        } catch ( Exception $e ) {
    576585            return new WP_Error( 'image_flip_error', $e->getMessage() );
    577586        }
    … …  
    592601
    593602        if ( ! is_wp_error( $saved ) ) {
    594             $this->file = $saved['path'];
     603            $this->file      = $saved['path'];
    595604            $this->mime_type = $saved['mime-type'];
    596605
    597606            try {
    598607                $this->image->setImageFormat( strtoupper( $this->get_extension( $this->mime_type ) ) );
    599             }
    600             catch ( Exception $e ) {
     608            } catch ( Exception $e ) {
    601609                return new WP_Error( 'image_save_error', $e->getMessage(), $this->file );
    602610            }
    … …  
    607615
    608616    /**
    609      *
    610617     * @param Imagick $image
    611618     * @param string $filename
    … …  
    616623        list( $filename, $extension, $mime_type ) = $this->get_output_format( $filename, $mime_type );
    617624
    618         if ( ! $filename )
     625        if ( ! $filename ) {
    619626            $filename = $this->generate_filename( null, null, $extension );
     627        }
    620628
    621629        try {
    … …  
    628636            // Reset original Format
    629637            $this->image->setImageFormat( $orig_format );
    630         }
    631         catch ( Exception $e ) {
     638        } catch ( Exception $e ) {
    632639            return new WP_Error( 'image_save_error', $e->getMessage(), $filename );
    633640        }
    634641
    635642        // Set correct file permissions
    636         $stat = stat( dirname( $filename ) );
     643        $stat  = stat( dirname( $filename ) );
    637644        $perms = $stat['mode'] & 0000666; //same permissions as parent folder, strip off the executable bits
    638645        @ chmod( $filename, $perms );
    … …  
    669676            // Reset Image to original Format
    670677            $this->image->setImageFormat( $this->get_extension( $this->mime_type ) );
    671         }
    672         catch ( Exception $e ) {
     678        } catch ( Exception $e ) {
    673679            return new WP_Error( 'image_stream_error', $e->getMessage() );
    674680        }
    … …  
    720726                }
    721727            }
    722 
    723728        } catch ( Exception $e ) {
    724729            return new WP_Error( 'image_strip_meta_error', $e->getMessage() );
    … …  
    744749            // Only load the first page.
    745750            return $this->file . '[0]';
    746         }
    747         catch ( Exception $e ) {
     751        } catch ( Exception $e ) {
    748752            return new WP_Error( 'pdf_setup_failed', $e->getMessage(), $this->file );
    749753        }
Note: See TracChangeset for help on using the changeset viewer.