Make WordPress Core


Ignore:
Timestamp:
04/02/2019 10:14:19 PM (8 years ago)
Author:
SergeyBiryukov
Message:

Widgets: Make sure changes to media widgets' instance schema via widget_{$this->id_base}_instance_schema filter are not overridden by subclasses.

Add unit tests missed in [45100].

Props Toro_Unit, birgire.
See #45029.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/tests/phpunit/tests/widgets/media-image-widget.php

    r43571 r45101  
    3535                $widget = new WP_Widget_Media_Image();
    3636                $schema = $widget->get_instance_schema();
    37 
    3837                $this->assertEqualSets(
    3938                        array(
    … …  
    5958
    6059        /**
     60         * Test schema filtering.
     61         *
     62         * @covers WP_Widget_Media_Image::get_instance_schema
     63         *
     64         * @ticket 45029
     65         */
     66        function test_get_instance_schema_filtering() {
     67                $widget = new WP_Widget_Media_Image();
     68                $schema = $widget->get_instance_schema();
     69
     70                add_filter( 'widget_media_image_instance_schema', array( $this, 'filter_instance_schema' ), 10, 2 );
     71                $schema = $widget->get_instance_schema();
     72
     73                $this->assertSame( 'large', $schema['size']['default'] );
     74        }
     75
     76        /**
     77         * Filters instance schema.
     78         *
     79         * @since 5.2.0
     80         *
     81         * @param array                 $schema Schema.
     82         * @param WP_Widget_Media_Image $widget Widget.
     83         *
     84         * @return array
     85         */
     86        public function filter_instance_schema( $schema, $widget ) {
     87                // Override the default size value ('medium').
     88                $schema['size']['default'] = 'large';
     89                return $schema;
     90        }
     91
     92        /**
    6193         * Test constructor.
    6294         *
Note: See TracChangeset for help on using the changeset viewer.