WordPress.org

Make WordPress Core

Ticket #35215: 35215.diff

File 35215.diff, 3.2 KB (added by swissspidy, 5 years ago)
  • src/wp-admin/includes/class-wp-screen.php

    diff --git src/wp-admin/includes/class-wp-screen.php src/wp-admin/includes/class-wp-screen.php
    index 2537319..7f17c75 100644
    final class WP_Screen { 
    530530                        }
    531531                }
    532532
    533                 sort( $priorities );
     533                ksort( $priorities );
    534534
    535535                $sorted = array();
    536536                foreach ( $priorities as $list ) {
    final class WP_Screen { 
    998998                 * Filter whether to show the Screen Options submit button.
    999999                 *
    10001000                 * @since 4.4.0
    1001                  * 
     1001                 *
    10021002                 * @param bool      $show_button Whether to show Screen Options submit button.
    10031003                 *                               Default false.
    10041004                 * @param WP_Screen $this        Current WP_Screen instance.
  • tests/phpunit/tests/admin/includesScreen.php

    diff --git tests/phpunit/tests/admin/includesScreen.php tests/phpunit/tests/admin/includesScreen.php
    index 65603e3..484ab72 100644
    class Tests_Admin_includesScreen extends WP_UnitTestCase { 
    187187        function test_help_tabs_priority() {
    188188                $tab_1      = rand_str();
    189189                $tab_1_args = array(
    190                         'id'       => $tab_1,
    191190                        'title'    => 'Help!',
     191                        'id'       => $tab_1,
    192192                        'content'  => 'Some content',
    193193                        'callback' => false,
    194                         'priority' => 11,
     194                        'priority' => 10,
    195195                );
    196196
    197197                $tab_2      = rand_str();
    198198                $tab_2_args = array(
     199                        'title'    => 'Help!',
    199200                        'id'       => $tab_2,
     201                        'content'  => 'Some content',
     202                        'callback' => false,
     203                        'priority' => 2,
     204                );
     205                $tab_3      = rand_str();
     206                $tab_3_args = array(
    200207                        'title'    => 'Help!',
     208                        'id'       => $tab_3,
    201209                        'content'  => 'Some content',
    202210                        'callback' => false,
    203                         'priority' => 9,
     211                        'priority' => 40,
    204212                );
    205213
    206214                $screen = get_current_screen();
    class Tests_Admin_includesScreen extends WP_UnitTestCase { 
    213221                $screen->add_help_tab( $tab_2_args );
    214222                $this->assertEquals( $screen->get_help_tab( $tab_2 ), $tab_2_args );
    215223
     224                $screen->add_help_tab( $tab_3_args );
     225                $this->assertEquals( $screen->get_help_tab( $tab_3 ), $tab_3_args );
     226
    216227                $tabs = $screen->get_help_tabs();
    217                 $this->assertEquals( 2, count( $tabs ) );
     228                $this->assertEquals( 3, count( $tabs ) );
    218229                $this->assertArrayHasKey( $tab_1, $tabs );
    219230                $this->assertArrayHasKey( $tab_2, $tabs );
     231                $this->assertArrayHasKey( $tab_3, $tabs );
    220232
    221233                // Test priority order.
    222234
    223                 $this->assertEquals( $tabs, array(
     235                $this->assertSame( array(
    224236                        $tab_2 => $tab_2_args,
    225237                        $tab_1 => $tab_1_args,
    226                 ) );
     238                        $tab_3 => $tab_3_args,
     239                ), $tabs );
    227240
    228241                $screen->remove_help_tab( $tab_1 );
    229242                $this->assertNull( $screen->get_help_tab( $tab_1 ) );
    230                 $this->assertEquals( 1, count( $screen->get_help_tabs() ) );
     243                $this->assertSame( 2, count( $screen->get_help_tabs() ) );
    231244
    232245                $screen->remove_help_tab( $tab_2 );
    233246                $this->assertNull( $screen->get_help_tab( $tab_2 ) );
    234                 $this->assertEquals( 0, count( $screen->get_help_tabs() ) );
     247                $this->assertSame( 1, count( $screen->get_help_tabs() ) );
     248
     249                $screen->remove_help_tab( $tab_3 );
     250                $this->assertNull( $screen->get_help_tab( $tab_3 ) );
     251                $this->assertSame( 0, count( $screen->get_help_tabs() ) );
    235252
    236253                $screen->remove_help_tabs();
    237                 $this->assertEquals( $screen->get_help_tabs(), array() );
     254                $this->assertEquals( array(), $screen->get_help_tabs() );
    238255        }
    239256
    240257        /**