WordPress.org
Get WordPress

Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#35799 closed enhancement (fixed)

Update class header for WP_REST_Request to communicate parameter priorities

Reported by: danielbachhuber Owned by: DrewAPicture
Priority: normal Milestone: 4.5
Component: REST API Version:
Severity: normal Keywords: needs-docs
Cc: Focuses: docs

Description

We should summarize ​https://github.com/WP-API/WP-API/issues/2114#issuecomment-182873506 in the class header so endpoint developers know what they're getting into.

Attachments (2)

35799.1.diff​ (785 bytes ) - added by danielbachhuber 11 years ago.
35799.diff​ (932 bytes ) - added by DrewAPicture 11 years ago.

Download all attachments as: .zip

Change History (8)

This ticket was mentioned in ​Slack in #core by chriscct7. ​View the logs.
11 years ago

#2 @chriscct7
11 years ago

@danielbachhuber You're proposing a docbloc that summarizes that comment correct? Or like a codex doc

@danielbachhuber
11 years ago

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

@DrewAPicture
11 years ago

#4 @DrewAPicture
11 years ago

  • Owner set to DrewAPicture
  • Status new → reviewing

@danielbachhuber Based on the comment on GitHub, I've adjusted your patch a little bit to try to be more clear about what's expected. Would you mind reviewing 35799.diff​ for accuracy?

#5 @danielbachhuber
11 years ago

@DrewAPicture works for me — :shipit:

#6 @DrewAPicture
11 years ago

  • Resolution → fixed
  • Status reviewing → closed

In 36636:

Docs: Improve documentation for WP_REST_Request to highlight a caveat of ArrayAccess when it comes to passing similar arguments for multiple request methods.

Props danielbachhuber, DrewAPicture.
Fixes #35799.

Note: See TracTickets for help on using tickets.