Changeset 60814 for trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-users-controller.php

Timestamp:

09/30/2025 03:49:18 PM (11 months ago)

Author:

johnbillion

Message:

REST API: Increase the specificity of capability checks for collections when the edit context is in use.

The edit access in now taken into account for each individual post, term, or user in the response.

Props andraganescu, desrosj, ehti, hurayraiit, iandunn, joehoyle, johnbillion, jorbin, mnelson4, noisysocks, peterwilsoncc, rmccue, timothyblynjacobs, vortfu, whyisjake, zieladam.

File:

• trunk/src/wp-includes/rest-api/endpoints/class-wp-rest-users-controller.php (modified) (4 diffs)

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

@@ -221 +221 @@
                         return new WP_Error(
                                 'rest_forbidden_context',
-                                __( 'Sorry, you are not allowed to list users.' ),
+                                __( 'Sorry, you are not allowed to edit users.' ),
                                 array( 'status' => rest_authorization_required_code() )
                         );
@@ -380 +380 @@
 
                         foreach ( $query->get_results() as $user ) {
+                                if ( 'edit' === $request['context'] && ! current_user_can( 'edit_user', $user->ID ) ) {
+                                        continue;
+                                }
+
                                 $data    = $this->prepare_item_for_response( $user, $request );
                                 $users[] = $this->prepare_response_for_collection( $data );
@@ -480 +484 @@
                 }
 
-                if ( 'edit' === $request['context'] && ! current_user_can( 'list_users' ) ) {
-                        return new WP_Error(
-                                'rest_user_cannot_view',
-                                __( 'Sorry, you are not allowed to list users.' ),
+                if ( 'edit' === $request['context'] && ! current_user_can( 'edit_user', $user->ID ) ) {
+                        return new WP_Error(
+                                'rest_forbidden_context',
+                                __( 'Sorry, you are not allowed to edit this user.' ),
                                 array( 'status' => rest_authorization_required_code() )
                         );
-                } elseif ( ! count_user_posts( $user->ID, $types ) && ! current_user_can( 'edit_user', $user->ID ) && ! current_user_can( 'list_users' ) ) {
+                }
+
+                if ( ! current_user_can( 'edit_user', $user->ID ) && ! current_user_can( 'list_users' ) && ! count_user_posts( $user->ID, $types ) ) {
                         return new WP_Error(
                                 'rest_user_cannot_view',
@@ -1087 +1093 @@
                 }
 
-                if ( in_array( 'roles', $fields, true ) ) {
+                if ( in_array( 'roles', $fields, true ) && ( current_user_can( 'list_users' ) || current_user_can( 'edit_user', $user->ID ) ) ) {
                         // Defensively call array_values() to ensure an array is returned.
                         $data['roles'] = array_values( $user->roles );