Make WordPress Core


Ignore:
Timestamp:
06/28/2020 11:08:57 PM (5 years ago)
Author:
SergeyBiryukov
Message:

Themes: Add a return value to theme functions calling locate_template():

  • get_header()
  • get_footer()
  • get_sidebar()
  • get_template_part()

These functions now return false if the template file could not be found, to allow for easier debugging.

Props tferry, sphakka, johnbillion, pento, davidbinda, desrosj, birgire, garrett-eclipse, williampatton, davidbaumwald, SergeyBiryukov.
Fixes #40969.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-includes/general-template.php

    r48199 r48209  
    1717 *
    1818 * @since 1.5.0
     19 * @since 5.5.0 A return value was added.
    1920 *
    2021 * @param string $name The name of the specialised header.
     22 * @return void|false Void on success, false if the template does not exist.
    2123 */
    2224function get_header( $name = null ) {
     
    3941    $templates[] = 'header.php';
    4042
    41     locate_template( $templates, true );
     43    if ( ! locate_template( $templates, true ) ) {
     44        return false;
     45    }
    4246}
    4347
     
    5256 *
    5357 * @since 1.5.0
     58 * @since 5.5.0 A return value was added.
    5459 *
    5560 * @param string $name The name of the specialised footer.
     61 * @return void|false Void on success, false if the template does not exist.
    5662 */
    5763function get_footer( $name = null ) {
     
    7480    $templates[] = 'footer.php';
    7581
    76     locate_template( $templates, true );
     82    if ( ! locate_template( $templates, true ) ) {
     83        return false;
     84    }
    7785}
    7886
     
    8795 *
    8896 * @since 1.5.0
     97 * @since 5.5.0 A return value was added.
    8998 *
    9099 * @param string $name The name of the specialised sidebar.
     100 * @return void|false Void on success, false if the template does not exist.
    91101 */
    92102function get_sidebar( $name = null ) {
     
    109119    $templates[] = 'sidebar.php';
    110120
    111     locate_template( $templates, true );
     121    if ( ! locate_template( $templates, true ) ) {
     122        return false;
     123    }
    112124}
    113125
     
    129141 *
    130142 * @since 3.0.0
     143 * @since 5.5.0 A return value was added.
    131144 *
    132145 * @param string $slug The slug name for the generic template.
    133146 * @param string $name The name of the specialised template.
     147 * @return void|false Void on success, false if the template does not exist.
    134148 */
    135149function get_template_part( $slug, $name = null ) {
     
    166180    do_action( 'get_template_part', $slug, $name, $templates );
    167181
    168     locate_template( $templates, true, false );
     182    if ( ! locate_template( $templates, true, false ) ) {
     183        return false;
     184    }
    169185}
    170186
Note: See TracChangeset for help on using the changeset viewer.