WordPress.org

Make WordPress Core

Ticket #18375: 18375.4.diff

File 18375.4.diff, 22.8 KB (added by dipesh.kakadiya, 19 months ago)

Used sprintf() and _x() ensures translators can provide better contextual translations.

  • src/wp-admin/edit-form-advanced.php

     
    259259        add_meta_box( $tax_meta_box_id, $label, $taxonomy->meta_box_cb, null, 'side', 'core', array( 'taxonomy' => $tax_name ) );
    260260}
    261261
    262 if ( post_type_supports($post_type, 'page-attributes') )
    263         add_meta_box('pageparentdiv', 'page' == $post_type ? __('Page Attributes') : __('Attributes'), 'page_attributes_meta_box', null, 'side', 'core');
     262if ( post_type_supports( $post_type, 'page-attributes' ) ) {
    264263
     264        $label = esc_html( sprintf( _x( '%s Attributes', 'Singular page attributes meta box title' ), $post_type_object->labels->singular_name ) );
     265
     266        add_meta_box( 'pageparentdiv', $label, 'page_attributes_meta_box', null, 'side', 'core' );
     267}
     268
    265269if ( $thumbnail_support && current_user_can( 'upload_files' ) )
    266270        add_meta_box('postimagediv', esc_html( $post_type_object->labels->featured_image ), 'post_thumbnail_meta_box', null, 'side', 'low');
    267271
  • src/wp-admin/includes/class-wp-posts-list-table.php

     
    15291529                        </label>
    15301530
    15311531        <?php   endif; // !$bulk
    1532 
    1533                         if ( 'page' === $screen->post_type ) :
    15341532        ?>
    15351533
    15361534                        <label>
     
    15441542                                        $default_title = apply_filters( 'default_page_template_title',  __( 'Default Template' ), 'quick-edit' );
    15451543                                ?>
    15461544                                        <option value="default"><?php echo esc_html( $default_title ); ?></option>
    1547                                         <?php page_template_dropdown() ?>
     1545                                        <?php page_template_dropdown( $post->page_template, $screen->post_type ) ?>
    15481546                                </select>
    15491547                        </label>
    15501548
    15511549        <?php
    1552                         endif; // page post_type
    15531550                endif; // page-attributes
    15541551        ?>
    15551552
  • src/wp-admin/includes/meta-boxes.php

     
    788788 * @param object $post
    789789 */
    790790function page_attributes_meta_box($post) {
    791         $post_type_object = get_post_type_object($post->post_type);
    792         if ( $post_type_object->hierarchical ) {
     791        if ( is_post_type_hierarchical( $post->post_type ) ) {
    793792                $dropdown_args = array(
    794793                        'post_type'        => $post->post_type,
    795794                        'exclude_tree'     => $post->ID,
     
    820819<?php
    821820                } // end empty pages check
    822821        } // end hierarchical check.
    823         if ( 'page' == $post->post_type && 0 != count( get_page_templates( $post ) ) && get_option( 'page_for_posts' ) != $post->ID ) {
    824                 $template = !empty($post->page_template) ? $post->page_template : false;
     822
     823        if ( 0 < count( get_page_templates( $post ) ) && get_option( 'page_for_posts' ) != $post->ID ) {
     824                $template = ! empty( $post->page_template ) ? $post->page_template : false;
    825825                ?>
    826826<p><strong><?php _e('Template') ?></strong><?php
    827827        /**
     
    835835         */
    836836        do_action( 'page_attributes_meta_box_template', $template, $post );
    837837?></p>
    838 <label class="screen-reader-text" for="page_template"><?php _e('Page Template') ?></label><select name="page_template" id="page_template">
     838<label class="screen-reader-text" for="page_template"><?php _e('Template') ?></label><select name="page_template" id="page_template">
    839839<?php
    840840/**
    841841 * Filters the title of the default page template displayed in the drop-down.
     
    849849$default_title = apply_filters( 'default_page_template_title',  __( 'Default Template' ), 'meta-box' );
    850850?>
    851851<option value="default"><?php echo esc_html( $default_title ); ?></option>
    852 <?php page_template_dropdown($template); ?>
     852<?php page_template_dropdown( $template, $post->post_type ); ?>
    853853</select>
    854854<?php
    855855        } ?>
  • src/wp-admin/includes/template.php

     
    293293        <div class="ss">' . mysql2date( 's', $post->post_date, false ) . '</div>
    294294        <div class="post_password">' . esc_html( $post->post_password ) . '</div>';
    295295
    296         if ( $post_type_object->hierarchical )
     296        if ( $post_type_object->hierarchical ) {
    297297                echo '<div class="post_parent">' . $post->post_parent . '</div>';
     298        }
    298299
    299         if ( $post->post_type == 'page' )
    300                 echo '<div class="page_template">' . esc_html( get_post_meta( $post->ID, '_wp_page_template', true ) ) . '</div>';
     300        echo '<div class="page_template">' . esc_html( $post->page_template ) . '</div>';
    301301
    302         if ( post_type_supports( $post->post_type, 'page-attributes' ) )
     302        if ( post_type_supports( $post->post_type, 'page-attributes' ) ) {
    303303                echo '<div class="menu_order">' . $post->menu_order . '</div>';
     304        }
    304305
    305306        $taxonomy_names = get_object_taxonomies( $post->post_type );
    306307        foreach ( $taxonomy_names as $taxonomy_name) {
     
    761762 * Print out option HTML elements for the page templates drop-down.
    762763 *
    763764 * @since 1.5.0
     765 * @since 4.7.0 Added the `$post_type` parameter.
    764766 *
    765  * @param string $default Optional. The template file name. Default empty.
     767 * @param string $default   Optional. The template file name. Default empty.
     768 * @param string $post_type Optional. Post type to get templates for. Default 'post'.
    766769 */
    767 function page_template_dropdown( $default = '' ) {
    768         $templates = get_page_templates( get_post() );
     770function page_template_dropdown( $default = '', $post_type = 'page' ) {
     771        $templates = get_page_templates( null, $post_type );
    769772        ksort( $templates );
    770773        foreach ( array_keys( $templates ) as $template ) {
    771774                $selected = selected( $default, $templates[ $template ], false );
  • src/wp-admin/includes/theme.php

     
    102102 * Get the Page Templates available in this theme
    103103 *
    104104 * @since 1.5.0
     105 * @since 4.7.0 Added the `$post_type` parameter.
    105106 *
    106  * @param WP_Post|null $post Optional. The post being edited, provided for context.
     107 * @param WP_Post|null $post      Optional. The post being edited, provided for context.
     108 * @param string       $post_type Optional. Post type to get the templates for. Default 'page'.
    107109 * @return array Key is the template name, value is the filename of the template
    108110 */
    109 function get_page_templates( $post = null ) {
    110         return array_flip( wp_get_theme()->get_page_templates( $post ) );
     111function get_page_templates( $post = null, $post_type = 'page' ) {
     112        return array_flip( wp_get_theme()->get_page_templates( $post, $post_type ) );
    111113}
    112114
    113115/**
  • src/wp-includes/class-wp-post.php

     
    254254                        return true;
    255255
    256256                if ( 'page_template' == $key )
    257                         return ( 'page' == $this->post_type );
     257                        return true;
    258258
    259259                if ( 'post_category' == $key )
    260260                   return true;
  • src/wp-includes/class-wp-theme.php

     
    10061006        }
    10071007
    10081008        /**
    1009          * Returns the theme's page templates.
     1009         * Returns the theme's post templates.
    10101010         *
    1011          * @since 3.4.0
     1011         * @since 4.7.0
    10121012         * @access public
    10131013         *
    1014          * @param WP_Post|null $post Optional. The post being edited, provided for context.
    1015          * @return array Array of page templates, keyed by filename, with the value of the translated header name.
     1014         * @return array Array of page templates, keyed by filename and post type,
     1015         *               with the value of the translated header name.
    10161016         */
    1017         public function get_page_templates( $post = null ) {
     1017        public function get_post_templates() {
    10181018                // If you screw up your current theme and we invalidate your parent, most things still work. Let it slide.
    1019                 if ( $this->errors() && $this->errors()->get_error_codes() !== array( 'theme_parent_invalid' ) )
     1019                if ( $this->errors() && $this->errors()->get_error_codes() !== array( 'theme_parent_invalid' ) ) {
    10201020                        return array();
     1021                }
    10211022
    1022                 $page_templates = $this->cache_get( 'page_templates' );
     1023                $post_templates = $this->cache_get( 'post_templates' );
    10231024
    1024                 if ( ! is_array( $page_templates ) ) {
    1025                         $page_templates = array();
     1025                if ( ! is_array( $post_templates ) ) {
     1026                        $post_templates = array();
    10261027
    10271028                        $files = (array) $this->get_files( 'php', 1 );
    10281029
    10291030                        foreach ( $files as $file => $full_path ) {
    1030                                 if ( ! preg_match( '|Template Name:(.*)$|mi', file_get_contents( $full_path ), $header ) )
     1031                                $template_data = get_file_data( $full_path, array( 'Template Name' => 'Template Name', 'Template Type' => 'Template Type' ) );
     1032                                if ( empty( $template_data[ 'Template Name' ] ) ) {
    10311033                                        continue;
    1032                                 $page_templates[ $file ] = _cleanup_header_comment( $header[1] );
     1034                                }
     1035                               
     1036                                $types = array( 'page' );
     1037                                if ( !empty( $template_data[ 'Template Type' ] ) ) {
     1038                                        $types = explode( ',', _cleanup_header_comment( $template_data[ 'Template Type' ] ) );
     1039                                }
     1040
     1041                                foreach ( $types as $type ) {
     1042                                        $type = trim( $type );
     1043                                        if ( ! isset( $post_templates[ $type ] ) ) {
     1044                                                $post_templates[ $type ] = array();
     1045                                        }
     1046
     1047                                        $post_templates[ $type ][ $file ] = _cleanup_header_comment( $template_data[ 'Template Name' ] );
     1048                                }
    10331049                        }
    10341050
    1035                         $this->cache_add( 'page_templates', $page_templates );
     1051                        $this->cache_add( 'page_templates', $post_templates );
    10361052                }
    10371053
    10381054                if ( $this->load_textdomain() ) {
    1039                         foreach ( $page_templates as &$page_template ) {
    1040                                 $page_template = $this->translate_header( 'Template Name', $page_template );
     1055                        foreach ( $post_templates as &$post_type ) {
     1056                                foreach ( $post_type as &$post_template ) {
     1057                                        $post_template = $this->translate_header( 'Template Name', $post_template );
     1058                                }
    10411059                        }
    10421060                }
    10431061
    1044                 if ( $this->parent() )
    1045                         $page_templates += $this->parent()->get_page_templates( $post );
     1062                return $post_templates;
     1063        }
    10461064
     1065        /**
     1066         * Returns the theme's post templates for a given post type.
     1067         *
     1068         * @since 3.4.0
     1069         * @since 4.7.0 Added the `$post_type` parameter.
     1070         * @access public
     1071         *
     1072         * @param WP_Post|null $post      Optional. The post being edited, provided for context.
     1073         * @param string       $post_type Optional. Post type to get the templates for. Default 'page'.
     1074         *                                If a post is provided, its post type is used.
     1075         * @return array Array of page templates, keyed by filename, with the value of the translated header name.
     1076         */
     1077        public function get_page_templates( $post = null, $post_type = 'page' ) {
     1078                if ( $post ) {
     1079                        $post_type = get_post_type( $post );
     1080                }
     1081
     1082                $post_templates = $this->get_post_templates();
     1083                $post_templates = isset( $post_templates[ $post_type ] ) ? $post_templates[ $post_type ] : array();
     1084
     1085                if ( $this->parent() ) {
     1086                        $post_templates += $this->parent()->get_page_templates( $post );
     1087                }
     1088
    10471089                /**
    10481090                 * Filters list of page templates for a theme.
    10491091                 *
    10501092                 * @since 3.9.0
    10511093                 * @since 4.4.0 Converted to allow complete control over the `$page_templates` array.
     1094                 * @since 4.7.0 Added the `$post_type` parameter.
    10521095                 *
    1053                  * @param array        $page_templates Array of page templates. Keys are filenames,
     1096                 * @param array        $post_templates Array of page templates. Keys are filenames,
    10541097                 *                                     values are translated names.
    10551098                 * @param WP_Theme     $this           The theme object.
    10561099                 * @param WP_Post|null $post           The post being edited, provided for context, or null.
     1100                 * @param string       $post_type      Post type to get the templates for.
    10571101                 */
    1058                 return (array) apply_filters( 'theme_page_templates', $page_templates, $this, $post );
     1102                return (array) apply_filters( 'theme_page_templates', $post_templates, $this, $post, $post_type );
    10591103        }
    10601104
    10611105        /**
  • src/wp-includes/post-template.php

     
    16211621 *
    16221622 * @since 2.5.0
    16231623 * @since 4.2.0 The `$template` parameter was changed to also accept an array of page templates.
     1624 * @since 4.7.0 Now supports any post type, not just pages.
    16241625 *
    16251626 * @param string|array $template The specific template name or array of templates to match.
    16261627 * @return bool True on success, false on failure.
    16271628 */
    16281629function is_page_template( $template = '' ) {
    1629         if ( ! is_page() )
    1630                 return false;
    1631 
    16321630        $page_template = get_page_template_slug( get_queried_object_id() );
    16331631
    16341632        if ( empty( $template ) )
     
    16491647}
    16501648
    16511649/**
    1652  * Get the specific template name for a page.
     1650 * Get the specific template name for a given post.
    16531651 *
    16541652 * @since 3.4.0
     1653 * @since 4.7.0 Now supports all post types.
    16551654 *
    1656  * @param int $post_id Optional. The page ID to check. Defaults to the current post, when used in the loop.
     1655 * @param int|WP_Post $post Optional. Post ID or WP_Post object. Default is global $post.
    16571656 * @return string|false Page template filename. Returns an empty string when the default page template
    1658  *      is in use. Returns false if the post is not a page.
     1657 *      is in use. Returns false if the post does not exist.
    16591658 */
    1660 function get_page_template_slug( $post_id = null ) {
    1661         $post = get_post( $post_id );
    1662         if ( ! $post || 'page' != $post->post_type )
     1659function get_page_template_slug( $post = null ) {
     1660        $post = get_post( $post );
     1661
     1662        if ( ! $post ) {
    16631663                return false;
     1664        }
     1665
    16641666        $template = get_post_meta( $post->ID, '_wp_page_template', true );
    1665         if ( ! $template || 'default' == $template )
     1667
     1668        if ( ! $template || 'default' == $template ) {
    16661669                return '';
     1670        }
     1671
    16671672        return $template;
    16681673}
    16691674
  • src/wp-includes/post.php

     
    13261326 *                       post types. Default is 'Parent Page:'.
    13271327 * - `all_items` - Label to signify all items in a submenu link. Default is 'All Posts' / 'All Pages'.
    13281328 * - `archives` - Label for archives in nav menus. Default is 'Post Archives' / 'Page Archives'.
     1329 * - `attributes` - Label for the attributes meta box. Default is 'Post Attributes' / 'Page Attributes'.
    13291330 * - `insert_into_item` - Label for the media frame button. Default is 'Insert into post' / 'Insert into page'.
    13301331 * - `uploaded_to_this_item` - Label for the media frame filter. Default is 'Uploaded to this post' /
    13311332 *                           'Uploaded to this page'.
     
    16111612 * @since 4.4.0
    16121613 * @since 4.5.0 Added the ability to pass a post type name in addition to object.
    16131614 * @since 4.6.0 Converted the `$post_type` parameter to accept a WP_Post_Type object.
     1615 * @since 4.7.0 Added the `attributes` label.
    16141616 *
    16151617 * @param string|WP_Post_Type $post_type Post type name or object.
    16161618 * @return bool Whether the post type should be considered viewable.
     
    33933395
    33943396        $post = get_post( $post_ID );
    33953397
    3396         if ( ! empty( $postarr['page_template'] ) && 'page' == $data['post_type'] ) {
     3398        if ( ! empty( $postarr['page_template'] ) ) {
    33973399                $post->page_template = $postarr['page_template'];
    33983400                $page_templates = wp_get_theme()->get_page_templates( $post );
    33993401                if ( 'default' != $postarr['page_template'] && ! isset( $page_templates[ $postarr['page_template'] ] ) ) {
  • src/wp-includes/template.php

     
    477477        $templates = array();
    478478
    479479        if ( ! empty( $object->post_type ) ) {
     480                $template = get_page_template_slug( $object );
     481                if ( $template && 0 === validate_file( $template ) ) {
     482                        $templates[] = $template;
     483                }
    480484
    481485                $name_decoded = urldecode( $object->post_name );
    482486                if ( $name_decoded !== $object->post_name ) {
  • tests/phpunit/tests/admin/includesTheme.php

     
    4848
    4949                switch_theme( $theme['Template'], $theme['Stylesheet'] );
    5050
    51                 $templates = get_page_templates();
    52                 $this->assertCount( 3, $templates );
    53                 $this->assertEquals( "template-top-level.php", $templates['Top Level'] );
    54                 $this->assertEquals( "subdir/template-sub-dir.php", $templates['Sub Dir'] );
    55                 $this->assertEquals( "template-header.php", $templates['This Template Header Is On One Line'] );
     51                $this->assertEqualSetsWithIndex( array(
     52                        'Top Level'                           => 'template-top-level.php',
     53                        'Sub Dir'                             => 'subdir/template-sub-dir.php',
     54                        'This Template Header Is On One Line' => 'template-header.php',
     55                ), get_page_templates() );
    5656
    5757                $theme = wp_get_theme( 'page-templates' );
    5858                $this->assertNotEmpty( $theme );
     
    5959
    6060                switch_theme( $theme['Template'], $theme['Stylesheet'] );
    6161
    62                 $templates = get_page_templates();
    63                 $this->assertCount( 3, $templates );
    64                 $this->assertEquals( "template-top-level.php", $templates['Top Level'] );
    65                 $this->assertEquals( "subdir/template-sub-dir.php", $templates['Sub Dir'] );
    66                 $this->assertEquals( "template-header.php", $templates['This Template Header Is On One Line'] );
     62                $this->assertEqualSetsWithIndex( array(
     63                        'Top Level'                           => 'template-top-level.php',
     64                        'Sub Dir'                             => 'subdir/template-sub-dir.php',
     65                        'This Template Header Is On One Line' => 'template-header.php',
     66                ), get_page_templates() );
    6767        }
     68
     69        /**
     70         * @ticket 18375
     71         */
     72        function test_page_templates_different_post_types() {
     73                $theme = wp_get_theme( 'page-templates' );
     74                $this->assertNotEmpty( $theme );
     75
     76                switch_theme( $theme['Template'], $theme['Stylesheet'] );
     77
     78                $this->assertEqualSetsWithIndex( array(
     79                        'Top Level' => 'template-top-level-post-types.php',
     80                        'Sub Dir'   => 'subdir/template-sub-dir-post-types.php',
     81                ), get_page_templates( null, 'foo' ) );
     82                $this->assertEqualSetsWithIndex( array(
     83                        'Top Level' => 'template-top-level-post-types.php',
     84                        'Sub Dir'   => 'subdir/template-sub-dir-post-types.php',
     85                ), get_page_templates( null, 'post' ) );
     86                $this->assertEquals( array(), get_page_templates( null, 'bar' ) );
     87        }
    6888}
  • tests/phpunit/tests/post/objects.php

     
    134134                $this->assertEquals( array( 'Bar', 'Baz', 'Foo' ), $post['tags_input'] );
    135135        }
    136136
     137        /**
     138         * @ticket 18375
     139         */
    137140        function test_get_page_template_property() {
    138141                $post_id = self::factory()->post->create();
    139142                $post = get_post( $post_id );
     
    145148                update_post_meta( $post_id, '_wp_page_template', 'foo.php' );
    146149                $template = get_post_meta( $post->ID, '_wp_page_template', true );
    147150                $this->assertEquals( 'foo.php', $template );
    148                 // The post is not a page so the template is still empty
    149                 $this->assertEquals( '', $post->page_template );
    150 
    151                 // Now the post is a page and should retrieve the template
    152                 wp_update_post( array( 'ID' => $post->ID, 'post_type' => 'page' ) );
    153                 $post = get_post( $post_id );
    154151                $this->assertEquals( $template, $post->page_template );
    155152        }
    156153
  • tests/phpunit/tests/post/template.php

     
    271271
    272272        /**
    273273         * @ticket 31389
     274         * @ticket 18375
    274275         */
    275276        public function test_get_page_template_slug_non_page() {
    276                 $post_id = self::factory()->post->create( array(
    277                         'post_type' => 'post',
    278                 ) );
     277                $post_id = self::factory()->post->create();
    279278
    280                 $this->assertFalse( get_page_template_slug( $post_id ) );
     279                $this->assertEquals( '', get_page_template_slug( $post_id ) );
    281280
     281                update_post_meta( $post_id, '_wp_page_template', 'default' );
     282
     283                $this->assertEquals( '', get_page_template_slug( $post_id ) );
     284
     285                update_post_meta( $post_id, '_wp_page_template', 'example.php' );
     286                $this->assertEquals( 'example.php', get_page_template_slug( $post_id ) );
     287        }
     288
     289        /**
     290         * @ticket 18375
     291         */
     292        public function test_get_page_template_slug_non_page_from_loop() {
     293                $post_id = self::factory()->post->create();
     294
     295                update_post_meta( $post_id, '_wp_page_template', 'example.php' );
     296
    282297                $this->go_to( get_permalink( $post_id ) );
    283                 $this->assertFalse( get_page_template_slug() );
     298
     299                $this->assertEquals( 'example.php', get_page_template_slug() );
    284300        }
    285301
    286302        /**
  • tests/phpunit/tests/query/conditionals.php

     
    10441044        }
    10451045
    10461046        /**
     1047         * @ticket 18375
     1048         */
     1049        function test_is_page_template_other_post_type() {
     1050                $post_id = self::factory()->post->create( array( 'post_type' => 'post' ) );
     1051                update_post_meta( $post_id, '_wp_page_template', 'example.php' );
     1052                $this->go_to( get_post_permalink( $post_id ) );
     1053                $this->assertFalse( is_page_template( array( 'test.php' ) ) );
     1054                $this->assertTrue( is_page_template( array( 'test.php', 'example.php' ) ) );
     1055        }
     1056
     1057        /**
    10471058         * @ticket 35902
    10481059         */
    10491060        public function test_is_attachment_should_not_match_numeric_id_to_post_title_beginning_with_id() {
  • tests/phpunit/tests/template.php

     
    3636                        'post_date' => '1984-02-25 12:34:56',
    3737                ) );
    3838                set_post_format( self::$post, 'quote' );
     39                add_post_meta( self::$post->ID, '_wp_page_template', 'templates/post.php' );
    3940        }
    4041
    4142        public function setUp() {
     
    203204                ) );
    204205        }
    205206
     207        /**
     208         * @ticket 18375
     209         */
    206210        public function test_single_template_hierarchy_for_post() {
    207211                $this->assertTemplateHierarchy( get_permalink( self::$post ), array(
     212                        'templates/post.php',
    208213                        'single-post-post-name-😀.php',
    209214                        'single-post-post-name-%f0%9f%98%80.php',
    210215                        'single-post.php',
     
    228233                ) );
    229234        }
    230235
     236        /**
     237         * @ticket 18375
     238         */
     239        public function test_single_template_hierarchy_for_custom_post_type_with_page_template() {
     240                $cpt = self::factory()->post->create_and_get( array(
     241                        'post_type' => 'cpt',
     242                        'post_name' => 'cpt-name-😀',
     243                ) );
     244                add_post_meta( $cpt->ID, '_wp_page_template', 'templates/cpt.php' );
     245
     246                $this->assertTemplateHierarchy( get_permalink( $cpt ), array(
     247                        'templates/cpt.php',
     248                        'single-cpt-cpt-name-😀.php',
     249                        'single-cpt-cpt-name-%f0%9f%98%80.php',
     250                        'single-cpt.php',
     251                        'single.php',
     252                        'singular.php',
     253                ) );
     254        }
     255
    231256        public function test_attachment_template_hierarchy() {
    232257                $attachment = self::factory()->attachment->create_and_get( array(
    233258                        'post_name'      => 'attachment-name-😀',