Make WordPress Core

Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#57082 closed enhancement (fixed)

Document the globals in various functions

Reported by: krunal265's profile krunal265 Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 6.2 Priority: normal
Severity: normal Version:
Component: General Keywords: has-patch
Focuses: docs Cc:

Description

Comment on some functions that need to be improved. In the comment, need to mention the global variable that is used in the function.

Files:
1) wp-admin/includes/post.php

function name: the_block_editor_meta_boxes()

2) wp-includes/block-template-utils.php

function name: wp_generate_block_templates_export_file()

3) wp-includes/class-wp-oembed-controller.php

function name: get_proxy_item()

Attachments (1)

57082.patch (1.6 KB) - added by krunal265 2 years ago.
Patch added

Download all attachments as: .zip

Change History (6)

@krunal265
2 years ago

Patch added

#1 @krunal265
2 years ago

  • Keywords has-patch added

#2 @SergeyBiryukov
2 years ago

  • Milestone changed from Awaiting Review to 6.2

#3 @SergeyBiryukov
2 years ago

Thanks for the patch!

#4 @SergeyBiryukov
2 years ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 54831:

Docs: Document the usage of globals in some functions.

This affects:

  • the_block_editor_meta_boxes()
  • wp_generate_block_templates_export_file()
  • WP_oEmbed_Controller::get_proxy_item()

Follow-up to [44131], [48135], [51151], [53129].

Props krunal265.
Fixes #57082.

#5 @SergeyBiryukov
2 years ago

  • Summary changed from Comment on some functions needs to be improved to Document the globals in various functions
Note: See TracTickets for help on using tickets.