WordPress.org
Get WordPress

Make WordPress Core

Opened 6 months ago

Closed 6 months ago

#63744 closed enhancement (fixed)

Suggestion Regarding PHPDoc Usage for WP_REST_Request

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

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 6 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
6 months ago

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

#1 @mukesh27
6 months ago

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

Thanks @viralsampat for ticket and patch.

63744.patch​ Looks good to me.

mark ready for commit

#2 @audrasjb
6 months ago

  • Owner set to audrasjb
  • Status changed from new to accepted

#3 @audrasjb
6 months ago

  • Resolution set to fixed
  • Status changed from accepted to 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.