Changeset 3289 for sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/api/routes/class-plugin.php

Timestamp:

06/02/2016 06:55:49 AM (9 years ago)

Author:

dd32

Message:

Plugin Directory: First swipe at handlers for api.wordpress.org/plugins/info/*.
The API handlers are designed to be stand-alone scripts which run only with an object cache for caching, When cached data is unavailable, WordPress is included to fetch the data from a rest api endpoint.

See #1579

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/api/routes/class-plugin.php (modified) (4 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/api/routes/class-plugin.php

@@ -4 +4 @@
 use WordPressdotorg\Plugin_Directory\Template;
 use WordPressdotorg\Plugin_Directory\API\Base;
+use WP_REST_Server;
 
 /**
@@ -14 +15 @@
     function __construct() {
         register_rest_route( 'plugins/v1', '/plugin/(?P<plugin_slug>[^/]+)/?', array(
-            'methods'  => \WP_REST_Server::READABLE,
+            'methods'  => WP_REST_Server::READABLE,
             'callback' => array( $this, 'plugin_info' ),
             'args' => array(
@@ -31 +32 @@
      */
     function plugin_info( $request ) {
-        /*
-         * NOTE: Don't try to implement things such as the `fields` support in this endpoint.
-         *        It's highly unlikely we'll expose the REST API via api.wordpress.org as it is,
-         *        it'll probably have something in front of it.
-         *
-         * NOTE: Don't use any data from the client, such as the user_agent, just stick to what is presented in the GET payload.
-         *       It can't be cached if it's using data from other sources.
-         */
         $plugin_slug = $request['plugin_slug'];
 
@@ -76 +69 @@
         $result['contributors'] = array();
 
-        foreach ( (array)get_post_meta( $post_id, 'contributors', true ) as $contributor ) {
-            $user = get_user_by( 'slug', $contributor );
+        $contributors = get_post_meta( $post_id, 'contributors', true ) ?: array( (int) $post->post_author );
+        foreach ( $contributors as $contributor ) {
+            $user = is_int( $contributor ) ? get_user_by( 'id', $contributor ) : get_user_by( 'slug', $contributor );
             if ( ! $user ) {
                 continue;
             }
 
-            $result['contributors'][ $contributor ] = array(
+            $result['contributors'][ $user->user_nicename ] = array(
                 'profile' => $this->get_user_profile_link( $user ),
                 'avatar' => get_avatar_url( $user, array( 'default' => 'monsterid', 'rating' => 'g' ) ),