Context Navigation

← Previous Ticket
Next Ticket →

#48841 closed defect (bug) (fixed)

Add missing doc and refactor code for REST API: WP_REST_Attachments_Controller Class

Reported by:	dkarfa	Owned by:	SergeyBiryukov
Priority:	normal	Milestone:	5.4
Component:	REST API	Version:
Severity:	normal	Keywords:	has-patch
Cc:		Focuses:	rest-api

Description

Attachments (2)

48841.patch (3.1 KB ) - added by dkarfa 7 years ago.
48841.1.patch (3.1 KB ) - added by dkarfa 7 years ago.

Download all attachments as: .zip

Change History (8)

@dkarfa
7 years ago

Attachment 48841.patch added

#1 @SergeyBiryukov
7 years ago

Milestone Awaiting Review → 5.4
Owner set to SergeyBiryukov
Status new → reviewing

#2 @mukesh27
7 years ago

@dkarfa One small text change suggestion.

Registers routes for attachments.

Replace to

Register routes for the attachments.

@dkarfa
7 years ago

Attachment 48841.1.patch added

#3 @dkarfa
7 years ago

Hi @mukesh27,
I have updated the patch.
Thank you,

This ticket was mentioned in Slack in #core-restapi by timothybjacobs. View the logs.

6 years ago

This ticket was mentioned in Slack in #core by david.baumwald. View the logs.

6 years ago

#6 @SergeyBiryukov
6 years ago

Resolution → fixed
Status reviewing → closed

In 47391:

Docs: Improve documentation for WP_REST_Attachments_Controller.

This adds a missing DocBlock for ::register_routes() and parameter descriptions for ::post_process_item().

Includes minor documenation fixes in other REST API classes for consistency.

Props dkarfa, mukesh27, SergeyBiryukov.
Fixes #48841.

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core

Context Navigation

#48841 closed defect (bug) (fixed)

Add missing doc and refactor code for REST API: WP_REST_Attachments_Controller Class

Description

Attachments (2)

Change History (8)

@dkarfa 7 years ago

#1 @SergeyBiryukov 7 years ago

#2 @mukesh27 7 years ago

@dkarfa 7 years ago

#3 @dkarfa 7 years ago

This ticket was mentioned in ​Slack in #core-restapi by timothybjacobs. ​View the logs. 6 years ago

This ticket was mentioned in ​Slack in #core by david.baumwald. ​View the logs. 6 years ago

#6 @SergeyBiryukov 6 years ago

Download in other formats:

@dkarfa
7 years ago

#1 @SergeyBiryukov
7 years ago

#2 @mukesh27
7 years ago

@dkarfa
7 years ago

#3 @dkarfa
7 years ago

This ticket was mentioned in Slack in #core-restapi by timothybjacobs. View the logs.

6 years ago

This ticket was mentioned in Slack in #core by david.baumwald. View the logs.

6 years ago

#6 @SergeyBiryukov
6 years ago