Coding Standards fixes for 7.0

[desrosj]

Previously:

• #63168 (6.9)
• #62279 (6.8)
• #61607 (6.7)
• #60700 (6.6)
• #59650 (6.5)
• #58831 (6.4)
• #57839 (6.3)
• #56791 (6.2)
• #55647 (6.1)
• #54728 (6.0)
• #53359 (5.9)
• #52627 (5.8)
• #51799 (5.7)
• #50767 (5.6)
• #49542 (5.5)
• #49222 (5.4)
• #47632 (5.3)
• #45934 (5.1)

Per naming conventions, don’t abbreviate variable names unnecessarily; let the code be unambiguous and self-documenting.

​See PHP Coding Standards - Naming Conventions.

This PR includes renaming of the following variables:

• $post_ID to $post_id. - "Use lowercase letters in variable, action/filter, and function names"
• $ext_type to $extension_type.
• $ext to $extension.
• $id3data to $id3_data. - "Separate words via underscores."
• $msg to $message.
• $sb to $sidebar.
• $alt to $alternate.
• $pid to $post_id.
• $menu_obj to $menu_object.
• $noparents to $no_parents. - "Separate words via underscores."
• $sup to $supplemental.
• $cat_id to $category_id.
• $cat_name to $category_name.
• $out to $output.
• $r to $edit_result.
• $t to $taxonomy.
• $u to $update_result.
• $r to $result.
• $response to $response_data.
• $x to $response.

Trac ticket:
https://core.trac.wordpress.org/ticket/64226
https://core.trac.wordpress.org/ticket/63168
https://core.trac.wordpress.org/ticket/55647

Trac ticket:
https://core.trac.wordpress.org/ticket/64226
https://core.trac.wordpress.org/ticket/63168

[SergeyBiryukov]

In 61224:

Coding Standards: Use more meaningful variable names in Admin Ajax actions.

Per the ​Naming Conventions:

Don’t abbreviate variable names unnecessarily; let the code be unambiguous and self-documenting.

This commit includes renaming of the following variables:

• $post_ID to $post_id — “Use lowercase letters in variable, action/filter, and function names”.
• $ext_type to $extension_type.
• $ext to $extension.
• $id3data to $id3_data — “Separate words via underscores”.
• $msg to $message.
• $sb to $sidebar.
• $alt to $alternate.
• $pid to $post_id.
• $mid to $meta_id.
• $menu_obj to $menu_object.
• $noparents to $no_parents — “Separate words via underscores”.
• $sup to $supplemental.
• $cat_id to $category_id.
• $cat_name to $category_name.
• $out to $output.
• $r to $edit_result.
• $t to $taxonomy.
• $u to $update_result.
• $r to $result.
• $response to $response_data.
• $x to $response.

Follow-up to [53723], [55365].

Props costdev, mukesh27, SergeyBiryukov.
See #64226.

Thanks for the PR! Merged in r61224.

[westonruter]

In 61272:

Twenty Twenty-Five: Remove redundant comments for conditionally-defined functions.

This also improves consistency with other themes.

Follow-up to [59146].

Fixes #64265.
See #64226.
Props hbhalodia, huzaifaalmesbah.

[westonruter]

In 61298:

Docs: Update typing for wp_create_category().

• Ensure that wp_create_category() returns int as opposed to numeric-string.
• Rename $cat_name to $category_name to avoid abbreviating variables.
• Update docblock to remove erroneous int for $category_name param when only string is intended.
• Add missing tests for wp_create_category().

Developed in ​https://github.com/WordPress/wordpress-develop/pull/8861

Props justlevine, westonruter.
See #64238, #64226.

[westonruter]

In 61389:

Coding Standards: Improve formatting/readability of if statement in WP_Styles constructor.

Follow-up to [46287].

See #64226, #42804.

Renames the get_bloginfo(_rss?) $show parameter to $site_info as the functions don't show (ie, echo) the argument but return it.

Trac ticket: https://core.trac.wordpress.org/ticket/64226

[peterwilsoncc]

I'm in two minds about ​PR#10650.

It renames the $show parameter in the get_bloginfo(_rss)? functions to $site_info as the functions return the values rather than output them. To me, $show suggests the result is echoed.

@SergeyBiryukov Do you have thoughts on this?

[SergeyBiryukov]

Replying to peterwilsoncc:

Do you have thoughts on this?

Looks good to me at a glance, approved 👍

This PR updates variable names in wp-admin includes to strictly adhere to WordPress Coding Standards (lowercase with underscores, no unnecessary abbreviations). These changes focus on local variables to avoid backward compatibility issues.

Trac ticket: https://core.trac.wordpress.org/ticket/64226

### Changes

• src/wp-admin/includes/post.php: Renamed $id3data to $id3_data in edit_post().
• src/wp-admin/includes/media.php: Renamed $ext to $extension and $ext_type to $extension_type in media_handle_upload() and media_send_to_editor().
• src/wp-admin/includes/image-edit.php: Renamed $ext to $extension in wp_image_editor().

### Testing Instructions

1. Media Upload: Upload image, audio, and video files. Ensure uploads complete successfully and correct icons/metadata are displayed.
2. Image Editing: Go to Media Library, select an image, click "Edit Image", perform a crop or rotation, and save. Confirm the edit is applied successfully.
3. General Admin: Edit a post and verify the page loads and saves correctly without any PHP notices or errors.

Removes three instances of comments in src/wp-content/themes/twentytwentyfour/functions.php
that duplicate the function summaries in the DocBlocks immediately following them.

## Rationale
The comments “Register block styles.”, “Enqueue block stylesheets.”, and “Register pattern categories.” are unnecessary because the DocBlocks for the following functions already clearly describe their purposes:

• twentytwentyfour_block_styles
• twentytwentyfour_block_stylesheets
• twentytwentyfour_pattern_categories

Removing these duplicate comments improves code cleanliness and avoids redundant documentation.

[SergeyBiryukov]

In 61552:

Twenty Twenty-Four: Remove redundant comments for conditionally-defined functions.

This also improves consistency with other themes.

Follow-up to [56716], [61272].

Props huzaifaalmesbah, mukesh27, sabernhardt.
See #64226.

Thanks for the PR! Merged in r61552.

Removes the redundant docblock-style comment before function_exists(), and pluralizes "Patterns" in the function summary for twenty_twenty_one_register_block_pattern().

Props: huzaifaalmesbah

Trac 64226

[SergeyBiryukov]

In 61608:

Twenty Twenty-One: Remove redundant comment for conditionally-defined function.

This also improves consistency with other themes.

Follow-up to [49216], [53121], [61552], [61272].

Props huzaifaalmesbah, sabernhardt.
See #64226.

Thanks for the PR! Merged in r61608.

Trac ticket: https://core.trac.wordpress.org/ticket/64226

[westonruter]

In 61790:

Coding Standards: Apply changes from running composer format.

Developed in ​https://github.com/WordPress/wordpress-develop/pull/11111

Follow-up to r61745, r61703, r33268.

Props soean.
See #64226.

Committed in r61790 (976ca47)

[desrosj]

Opened #64897 for the 7.1 release cycle. With 7.0 RC1 due out today, going to close this out. This can be reopened or referenced if there are more changes needed related to this during 7.0.