Docs: Replace multiple single line comments with multi-line comments.

This changeset updates various comments as per WordPress PHP Inline Documentation Standards.
See https://developer.wordpress.org/coding-standards/inline-documentation-standards/php/#5-inline-comments.

Follow-up to [56174], [56175], [56176], [56177], [56178], [56179], [56180], [56191], [56192].

Props costdev, audrasjb.
See #58459.



Built from https://develop.svn.wordpress.org/trunk@56193


git-svn-id: http://core.svn.wordpress.org/trunk@55705 1a063a9b-81f0-0310-95a4-ce76da25c4cd
This commit is contained in:
audrasjb 2023-07-10 23:00:22 +00:00
parent 6263f22b10
commit 55b31a92be
12 changed files with 73 additions and 37 deletions

View File

@ -45,8 +45,10 @@ final class WP_Customize_Header_Image_Setting extends WP_Customize_Setting {
$custom_image_header = new Custom_Image_Header( $admin_head_callback, $admin_preview_callback ); $custom_image_header = new Custom_Image_Header( $admin_head_callback, $admin_preview_callback );
} }
// If the value doesn't exist (removed or random), /*
// use the header_image value. * If the value doesn't exist (removed or random),
* use the header_image value.
*/
if ( ! $value ) { if ( ! $value ) {
$value = $this->manager->get_setting( 'header_image' )->post_value(); $value = $this->manager->get_setting( 'header_image' )->post_value();
} }

View File

