Make WordPress Core

Opened 8 years ago

Closed 6 years ago

#45799 closed enhancement (invalid)

Add an optional default parameter for WP_REST_Request::get_param method

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

Description

This patch adds a default parameter for WP_REST_Request::get_param method, just like we have a default param for get_option function.

Attachments (1)

45799.diff (845 bytes ) - added by ediamin 8 years ago.
Proposed changes for WP_REST_Request::get_param method

Download all attachments as: .zip

Change History (8)

@ediamin
8 years ago

Proposed changes for WP_REST_Request::get_param method

#1 @ediamin
8 years ago

  • Type defect (bug)enhancement

#2 follow-up: @TimothyBlynJacobs
8 years ago

What is the use case for a default value that you wouldn't want to provide by using the JSON Schema default syntax which will automatically fill in a missing value with the default.

#3 in reply to: ↑ 2 @ediamin
8 years ago

The default property in JSON Schema is not a required property and anyone could skip it.

#4 @TimothyBlynJacobs
8 years ago

Skip it in what sense? The server/request will apply the default automatically.

In what instances do you know the proper default value for a param, but not in the position to notate that default in the schema.

#5 @pento
8 years ago

  • Version trunk

#6 @TimothyBlynJacobs
7 years ago

  • Keywords close added

Conditionally adding the close keyword. @ediamin if you could illustrate the scenario you had in mind that'd be great!

#7 @TimothyBlynJacobs
6 years ago

  • Milestone Awaiting Review
  • Resolutioninvalid
  • Status newclosed
Note: See TracTickets for help on using tickets.