Changeset 50819

Timestamp:

05/06/2021 11:09:34 AM (3 years ago)

Author:

SergeyBiryukov

Message:

Media: Remove _wp_webp_is_lossy() for now.

The function was only used in a single place in core.

Follow-up to [50810], [50814], [50815], [50818].

Props johnjamesjacoby.
See #35725.

Location:

trunk/src/wp-includes

Files:

• class-wp-image-editor-imagick.php (modified) (1 diff)
• media.php (modified) (1 diff)

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

@@ -204 +204 @@
                     break;
                 case 'image/webp':
-                    if ( _wp_webp_is_lossy( $this->file ) ) {
+                    $webp_info = wp_get_webp_info( $this->file );
+                    $type      = $webp_info['type'];
+
+                    if ( $type && 'lossy' === $type ) {
                         $this->image->setImageCompressionQuality( $quality );
                     } else {

trunk/src/wp-includes/media.php

@@ -5116 +5116 @@
     return compact( 'width', 'height', 'type' );
 }
-
-/**
- * Determines if a passed image is a lossy WebP image.
- *
- * @since 5.8.0
- *
- * @param string $filename The file path.
- * @return bool Whether the file is a lossy WebP file.
- */
-function _wp_webp_is_lossy( $filename ) {
-    $webp_info = wp_get_webp_info( $filename );
-    $type      = $webp_info['type'];
-
-    return $type && 'lossy' === $type;
-}