Changeset 40426 for branches/4.7/src/wp-includes/rest-api/endpoints/class-wp-rest-users-controller.php

Timestamp:

04/14/2017 08:46:22 AM (8 years ago)

Author:

swissspidy

Message:

REST API: Allow fetching multiple users at once via the slug parameter.

This matches similar changes previously made for posts (#38579) and terms (#40027).

Props curdin, MatheusGimenez.
Fixes #40213.

Merges [40378] to the 4.7 branch.

Location:

branches/4.7

Files:

• . (modified) (1 prop)
• src/wp-includes/rest-api/endpoints/class-wp-rest-users-controller.php (modified) (3 diffs)

branches/4.7/src/wp-includes/rest-api/endpoints/class-wp-rest-users-controller.php

@@ -221 +221 @@
             'search'   => 'search',
             'roles'    => 'role__in',
+            'slug'     => 'nicename__in',
         );
 
@@ -261 +262 @@
             $prepared_args['search'] = '*' . $prepared_args['search'] . '*';
         }
-
-        if ( isset( $registered['slug'] ) && ! empty( $request['slug'] ) ) {
-            $prepared_args['search'] = $request['slug'];
-            $prepared_args['search_columns'] = array( 'user_nicename' );
-        }
-
         /**
          * Filters WP_User_Query arguments when querying users via the REST API.
@@ -1362 +1357 @@
         $query_params['slug']    = array(
             'description'        => __( 'Limit result set to users with a specific slug.' ),
-            'type'               => 'string',
+            'type'               => 'array',
+            'items'              => array(
+                'type'               => 'string',
+            ),
         );