Changeset 5533

Timestamp:

06/01/2017 08:46:01 PM (7 years ago)

Author:

coreymckrill

Message:

WordCamp Post Types: Consolidate v2 REST functionality, parity w/ v1

This moves all the actions/filters related to v2 REST API endpoints
to a separate file and adds the (hopefully) last bit of response data
that was included in v1, but was missing in v2 (speaker avatars).

For the speaker avatar data, I decided to emulate how Core presents
the avatar data for post authors in API responses. So there is a top-
level avatar_urls field that is an object containing multiple
Gravatar URLs for different image sizes.

Location:

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wc-post-types

Files:

• inc/rest-api.php (added)
• wc-post-types.php (modified) (3 diffs)

sites/trunk/wordcamp.org/public_html/wp-content/plugins/wc-post-types/wc-post-types.php

@@ -57 +57 @@
         add_filter( 'the_content', array( $this, 'add_session_info_to_speaker_posts' ) );
 
-        // REST API
-        add_action( 'init',                     array( $this, 'expose_public_post_meta'  )        );
-        add_filter( 'rest_prepare_wcb_speaker', array( $this, 'link_speaker_to_sessions' ), 10, 2 );
-        add_filter( 'rest_prepare_wcb_session', array( $this, 'link_session_to_speakers' ), 10, 2 );
-
         add_filter( 'dashboard_glance_items', array( $this, 'glance_items' ) );
         add_filter( 'option_default_comment_status', array( $this, 'default_comment_ping_status' ) );
         add_filter( 'option_default_ping_status', array( $this, 'default_comment_ping_status' ) );
+
+        add_action( 'init', array( $this, 'rest_init' ), 9 );
     }
 
@@ -77 +74 @@
         register_setting( 'wcb_sponsor_options', 'wcb_sponsor_level_order', array( $this, 'validate_sponsor_options' ) );
         add_action( 'pre_get_posts', array( $this, 'admin_pre_get_posts' ) );
+    }
+
+    /**
+     * Runs during init, because rest_api_init is too late.
+     */
+    function rest_init() {
+        require_once( 'inc/rest-api.php' );
     }
 
@@ -1398 +1402 @@
 
         return $content . $sessions_html;
-    }
-
-    /**
-     * Add non-sensitive meta fields to the speaker/session REST API endpoints
-     *
-     * if we ever want to register meta for purposes other than exposing it in the API, then this function will
-     * probably need to be re-thought and re-factored.
-     */
-    public function expose_public_post_meta() {
-        $public_session_fields = array(
-            '_wcpt_session_time' => array(
-                'type'   => 'integer',
-                'single' => true,
-            ),
-
-            '_wcpt_session_type' => array(
-                'single' => true,
-            ),
-
-            '_wcpt_session_slides' => array(
-                'single' => true,
-            ),
-
-            '_wcpt_session_video' => array(
-                'single' => true,
-            ),
-        );
-
-        wcorg_register_meta_only_on_endpoint( 'post', $public_session_fields, '/wp-json/wp/v2/sessions/' );
-
-        $public_sponsor_fields = array(
-            '_wcpt_sponsor_website' => array(
-                'single' => true,
-            )
-        );
-
-        wcorg_register_meta_only_on_endpoint( 'post', $public_sponsor_fields, '/wp-json/wp/v2/sponsors/' );
-    }
-
-    /**
-     * Link all sessions to the speaker in the `speakers` API endpoint
-     *
-     * This allows clients to request a speaker and get all their sessions embedded in the response, avoiding
-     * extra HTTP requests
-     *
-     * @param WP_REST_Response $response
-     * @param WP_Post          $post
-     *
-     * @return WP_REST_Response
-     */
-    function link_speaker_to_sessions( $response, $post ) {
-        $sessions = get_posts( array(
-            'post_type'      => 'wcb_session',
-            'posts_per_page' => 100,
-            'fields'         => 'ids',
-
-            'meta_query' => array(
-                array(
-                    'key'   => '_wcpt_speaker_id',
-                    'value' => $post->ID,
-                ),
-            ),
-        ) );
-
-        foreach( $sessions as $session_id ) {
-            $response->add_link(
-                'sessions',
-                get_rest_url( null, "/wp/v2/sessions/$session_id" ),
-                array( 'embeddable' => true )
-            );
-        }
-
-        return $response;
-    }
-
-    /**
-     * Link all speakers to the session in the `sessions` API endpoint
-     *
-     * This allows clients to request a session and get all its speakers embedded in the response, avoiding extra
-     * HTTP requests
-     *
-     * @param WP_REST_Response $response
-     * @param WP_Post          $post
-     *
-     * @return WP_REST_Response
-     */
-    function link_session_to_speakers( $response, $post ) {
-        $speaker_ids = get_post_meta( $post->ID, '_wcpt_speaker_id', false );
-
-        foreach( $speaker_ids as $speaker_id ) {
-            $response->add_link(
-                'speakers',
-                get_rest_url( null, "/wp/v2/speakers/$speaker_id" ),
-                array( 'embeddable' => true )
-            );
-        }
-
-        return $response;
     }