WordPress.org
Get WordPress

Make WordPress Core

Ticket #37782: 37782.13.diff

File 37782.13.diff, 12.9 KB (added by garrett-eclipse, 6 years ago)

Re-introduced unit tests that got dropped in v12 due to me not running svn add.

  • src/wp-admin/css/nav-menus.css

     
    413413        margin-top: -4px;
    414414}
    415415
     416.menu-item-title .post-state {
     417        font-weight: 600;
     418}
     419
    416420/* Nav Menu */
    417421#menu-container .inside {
    418422        padding-bottom: 10px;

  • src/wp-admin/includes/class-walker-nav-menu-checklist.php

     
    9494                } elseif ( isset( $item->post_type ) ) {
    9595                        /** This filter is documented in wp-includes/post-template.php */
    9696                        $title = apply_filters( 'the_title', $item->post_title, $item->ID );
    97                         if ( ! empty( $item->front_or_home ) && _x( 'Home', 'nav menu home label' ) !== $title ) {
    98                                 /* translators: %s: front page title */
    99                                 $title = sprintf( _x( 'Home: %s', 'nav menu front page title' ), $title );
    100                         }
    10197                }
    10298
    10399                $output .= isset( $title ) ? esc_html( $title ) : esc_html( $item->title );
     100
     101                if ( empty( $item->label ) && isset( $item->post_type ) && 'page' === $item->post_type ) {
     102                        // Append post states.
     103                        $output .= _post_states( $item, false );
     104                }
     105
    104106                $output .= '</label>';
    105107
    106108                // Menu item hidden fields

  • src/wp-admin/includes/nav-menu.php

     
    354354                $args = array_merge( $args, (array) $box['args']->_default_query );
    355355        }
    356356
     357        /*
     358         * If we're dealing with pages, let's prioritize the Front Page,
     359         * Posts Page and Privacy Policy Page at the top of the list.
     360         */
     361        $important_pages = array();
     362        if ( 'page' == $post_type_name ) {
     363                $suppress_page_ids = array();
     364
     365                $front_page = 'page' == get_option( 'show_on_front' ) ? (int) get_option( 'page_on_front' ) : 0;
     366                $posts_page = 'page' == get_option( 'show_on_front' ) ? (int) get_option( 'page_for_posts' ) : 0;
     367
     368                $privacy_policy_page_id        = (int) get_option( 'wp_page_for_privacy_policy' );
     369                $privacy_policy_page_available = false;
     370
     371                if ( ! empty( $privacy_policy_page_id ) ) {
     372                        $privacy_policy_page           = get_post( $privacy_policy_page_id );
     373                        $privacy_policy_page_available = ( $privacy_policy_page instanceof WP_Post && 'publish' === $privacy_policy_page->post_status );
     374                }
     375
     376                // Insert Front Page or custom Home link.
     377                $front_page_obj = null;
     378                if ( ! empty( $front_page ) ) {
     379                        $front_page_obj                = get_post( $front_page );
     380                        $front_page_obj->front_or_home = true;
     381
     382                        $important_pages[]   = $front_page_obj;
     383                        $suppress_page_ids[] = $front_page_obj->ID;
     384                } else {
     385                        $_nav_menu_placeholder = ( 0 > $_nav_menu_placeholder ) ? intval( $_nav_menu_placeholder ) - 1 : -1;
     386                        $front_page_obj        = (object) array(
     387                                'front_or_home' => true,
     388                                'ID'            => 0,
     389                                'object_id'     => $_nav_menu_placeholder,
     390                                'post_content'  => '',
     391                                'post_excerpt'  => '',
     392                                'post_parent'   => '',
     393                                'post_title'    => _x( 'Home', 'nav menu home label' ),
     394                                'post_type'     => 'nav_menu_item',
     395                                'type'          => 'custom',
     396                                'url'           => home_url( '/' ),
     397                        );
     398
     399                        $important_pages[] = $front_page_obj;
     400                }
     401
     402                // Insert Posts Page.
     403                if ( ! empty( $posts_page ) ) {
     404                        $posts_page_obj             = get_post( $posts_page );
     405                        $posts_page_obj->posts_page = true;
     406
     407                        $important_pages[]   = $posts_page_obj;
     408                        $suppress_page_ids[] = $posts_page_obj->ID;
     409                }
     410
     411                // Insert Privacy Policy Page.
     412                if ( $privacy_policy_page_available ) {
     413                        $privacy_policy_page_obj                      = get_post( $privacy_policy_page_id );
     414                        $privacy_policy_page_obj->privacy_policy_page = true;
     415
     416                        $important_pages[]   = $privacy_policy_page_obj;
     417                        $suppress_page_ids[] = $privacy_policy_page_obj->ID;
     418                }
     419
     420                // Add suppression array to arguments for WP_Query.
     421                if ( ! empty( $suppress_page_ids ) ) {
     422                        $args['post__not_in'] = $suppress_page_ids;
     423                }
     424        }
     425
    357426        // @todo transient caching of these results with proper invalidation on updating of a post of this type
    358427        $get_posts = new WP_Query;
    359428        $posts     = $get_posts->query( $args );
     429
     430        // Only suppress and insert when more than just suppression pages available.
    360431        if ( ! $get_posts->post_count ) {
    361                 echo '<p>' . __( 'No items.' ) . '</p>';
    362                 return;
     432                if ( ! empty( $suppress_page_ids ) ) {
     433                        unset( $args['post__not_in'] );
     434                        $get_posts = new WP_Query;
     435                        $posts     = $get_posts->query( $args );
     436                } else {
     437                        echo '<p>' . __( 'No items.' ) . '</p>';
     438                        return;
     439                }
     440        } else {
     441                if ( ! empty( $important_pages ) ) {
     442                        $posts = array_merge( $important_pages, $posts );
     443                }
    363444        }
    364445
    365446        $num_pages = $get_posts->max_num_pages;
    … …  
    520601                                <?php
    521602                                $args['walker'] = $walker;
    522603
    523                                 /*
    524                                  * If we're dealing with pages, let's put a checkbox for the front
    525                                  * page at the top of the list.
    526                                  */
    527                                 if ( 'page' == $post_type_name ) {
    528                                         $front_page = 'page' == get_option( 'show_on_front' ) ? (int) get_option( 'page_on_front' ) : 0;
    529                                         if ( ! empty( $front_page ) ) {
    530                                                 $front_page_obj                = get_post( $front_page );
    531                                                 $front_page_obj->front_or_home = true;
    532                                                 array_unshift( $posts, $front_page_obj );
    533                                         } else {
    534                                                 $_nav_menu_placeholder = ( 0 > $_nav_menu_placeholder ) ? intval( $_nav_menu_placeholder ) - 1 : -1;
    535                                                 array_unshift(
    536                                                         $posts,
    537                                                         (object) array(
    538                                                                 'front_or_home' => true,
    539                                                                 'ID'            => 0,
    540                                                                 'object_id'     => $_nav_menu_placeholder,
    541                                                                 'post_content'  => '',
    542                                                                 'post_excerpt'  => '',
    543                                                                 'post_parent'   => '',
    544                                                                 'post_title'    => _x( 'Home', 'nav menu home label' ),
    545                                                                 'post_type'     => 'nav_menu_item',
    546                                                                 'type'          => 'custom',
    547                                                                 'url'           => home_url( '/' ),
    548                                                         )
    549                                                 );
    550                                         }
    551                                 }
    552 
    553604                                $post_type = get_post_type_object( $post_type_name );
    554605
    555606                                if ( $post_type->has_archive ) {

  • src/wp-admin/includes/template.php

     
    20092009}
    20102010
    20112011/**
    2012  * @param WP_Post $post
     2012 * Function to echo or return the Post States as HTML.
     2013 *
     2014 * @param WP_Post $post The post to retrieve states for.
     2015 * @param boolean $echo Optional. Whether to echo or return the Post States as an HTML string. Default true for echo.
     2016 *
     2017 * @return string|void Post States string when echo is false.
    20132018 */
    2014 function _post_states( $post ) {
     2019function _post_states( $post, $echo = true ) {
     2020        $post_states        = get_post_states( $post );
     2021        $post_states_string = '';
     2022
     2023        if ( ! empty( $post_states ) ) {
     2024                $state_count = count( $post_states );
     2025                $i           = 0;
     2026
     2027                $post_states_string .= ' &mdash; ';
     2028                foreach ( $post_states as $state ) {
     2029                        ++$i;
     2030                        ( $i == $state_count ) ? $sep = '' : $sep = ', ';
     2031                        $post_states_string          .= "<span class='post-state'>$state$sep</span>";
     2032                }
     2033        }
     2034
     2035        if ( $echo ) {
     2036                echo $post_states_string;
     2037        }
     2038
     2039        return $post_states_string;
     2040}
     2041
     2042/**
     2043 * Function to retrieve an array of Post States from a Post.
     2044 *
     2045 * @since 5.2.0
     2046 *
     2047 * @param WP_Post $post The post to retrieve states for.
     2048 *
     2049 * @return array $post_states The array of translated post states.
     2050 *
     2051 */
     2052function get_post_states( $post ) {
    20152053        $post_states = array();
    20162054        if ( isset( $_REQUEST['post_status'] ) ) {
    20172055                $post_status = $_REQUEST['post_status'];
    … …  
    20222060        if ( ! empty( $post->post_password ) ) {
    20232061                $post_states['protected'] = __( 'Password protected' );
    20242062        }
     2063
    20252064        if ( 'private' == $post->post_status && 'private' != $post_status ) {
    20262065                $post_states['private'] = __( 'Private' );
    20272066        }
     2067
    20282068        if ( 'draft' === $post->post_status ) {
    20292069                if ( get_post_meta( $post->ID, '_customize_changeset_uuid', true ) ) {
    20302070                        $post_states[] = __( 'Customization Draft' );
    … …  
    20342074        } elseif ( 'trash' === $post->post_status && get_post_meta( $post->ID, '_customize_changeset_uuid', true ) ) {
    20352075                $post_states[] = __( 'Customization Draft' );
    20362076        }
     2077
    20372078        if ( 'pending' == $post->post_status && 'pending' != $post_status ) {
    20382079                $post_states['pending'] = _x( 'Pending', 'post status' );
    20392080        }
     2081
    20402082        if ( is_sticky( $post->ID ) ) {
    20412083                $post_states['sticky'] = __( 'Sticky' );
    20422084        }
    … …  
    20682110         * @param string[] $post_states An array of post display states.
    20692111         * @param WP_Post  $post        The current post object.
    20702112         */
    2071         $post_states = apply_filters( 'display_post_states', $post_states, $post );
    2072 
    2073         if ( ! empty( $post_states ) ) {
    2074                 $state_count = count( $post_states );
    2075                 $i           = 0;
    2076                 echo ' &mdash; ';
    2077                 foreach ( $post_states as $state ) {
    2078                         ++$i;
    2079                         ( $i == $state_count ) ? $sep = '' : $sep = ', ';
    2080                         echo "<span class='post-state'>$state$sep</span>";
    2081                 }
    2082         }
    2083 
     2113        return apply_filters( 'display_post_states', $post_states, $post );
    20842114}
    20852115
    20862116/**

  • tests/phpunit/tests/menu/wpNavMenuItemPostTypeMetaBox.php

     
     1<?php
     2/**
     3 * Test cases for the `wp_nav_menu_item_post_type_meta_box()` function.
     4 *
     5 * @package WordPress
     6 * @subpackage UnitTests
     7 * @since 5.2.0
     8 */
     9
     10require_once ABSPATH . 'wp-admin/includes/nav-menu.php';
     11
     12/**
     13 * Tests_Menu_WpNavMenuItemPostTypeMetaBox class.
     14 *
     15 * @group navmenus
     16 * @covers ::wp_nav_menu_item_post_type_meta_box()
     17 *
     18 * @since 5.2.0
     19 */
     20class Tests_Menu_WpNavMenuItemPostTypeMetaBox extends WP_UnitTestCase {
     21
     22        /**
     23         * The function should contain no items without pages.
     24         *
     25         * @ticket 37782
     26         */
     27        public function test_should_contain_no_items_without_pages() {
     28
     29                $item = apply_filters( 'nav_menu_meta_box_object', get_post_type_object( 'page' ) );
     30
     31                $box = array(
     32                        'id'       => 'add-' . $item->name,
     33                        'title'    => $item->labels->name,
     34                        'callback' => 'wp_nav_menu_item_post_type_meta_box',
     35                        'args'     => $item,
     36                );
     37
     38                $output = get_echo( 'wp_nav_menu_item_post_type_meta_box', array( null, $box ) );
     39               
     40                $this->assertContains( 'No items', $output );
     41                $this->assertSame( 0, substr_count( $output, '<li>' ) );
     42        }
     43
     44        /**
     45         * The function should include a page only once, when viewing all pages.
     46         *
     47         * @ticket 37782
     48         */
     49        public function test_should_contain_page_only_once_when_viewing_all() {
     50                $page = $this->factory()->post->create_and_get(
     51                        array(
     52                                'post_title' => 'My Test Page',
     53                                'post_type'  => 'page',
     54                        )
     55                );
     56
     57                $item = apply_filters( 'nav_menu_meta_box_object', get_post_type_object( 'page' ) );
     58
     59                $box = array(
     60                        'id'       => 'add-' . $item->name,
     61                        'title'    => $item->labels->name,
     62                        'callback' => 'wp_nav_menu_item_post_type_meta_box',
     63                        'args'     => $item,
     64                );
     65
     66                $output = get_echo( 'wp_nav_menu_item_post_type_meta_box', array( null, $box ) );
     67
     68                // Clean up.
     69                wp_delete_post( $page->ID );
     70
     71                // Extract the "View All" part.
     72                preg_match( '~<div id="page-all"[^>]*>.*?</div>~i', str_replace( "\n", '', $output ), $matches );
     73
     74                $this->assertTrue( isset( $matches[0] ) );
     75                $this->assertSame( 2, substr_count( $matches[0], '<li>' ) );
     76                $this->assertContains( sprintf( '> %s</label>', $page->post_title ), $matches[0] );
     77                $this->assertContains( '> Home</label>', $matches[0] );
     78        }
     79
     80        /**
     81         * The function should include the front page only once, when viewing all pages.
     82         *
     83         * @ticket 37782
     84         */
     85        public function test_should_contain_front_page_only_once_when_viewing_all() {
     86                $page = $this->factory()->post->create_and_get(
     87                        array(
     88                                'post_title' => 'My Test Page',
     89                                'post_type'  => 'page',
     90                        )
     91                );
     92
     93                $item = apply_filters( 'nav_menu_meta_box_object', get_post_type_object( 'page' ) );
     94
     95                $box = array(
     96                        'id'       => 'add-' . $item->name,
     97                        'title'    => $item->labels->name,
     98                        'callback' => 'wp_nav_menu_item_post_type_meta_box',
     99                        'args'     => $item,
     100                );
     101
     102                // Front page setup.
     103                update_option( 'show_on_front', 'page' );
     104                update_option( 'page_on_front', $page->ID );
     105
     106                $output = get_echo( 'wp_nav_menu_item_post_type_meta_box', array( null, $box ) );
     107
     108                // Clean up.
     109                delete_option( 'show_on_front' );
     110                delete_option( 'page_on_front' );
     111                wp_delete_post( $page->ID );
     112
     113                // Extract the "View All" part.
     114                preg_match( '~<div id="page-all"[^>]*>.*?</div>~i', str_replace( "\n", '', $output ), $matches );
     115
     116                $this->assertTrue( isset( $matches[0] ) );
     117                $this->assertSame( 1, substr_count( $matches[0], '<li>' ) );
     118                $this->assertContains( sprintf( "> %s &mdash; <span class='post-state'>Front Page</span></label>", $page->post_title ), $matches[0] );
     119                $this->assertNotContains( sprintf( "> %s</label>", $page->post_title ), $matches[0] );
     120        }
     121}