Changeset 39161 for trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-posts-controller.php

Timestamp:

11/08/2016 01:08:49 PM (9 years ago)

Author:

joehoyle

Message:

REST API: Remove rest_get_post filter and get_post abstraction.

This filter was originally introduced in ​https://github.com/WP-API/WP-API/pull/2535 to support Customizer Changesets (née Transactions). This is a super broad filter and doesn't really fit with the design of the API, nor is it (arguably) the right level to do this.

Props rmccue.
Fixes #38701.

File:

• trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-posts-controller.php (modified) (12 diffs)

trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-posts-controller.php

@@ -351 +351 @@
     public function get_item_permissions_check( $request ) {
 
-        $post = $this->get_post( (int) $request['id'] );
+        $post = get_post( (int) $request['id'] );
 
         if ( 'edit' === $request['context'] && $post && ! $this->check_update_permission( $post ) ) {
@@ -420 +420 @@
     public function get_item( $request ) {
         $id   = (int) $request['id'];
-        $post = $this->get_post( $id );
+        $post = get_post( $id );
 
         if ( empty( $id ) || empty( $post->ID ) || $this->post_type !== $post->post_type ) {
@@ -532 +532 @@
         }
 
-        $post = $this->get_post( $post_id );
+        $post = get_post( $post_id );
 
         if ( ! empty( $schema['properties']['meta'] ) && isset( $request['meta'] ) ) {
@@ -583 +583 @@
     public function update_item_permissions_check( $request ) {
 
-        $post = $this->get_post( $request['id'] );
+        $post = get_post( $request['id'] );
         $post_type = get_post_type_object( $this->post_type );
 
@@ -616 +616 @@
     public function update_item( $request ) {
         $id   = (int) $request['id'];
-        $post = $this->get_post( $id );
+        $post = get_post( $id );
 
         if ( empty( $id ) || empty( $post->ID ) || $this->post_type !== $post->post_type ) {
@@ -668 +668 @@
         }
 
-        $post = $this->get_post( $post_id );
+        $post = get_post( $post_id );
 
         if ( ! empty( $schema['properties']['meta'] ) && isset( $request['meta'] ) ) {
@@ -705 +705 @@
     public function delete_item_permissions_check( $request ) {
 
-        $post = $this->get_post( $request['id'] );
+        $post = get_post( $request['id'] );
 
         if ( $post && ! $this->check_delete_permission( $post ) ) {
@@ -727 +727 @@
         $force = (bool) $request['force'];
 
-        $post = $this->get_post( $id );
+        $post = get_post( $id );
 
         if ( empty( $id ) || empty( $post->ID ) || $this->post_type !== $post->post_type ) {
@@ -780 +780 @@
             // the trash is disabled.)
             $result = wp_trash_post( $id );
-            $post = $this->get_post( $id );
+            $post = get_post( $id );
             $response = $this->prepare_item_for_response( $post, $request );
         }
@@ -1072 +1072 @@
         // Parent.
         if ( ! empty( $schema['properties']['parent'] ) && ! empty( $request['parent'] ) ) {
-            $parent = $this->get_post( (int) $request['parent'] );
+            $parent = get_post( (int) $request['parent'] );
 
             if ( empty( $parent ) ) {
@@ -1184 +1184 @@
      */
     public function handle_template( $template, $post_id ) {
-        if ( in_array( $template, array_keys( wp_get_theme()->get_page_templates( $this->get_post( $post_id ) ) ), true ) ) {
+        if ( in_array( $template, array_keys( wp_get_theme()->get_page_templates( get_post( $post_id ) ) ), true ) ) {
             update_post_meta( $post_id, '_wp_page_template', $template );
         } else {
@@ -1301 +1301 @@
         // Can we read the parent if we're inheriting?
         if ( 'inherit' === $post->post_status && $post->post_parent > 0 ) {
-            $parent = $this->get_post( $post->post_parent );
+            $parent = get_post( $post->post_parent );
             return $this->check_read_permission( $parent );
         }