Changeset 6484

Timestamp:

01/31/2018 04:07:08 AM (7 years ago)

Author:

dd32

Message:

Theme Directory: API: Allow for querying multiple themes on the theme_information action through the 'slugs' parameter.

Example: ​https://api.wordpress.org/themes/info/1.2/?action=theme_information&request[slugs][]=twentyten&request[slugs][]=twentyeleven

See #1304

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/theme-directory/class-themes-api.php (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/plugins/theme-directory/class-themes-api.php

@@ -294 +294 @@
 
     /**
+     * Retrieve specific information about multiple theme.
+     */
+    public function theme_information_multiple() {
+        if ( empty( $this->request->slugs ) ) {
+            $this->response = (object) array( 'error' => 'Slugs not provided' );
+            return;
+        }
+
+        $slugs = is_array( $this->request->slugs ) ? $this->request->slugs : explode( ',', $this->request->slugs );
+        $slugs = array_unique( array_map( 'trim', $slugs ) );
+
+        if ( count( $slugs ) > 100 ) {
+            $this->response = (object) array( 'error' => 'A maximum of 100 themes can be queried at once.' );
+            return;
+        }
+
+        $response = array();
+        unset( $this->request->slugs ); // So it doesn't affect caching.
+        foreach ( $slugs as $slug ) {
+            $this->request->slug = $slug;
+            $this->theme_information();
+            $response[ $slug ] = $this->response;
+        }
+
+        $this->response = $response;
+    }
+
+    /**
      * Retrieve specific information about a theme.
      */
     public function theme_information() {
         global $post;
+
+        // Support the 'slugs' parameter to fetch multiple themes at once.
+        if ( ! empty( $this->request->slugs ) ) {
+            $this->theme_information_multiple();
+            return;
+        }
+
         // Theme slug to identify theme.
         if ( empty( $this->request->slug ) ) {