WordPress.org
Get WordPress

Make WordPress Core

Ticket #23049: 23049.2.diff

File 23049.2.diff, 3.7 KB (added by johnbillion, 7 years ago)

  • src/wp-includes/template.php

     
    8585 * and {@see '$type_template'} dynamic hooks, where `$type` is '404'.
    8686 *
    8787 * @since 1.5.0
     88 * @since 4.9.0 The hierachy was extended to include specific templates dependent on the query.
    8889 *
    8990 * @see get_query_template()
    9091 *
    … …  
    9192 * @return string Full path to 404 template file.
    9293 */
    9394function get_404_template() {
    94         return get_query_template('404');
     95        $checks = array(
     96                'taxonomy'      => 'taxonomy-%s',
     97                'attachment_id' => 'attachment',
     98                'page_id'       => 'page',
     99                'pagename'      => 'page',
     100                'post_type'     => 'single-%s',
     101                'p'             => 'single',
     102                'name'          => 'single',
     103                'cat'           => 'category',
     104                'category_name' => 'category',
     105                'tag'           => 'tag',
     106                'author'        => 'author',
     107                'm'             => 'date',
     108                'year'          => 'date',
     109                'monthnum'      => 'date',
     110                'day'           => 'date',
     111        );
     112
     113        $templates = array();
     114
     115        foreach ( $checks as $query_var => $template ) {
     116                $query_val = get_query_var( $query_var );
     117                if ( ! empty( $query_val ) ) {
     118                        $templates[] = '404-' . sprintf( $template, $query_val ) . '.php';
     119                        break;
     120                }
     121        }
     122
     123        $templates[] = '404.php';
     124
     125        return get_query_template( '404', $templates );
    95126}
    96127
    97128/**

  • tests/phpunit/tests/template.php

     
    4141
    4242        public function setUp() {
    4343                parent::setUp();
     44                $this->set_permalink_structure( '/%year%/%monthnum%/%day%/%postname%/' );
    4445                register_post_type( 'cpt', array(
    4546                        'public' => true,
    4647                ) );
    … …  
    5859                parent::tearDown();
    5960        }
    6061
     62        /**
     63         * @ticket 23049
     64         */
     65        public function test_404_template_hierarchy_for_term() {
     66                $url = home_url( 'taxo/not-found' );
    6167
    62         public function test_404_template_hierarchy() {
     68                $this->assertTemplateHierarchy( $url, array(
     69                        '404-taxonomy-taxo.php',
     70                        '404.php',
     71                ) );
     72        }
     73
     74        /**
     75         * @ticket 23049
     76         */
     77        public function test_404_template_hierarchy_for_attachment() {
    6378                $url = add_query_arg( array(
    64                         'p' => '-1',
     79                        'attachment_id' => '99999999',
    6580                ), home_url() );
    6681
    6782                $this->assertTemplateHierarchy( $url, array(
     83                        '404-attachment.php',
    6884                        '404.php',
    6985                ) );
    7086        }
    7187
     88        /**
     89         * @ticket 23049
     90         */
     91        public function test_404_template_hierarchy_for_page() {
     92                $url = home_url( 'not-found' );
     93
     94                $this->assertTemplateHierarchy( $url, array(
     95                        '404-page.php',
     96                        '404.php',
     97                ) );
     98        }
     99
     100        /**
     101         * @ticket 23049
     102         */
     103        public function test_404_template_hierarchy_for_single_post() {
     104                $url = home_url( '1984/02/25/not-found' );
     105
     106                $this->assertTemplateHierarchy( $url, array(
     107                        '404-single.php',
     108                        '404.php',
     109                ) );
     110        }
     111
     112        /**
     113         * @ticket 23049
     114         */
     115        public function test_404_template_hierarchy_for_single_post_type() {
     116                $url = home_url( 'cpt/not-found' );
     117
     118                $this->assertTemplateHierarchy( $url, array(
     119                        '404-single-cpt.php',
     120                        '404.php',
     121                ) );
     122        }
     123
     124        /**
     125         * @ticket 23049
     126         */
     127        public function test_404_template_hierarchy_for_category() {
     128                $url = home_url( 'category/not-found' );
     129
     130                $this->assertTemplateHierarchy( $url, array(
     131                        '404-category.php',
     132                        '404.php',
     133                ) );
     134        }
     135
     136        /**
     137         * @ticket 23049
     138         */
     139        public function test_404_template_hierarchy_for_tag() {
     140                $url = home_url( 'tag/not-found' );
     141
     142                $this->assertTemplateHierarchy( $url, array(
     143                        '404-tag.php',
     144                        '404.php',
     145                ) );
     146        }
     147
    72148        public function test_author_template_hierarchy() {
    73149                $author = self::factory()->user->create_and_get( array(
    74150                        'user_nicename' => 'foo',