@ -88,8 +88,10 @@ class WP_Customize_Media_Control extends WP_Customize_Control {
if ( is_object( $this->setting ) ) { if ( is_object( $this->setting ) ) {
if ( $this->setting->default ) { if ( $this->setting->default ) {
// Fake an attachment model - needs all fields used by template. /*
// Note that the default value must be a URL, NOT an attachment ID. * Fake an attachment model - needs all fields used by template.
* Note that the default value must be a URL, NOT an attachment ID.
*/
$ext = substr( $this->setting->default, -3 ); $ext = substr( $this->setting->default, -3 );
$type = in_array( $ext, array( 'jpg', 'png', 'gif', 'bmp', 'webp' ), true ) ? 'image' : 'document'; $type = in_array( $ext, array( 'jpg', 'png', 'gif', 'bmp', 'webp' ), true ) ? 'image' : 'document';

View File

@ -184,8 +184,10 @@ class WP_Customize_Themes_Section extends WP_Customize_Section {
* @since 4.9.0 * @since 4.9.0
*/ */
protected function filter_drawer_content_template() { protected function filter_drawer_content_template() {
// @todo Use the .org API instead of the local core feature list. /*
// The .org API is currently outdated and will be reconciled when the .org themes directory is next redesigned. * @todo Use the .org API instead of the local core feature list.
* The .org API is currently outdated and will be reconciled when the .org themes directory is next redesigned.
*/
$feature_list = get_theme_feature_list( false ); $feature_list = get_theme_feature_list( false );
?> ?>
<# if ( 'wporg' === data.action ) { #> <# if ( 'wporg' === data.action ) { #>

View File

@ -473,8 +473,10 @@ class WP_REST_Request implements ArrayAccess {
$params = array(); $params = array();
foreach ( $order as $type ) { foreach ( $order as $type ) {
// array_merge() / the "+" operator will mess up /*
// numeric keys, so instead do a manual foreach. * array_merge() / the "+" operator will mess up
* numeric keys, so instead do a manual foreach.
*/
foreach ( (array) $this->params[ $type ] as $key => $value ) { foreach ( (array) $this->params[ $type ] as $key => $value ) {
$params[ $key ] = $value; $params[ $key ] = $value;
} }

View File

@ -701,8 +701,10 @@ class WP_REST_Server {
$embedded = array(); $embedded = array();
foreach ( $data['_links'] as $rel => $links ) { foreach ( $data['_links'] as $rel => $links ) {
// If a list of relations was specified, and the link relation /*
// is not in the list of allowed relations, don't process the link. * If a list of relations was specified, and the link relation
* is not in the list of allowed relations, don't process the link.
*/
if ( is_array( $embed ) && ! in_array( $rel, $embed, true ) ) { if ( is_array( $embed ) && ! in_array( $rel, $embed, true ) ) {
continue; continue;
} }

View File

@ -202,8 +202,10 @@ class WP_REST_Attachments_Controller extends WP_REST_Posts_Controller {
wp_after_insert_post( $attachment, false, null ); wp_after_insert_post( $attachment, false, null );
if ( defined( 'REST_REQUEST' ) && REST_REQUEST ) { if ( defined( 'REST_REQUEST' ) && REST_REQUEST ) {
// Set a custom header with the attachment_id. /*
// Used by the browser/client to resume creating image sub-sizes after a PHP fatal error. * Set a custom header with the attachment_id.
* Used by the browser/client to resume creating image sub-sizes after a PHP fatal error.
*/
header( 'X-WP-Upload-Attachment-ID: ' . $attachment_id ); header( 'X-WP-Upload-Attachment-ID: ' . $attachment_id );
} }
@ -211,8 +213,10 @@ class WP_REST_Attachments_Controller extends WP_REST_Posts_Controller {
require_once ABSPATH . 'wp-admin/includes/media.php'; require_once ABSPATH . 'wp-admin/includes/media.php';
require_once ABSPATH . 'wp-admin/includes/image.php'; require_once ABSPATH . 'wp-admin/includes/image.php';
// Post-process the upload (create image sub-sizes, make PDF thumbnails, etc.) and insert attachment meta. /*
// At this point the server may run out of resources and post-processing of uploaded images may fail. * Post-process the upload (create image sub-sizes, make PDF thumbnails, etc.) and insert attachment meta.
* At this point the server may run out of resources and post-processing of uploaded images may fail.
*/
wp_update_attachment_metadata( $attachment_id, wp_generate_attachment_metadata( $attachment_id, $file ) ); wp_update_attachment_metadata( $attachment_id, wp_generate_attachment_metadata( $attachment_id, $file ) );
$response = $this->prepare_item_for_response( $attachment, $request ); $response = $this->prepare_item_for_response( $attachment, $request );
@ -562,8 +566,10 @@ class WP_REST_Attachments_Controller extends WP_REST_Posts_Controller {
$image_ext = pathinfo( $image_file, PATHINFO_EXTENSION ); $image_ext = pathinfo( $image_file, PATHINFO_EXTENSION );
$image_name = wp_basename( $image_file, ".{$image_ext}" ); $image_name = wp_basename( $image_file, ".{$image_ext}" );
// Do not append multiple `-edited` to the file name. /*
// The user may be editing a previously edited image. * Do not append multiple `-edited` to the file name.
* The user may be editing a previously edited image.
*/
if ( preg_match( '/-edited(-\d+)?$/', $image_name ) ) { if ( preg_match( '/-edited(-\d+)?$/', $image_name ) ) {
// Remove any `-1`, `-2`, etc. `wp_unique_filename()` will add the proper number. // Remove any `-1`, `-2`, etc. `wp_unique_filename()` will add the proper number.
$image_name = preg_replace( '/-edited(-\d+)?$/', '-edited', $image_name ); $image_name = preg_replace( '/-edited(-\d+)?$/', '-edited', $image_name );
@ -625,8 +631,10 @@ class WP_REST_Attachments_Controller extends WP_REST_Posts_Controller {
} }
if ( defined( 'REST_REQUEST' ) && REST_REQUEST ) { if ( defined( 'REST_REQUEST' ) && REST_REQUEST ) {
// Set a custom header with the attachment_id. /*
// Used by the browser/client to resume creating image sub-sizes after a PHP fatal error. * Set a custom header with the attachment_id.
* Used by the browser/client to resume creating image sub-sizes after a PHP fatal error.
*/
header( 'X-WP-Upload-Attachment-ID: ' . $new_attachment_id ); header( 'X-WP-Upload-Attachment-ID: ' . $new_attachment_id );
} }

View File

@ -229,8 +229,10 @@ class WP_REST_Autosaves_Controller extends WP_REST_Revisions_Controller {
$is_draft = 'draft' === $post->post_status || 'auto-draft' === $post->post_status; $is_draft = 'draft' === $post->post_status || 'auto-draft' === $post->post_status;
if ( $is_draft && (int) $post->post_author === $user_id && ! $post_lock ) { if ( $is_draft && (int) $post->post_author === $user_id && ! $post_lock ) {
// Draft posts for the same author: autosaving updates the post and does not create a revision. /*
// Convert the post object to an array and add slashes, wp_update_post() expects escaped array. * Draft posts for the same author: autosaving updates the post and does not create a revision.
* Convert the post object to an array and add slashes, wp_update_post() expects escaped array.
*/
$autosave_id = wp_update_post( wp_slash( (array) $prepared_post ), true ); $autosave_id = wp_update_post( wp_slash( (array) $prepared_post ), true );
} else { } else {
// Non-draft posts: create or update the post autosave. // Non-draft posts: create or update the post autosave.

View File

@ -1761,8 +1761,10 @@ class WP_REST_Comments_Controller extends WP_REST_Controller {
$posts_controller = $post_type->get_rest_controller(); $posts_controller = $post_type->get_rest_controller();
// Ensure the posts controller is specifically a WP_REST_Posts_Controller instance /*
// before using methods specific to that controller. * Ensure the posts controller is specifically a WP_REST_Posts_Controller instance
* before using methods specific to that controller.
*/
if ( ! $posts_controller instanceof WP_REST_Posts_Controller ) { if ( ! $posts_controller instanceof WP_REST_Posts_Controller ) {
$posts_controller = new WP_REST_Posts_Controller( $post->post_type ); $posts_controller = new WP_REST_Posts_Controller( $post->post_type );
} }

View File

@ -576,8 +576,10 @@ abstract class WP_REST_Controller {
$additional_fields = $this->get_additional_fields(); $additional_fields = $this->get_additional_fields();
foreach ( $additional_fields as $field_name => $field_options ) { foreach ( $additional_fields as $field_name => $field_options ) {
// For back-compat, include any field with an empty schema /*
// because it won't be present in $this->get_item_schema(). * For back-compat, include any field with an empty schema
* because it won't be present in $this->get_item_schema().
*/
if ( is_null( $field_options['schema'] ) ) { if ( is_null( $field_options['schema'] ) ) {
$properties[ $field_name ] = $field_options; $properties[ $field_name ] = $field_options;
} }
@ -630,8 +632,10 @@ abstract class WP_REST_Controller {
} }
// Check for nested fields if $field is not a direct match. // Check for nested fields if $field is not a direct match.
$nested_fields = explode( '.', $field ); $nested_fields = explode( '.', $field );
// A nested field is included so long as its top-level property /*
// is present in the schema. * A nested field is included so long as its top-level property
* is present in the schema.
*/
if ( in_array( $nested_fields[0], $fields, true ) ) { if ( in_array( $nested_fields[0], $fields, true ) ) {
$response_fields[] = $field; $response_fields[] = $field;
} }

View File

@ -61,8 +61,10 @@ class WP_REST_Global_Styles_Controller extends WP_REST_Controller {
sprintf( sprintf(
'/%s/themes/(?P<stylesheet>%s)', '/%s/themes/(?P<stylesheet>%s)',
$this->rest_base, $this->rest_base,
// Matches theme's directory: `/themes/<subdirectory>/<theme>/` or `/themes/<theme>/`. /*
// Excludes invalid directory name characters: `/:<>*?"|`. * Matches theme's directory: `/themes/<subdirectory>/<theme>/` or `/themes/<theme>/`.
* Excludes invalid directory name characters: `/:<>*?"|`.
*/
'[^\/:<>\*\?"\|]+(?:\/[^\/:<>\*\?"\|]+)?' '[^\/:<>\*\?"\|]+(?:\/[^\/:<>\*\?"\|]+)?'
), ),
array( array(
@ -565,8 +567,10 @@ class WP_REST_Global_Styles_Controller extends WP_REST_Controller {
* @return true|WP_Error True if the request has read access for the item, WP_Error object otherwise. * @return true|WP_Error True if the request has read access for the item, WP_Error object otherwise.
*/ */
public function get_theme_item_permissions_check( $request ) { public function get_theme_item_permissions_check( $request ) {
// Verify if the current user has edit_theme_options capability. /*
// This capability is required to edit/view/delete templates. * Verify if the current user has edit_theme_options capability.
* This capability is required to edit/view/delete templates.
*/
if ( ! current_user_can( 'edit_theme_options' ) ) { if ( ! current_user_can( 'edit_theme_options' ) ) {
return new WP_Error( return new WP_Error(
'rest_cannot_manage_global_styles', 'rest_cannot_manage_global_styles',
@ -638,8 +642,10 @@ class WP_REST_Global_Styles_Controller extends WP_REST_Controller {
* @return true|WP_Error True if the request has read access for the item, WP_Error object otherwise. * @return true|WP_Error True if the request has read access for the item, WP_Error object otherwise.
*/ */
public function get_theme_items_permissions_check( $request ) { // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable public function get_theme_items_permissions_check( $request ) { // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
// Verify if the current user has edit_theme_options capability. /*
// This capability is required to edit/view/delete templates. * Verify if the current user has edit_theme_options capability.
* This capability is required to edit/view/delete templates.
*/
if ( ! current_user_can( 'edit_theme_options' ) ) { if ( ! current_user_can( 'edit_theme_options' ) ) {
return new WP_Error( return new WP_Error(
'rest_cannot_manage_global_styles', 'rest_cannot_manage_global_styles',

View File

@ -1063,8 +1063,10 @@ class WP_REST_Posts_Controller extends WP_REST_Controller {
); );
} }
// (Note that internally this falls through to `wp_delete_post()` /*
// if the Trash is disabled.) * (Note that internally this falls through to `wp_delete_post()`
* if the Trash is disabled.)
*/
$result = wp_trash_post( $id ); $result = wp_trash_post( $id );
$post = get_post( $id ); $post = get_post( $id );
$response = $this->prepare_item_for_response( $post, $request ); $response = $this->prepare_item_for_response( $post, $request );
@ -1269,8 +1271,10 @@ class WP_REST_Posts_Controller extends WP_REST_Controller {
} }
} }
// Sending a null date or date_gmt value resets date and date_gmt to their /*
// default values (`0000-00-00 00:00:00`). * Sending a null date or date_gmt value resets date and date_gmt to their
* default values (`0000-00-00 00:00:00`).
*/
if ( if (
( ! empty( $schema['properties']['date_gmt'] ) && $request->has_param( 'date_gmt' ) && null === $request['date_gmt'] ) || ( ! empty( $schema['properties']['date_gmt'] ) && $request->has_param( 'date_gmt' ) && null === $request['date_gmt'] ) ||
( ! empty( $schema['properties']['date'] ) && $request->has_param( 'date' ) && null === $request['date'] ) ( ! empty( $schema['properties']['date'] ) && $request->has_param( 'date' ) && null === $request['date'] )

View File

@ -16,7 +16,7 @@
* *
* @global string $wp_version * @global string $wp_version
*/ */
$wp_version = '6.3-beta3-56192'; $wp_version = '6.3-beta3-56193';
/** /**
* Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema. * Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema.