Skip to content

Commit

Permalink
REST API: Ensure string returned in WP_REST_Templates_Controller::get…
Browse files Browse the repository at this point in the history
…_wp_templates_author_text_field().

Adds a fail-safe to return an empty string should the `switch` ever fall through without returning.

Currently, `WP_REST_Templates_Controller::get_wp_templates_author_text_field()` is tightly coupled to `WP_REST_Templates_Controller::get_wp_templates_original_source_field()`. However, if the `$original_source` values change in either method, but not both, it is possible a `void` or `null` will be returned, rather than a `string`.

Follow-up to [57366].

Props antonvlasenko, hellofromTonya, debarghyabanerjee.
Fixes #61580.

git-svn-id: https://develop.svn.wordpress.org/trunk@58705 602fd350-edb4-49c9-b593-d223f7449a82
  • Loading branch information
hellofromtonya committed Jul 10, 2024
1 parent 9cff1de commit 8557f28
Showing 1 changed file with 3 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -871,6 +871,9 @@ private static function get_wp_templates_author_text_field( $template_object ) {
}
return $author->get( 'display_name' );
}

// Fail-safe to return a string should the original source ever fall through.
return '';
}


Expand Down

0 comments on commit 8557f28

Please sign in to comment.