getMessage() ); /* This used to use 0 (ExifBitmapHandler::OLD_BROKEN_FILE) for the cases * * No metadata in the file * * Something is broken in the file. * However, if the metadata support gets expanded then you can't tell if the 0 is from * a broken file, or just no props found. A broken file is likely to stay broken, but * a file which had no props could have props once the metadata support is improved. * Thus switch to using -1 to denote only a broken file, and use an array with only * MEDIAWIKI_EXIF_VERSION to denote no props. */ return ExifBitmapHandler::BROKEN_FILE; } } /** * @param File $file * @param array $params Rotate parameters. * 'rotation' clockwise rotation in degrees, allowed are multiples of 90 * @since 1.21 * @return bool|MediaTransformError */ public function rotate( $file, $params ) { global $wgJpegTran; $rotation = ( $params['rotation'] + $this->getRotation( $file ) ) % 360; if ( $wgJpegTran && is_executable( $wgJpegTran ) ) { $command = Shell::command( $wgJpegTran, '-rotate', $rotation, '-outfile', $params['dstPath'], $params['srcPath'] ); $result = $command ->includeStderr() ->execute(); if ( $result->getExitCode() !== 0 ) { $this->logErrorForExternalProcess( $result->getExitCode(), $result->getStdout(), $command ); return new MediaTransformError( 'thumbnail_error', 0, 0, $result->getStdout() ); } return false; } else { return parent::rotate( $file, $params ); } } public function supportsBucketing() { return true; } public function sanitizeParamsForBucketing( $params ) { $params = parent::sanitizeParamsForBucketing( $params ); // Quality needs to be cleared for bucketing. Buckets need to be default quality if ( isset( $params['quality'] ) ) { unset( $params['quality'] ); } return $params; } /** * @inheritDoc */ protected function transformImageMagick( $image, $params ) { global $wgUseTinyRGBForJPGThumbnails; $ret = parent::transformImageMagick( $image, $params ); if ( $ret ) { return $ret; } if ( $wgUseTinyRGBForJPGThumbnails ) { // T100976 If the profile embedded in the JPG is sRGB, swap it for the smaller // (and free) TinyRGB /** * We'll want to replace the color profile for JPGs: * * in the sRGB color space, or with the sRGB profile * (other profiles will be left untouched) * * without color space or profile, in which case browsers * should assume sRGB, but don't always do (e.g. on wide-gamut * monitors (unless it's meant for low bandwith) * @see https://phabricator.wikimedia.org/T134498 */ $colorSpaces = [ self::SRGB_EXIF_COLOR_SPACE, '-' ]; $profiles = [ self::SRGB_ICC_PROFILE_DESCRIPTION ]; // we'll also add TinyRGB profile to images lacking a profile, but // only if they're not low quality (which are meant to save bandwith // and we don't want to increase the filesize by adding a profile) if ( isset( $params['quality'] ) && $params['quality'] > 30 ) { $profiles[] = '-'; } $this->swapICCProfile( $params['dstPath'], $colorSpaces, $profiles, realpath( __DIR__ ) . '/tinyrgb.icc' ); } return false; } /** * Swaps an embedded ICC profile for another, if found. * Depends on exiftool, no-op if not installed. * @param string $filepath File to be manipulated (will be overwritten) * @param array $colorSpaces Only process files with this/these Color Space(s) * @param array $oldProfileStrings Exact name(s) of color profile to look for * (the one that will be replaced) * @param string $profileFilepath ICC profile file to apply to the file * @since 1.26 * @return bool */ public function swapICCProfile( $filepath, array $colorSpaces, array $oldProfileStrings, $profileFilepath ) { global $wgExiftool; if ( !$wgExiftool || !is_executable( $wgExiftool ) ) { return false; } $result = Shell::command( $wgExiftool, '-EXIF:ColorSpace', '-ICC_Profile:ProfileDescription', '-S', '-T', $filepath ) ->includeStderr() ->execute(); // Explode EXIF data into an array with [0 => Color Space, 1 => Device Model Desc] $data = explode( "\t", trim( $result->getStdout() ) ); if ( $result->getExitCode() !== 0 ) { return false; } // Make a regex out of the source data to match it to an array of color // spaces in a case-insensitive way $colorSpaceRegex = '/' . preg_quote( $data[0], '/' ) . '/i'; if ( empty( preg_grep( $colorSpaceRegex, $colorSpaces ) ) ) { // We can't establish that this file matches the color space, don't process it return false; } $profileRegex = '/' . preg_quote( $data[1], '/' ) . '/i'; if ( empty( preg_grep( $profileRegex, $oldProfileStrings ) ) ) { // We can't establish that this file has the expected ICC profile, don't process it return false; } $command = Shell::command( $wgExiftool, '-overwrite_original', '-icc_profile<=' . $profileFilepath, $filepath ); $result = $command ->includeStderr() ->execute(); if ( $result->getExitCode() !== 0 ) { $this->logErrorForExternalProcess( $result->getExitCode(), $result->getStdout(), $command ); return false; } return true; } }