Changeset 60420 for branches/6.8/tests/phpunit/tests/media.php

Timestamp:

07/07/2025 05:07:55 PM (6 months ago)

Author:

audrasjb

Message:

Media: expose height and width attributes to the wp_get_attachment_image_attributes filter.

Include the image height and width in the attributes passed to the wp_get_attachment_image_attributes filter. Developers can use this to adjust the width and height attributes returned from the wp_get_attachment_image_attributes function.

Reviewed by audrasjb.
Merges [60415] to the 6.8 branch.
Props divinenephron, nacin, Sam_a, wpsmith, anatolbroder, ericlewis, puggan, SergeyBiryukov, spacedmonkey, adamsilverstein, flixos90, sandeepdahiya, SirLouen.
Fixes #14110.

Location:

branches/6.8

Files:

• . (modified) (1 prop)
• tests/phpunit/tests/media.php (modified) (1 diff)

branches/6.8/tests/phpunit/tests/media.php

@@ -1583 +1583 @@
 
         $this->assertSame( $expected, $output );
+    }
+
+    /**
+     * @ticket 14110
+     */
+    public function test_wp_get_attachment_image_filter_with_height_width() {
+        $mock_action = new MockAction();
+        add_filter( 'wp_get_attachment_image_attributes', array( $mock_action, 'filter' ) );
+        wp_get_attachment_image( self::$large_id );
+        $args = $mock_action->get_args();
+        $this->assertArrayHasKey( '0', $args, 'First argument should be an array.' );
+        $this->assertArrayHasKey( '0', $args[0], 'First argument should be an array.' );
+        $this->assertArrayHasKey( 'width', $args[0][0], 'Width should be set.' );
+        $this->assertArrayHasKey( 'height', $args[0][0], 'Height should be set.' );
+    }
+
+    /**
+     * @ticket 14110
+     */
+    public function test_wp_get_attachment_image_filter_change_height_width() {
+        add_filter(
+            'wp_get_attachment_image_attributes',
+            static function ( $args ) {
+                $args['height'] = '999';
+                $args['width']  = '999';
+                return $args;
+            }
+        );
+        $output = wp_get_attachment_image( self::$large_id );
+        $this->assertStringContainsString( 'width="999"', $output, 'Width should be changed.' );
+        $this->assertStringContainsString( 'height="999"', $output, 'Height should be changed.' );
+    }
+
+    /**
+     * @ticket 14110
+     */
+    public function test_wp_get_attachment_image_filter_unset_height_width() {
+        add_filter(
+            'wp_get_attachment_image_attributes',
+            static function ( $args ) {
+                unset( $args['height'], $args['width'] );
+                return $args;
+            }
+        );
+        $output = wp_get_attachment_image( self::$large_id );
+        $this->assertStringContainsString( 'width="150"', $output, 'Width should not be changed.' );
+        $this->assertStringContainsString( 'height="150"', $output, 'Height should not be changed.' );
     }