Changeset 3289

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

Location:

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory

Files:

• api/class-base.php (modified) (1 diff)
• api/routes/class-plugin.php (modified) (4 diffs)
• api/routes/class-popular-categories.php (added)
• api/routes/class-query-plugins.php (added)
• class-plugin-directory.php (modified) (1 diff)
• standalone (added)
• standalone/class-plugins-info-api-request.php (added)
• standalone/class-plugins-info-api.php (added)
• standalone/downloads.php (moved) (moved from sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/download-handler.php) (1 diff)
• standalone/plugins-info-api.php (added)

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

@@ -14 +14 @@
         new Routes\Internal_Stats();
         new Routes\Plugin();
+        new Routes\Popular_Categories();
+        new Routes\Query_Plugins();
     }
 

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' ) ),

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

@@ -373 +373 @@
         }
         if ( 'plugin_built_for' == $term->taxonomy ) {
-            return $this->package_link( false, $this->get_plugin_post( $term->slug ) );
+            // Term slug = Post Slug = /%postname%/
+            return trailingslashit( home_url( $term->slug ) );
         }
 

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/standalone/downloads.php

@@ -12 +12 @@
  */
 
-include __DIR__ . '/class-autoloader.php';
-Autoloader\register_class_path( __NAMESPACE__, __DIR__ );
-
+include dirname( __DIR__ ) . '/class-autoloader.php';
+Autoloader\register_class_path( __NAMESPACE__, dirname( __DIR__ ) );
 $serve = new Zip\Serve();