WordPress.org
Get WordPress

Make WordPress Core

Opened 15 months ago

Closed 15 months ago

#63744 closed enhancement (fixed)

Suggestion Regarding PHPDoc Usage for WP_REST_Request

Reported by: viralsampat Owned by: audrasjb
Priority: normal Milestone: 6.9
Component: REST API Version: 6.5
Severity: minor Keywords: has-patch commit
Cc: Focuses: docs

Description

Hello Team,

While reviewing the WordPress core files, I noticed the use of a PHPDoc block for the WP_REST_Request. I believe this is a good practice, as it enhances code clarity and serves as helpful documentation for developers.

Just wanted to highlight this observation.

Thanks,

Attachments (1)

63744.patch​ (711 bytes ) - added by viralsampat 15 months ago.
I have checked above mentioned issue and resolved it. Here, I have added my patch.

Download all attachments as: .zip

Change History (4)

@viralsampat
15 months ago

I have checked above mentioned issue and resolved it. Here, I have added my patch.

#1 @mukesh27
15 months ago

  • Component General → REST API
  • Keywords commit added; dev-feedback needs-testing removed
  • Milestone Awaiting Review → 6.9
  • Severity normal → minor
  • Type defect (bug) → enhancement
  • Version → 6.5

Thanks @viralsampat for ticket and patch.

63744.patch​ Looks good to me.

mark ready for commit

#2 @audrasjb
15 months ago

  • Owner set to audrasjb
  • Status new → accepted

#3 @audrasjb
15 months ago

  • Resolution → fixed
  • Status accepted → closed

In 60525:

Docs: Docblock improvement in WP_REST_Font_Collections_Controller.

Props viralsampat , mukesh27.
Fixes #63744.
See #63166.

Note: See TracTickets for help on using tickets.