WordPress.org
Get WordPress

Make WordPress Core

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#34768 closed enhancement (fixed)

Mark WP_REST_Server::get_raw_data as static

Reported by: rmccue Owned by: rmccue
Priority: low Milestone: 4.4
Component: REST API Version:
Severity: normal Keywords: needs-patch
Cc: Focuses:

Description

This isn't linked to any state in the server instance, and would be handy to use outside of the server class (such as ​in authentication).

This doesn't break BC (as you can call static methods via $this->method fine), but it's also not technically a bug, so potentially too late for 4.4.

Change History (4)

#1 @swissspidy
11 years ago

  • Keywords needs-patch added
  • Milestone Awaiting Review → 4.4

As per ​https://wordpress.slack.com/archives/core/p1448251995001841, this can be added before RC.

#2 @ocean90
11 years ago

  • Milestone 4.4 → Future Release

Let's only move it to 4.4 if it has a patch.

#3 @rmccue
11 years ago

  • Owner set to rmccue
  • Resolution → fixed
  • Status new → closed

In 35741:

REST API: Mark WP_REST_Server::get_raw_data as static.

This is just a utility function for getting the request body, not
tied to the server class.

Fixes #34768.

#4 @rmccue
11 years ago

  • Milestone Future Release → 4.4
Note: See TracTickets for help on using tickets.