WordPress.org

Make WordPress Core

Opened 2 years ago

Last modified 42 hours ago

#41411 assigned defect (bug)

REST API: Terms endpoint responses for PUT & POST requests don't set context to edit

Reported by: westonruter Owned by: kadamwhite
Milestone: 5.3 Priority: normal
Severity: normal Version: 4.7
Component: REST API Keywords: has-patch has-unit-tests
Focuses: Cc:

Description

The endpoints for posts, users, and comments all set the context to edit when returning the resource in the response for POST and PUT requests. The terms endpoint, however, unexpectedly returns the resource with the view context. In practice this won't cause a problem since all of the fields (in core) have both the view and edit contexts, but if a plugin adds additional fields that should only appear in the edit context then this could cause a problem where the unexpected fields would be returned in the response.

Attachments (2)

41411.0.diff (935 bytes) - added by westonruter 2 years ago.
41411.1.diff (2.8 KB) - added by dlh 42 hours ago.

Download all attachments as: .zip

Change History (8)

@westonruter
2 years ago

#1 @westonruter
2 years ago

  • Keywords has-patch added

#2 @rmccue
2 years ago

  • Keywords needs-unit-tests added

lgtm, we should add unit tests to check this though.

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


6 days ago

#4 @kadamwhite
6 days ago

  • Milestone changed from Awaiting Review to 5.3
  • Owner set to kadamwhite
  • Status changed from new to assigned

Accepting for 5.3 as a quick win. I'll look into writing up a test.

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


3 days ago

@dlh
42 hours ago

#6 @dlh
42 hours ago

  • Keywords has-unit-tests added; needs-unit-tests removed

41411.1.diff adds a test.

Note: See TracTickets for help on using tickets.