Ticket #38962: 38962.2.diff

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

@@ -672 +672 @@
          * @return WP_REST_Response|WP_Error Response object on success, or WP_Error object on failure.
          */
         public function delete_item( $request ) {
+                // We don't support delete requests in multisite.
+                if ( is_multisite() ) {
+                        return new WP_Error( 'rest_cannot_delete', __( 'The user cannot be deleted.' ), array( 'status' => 501 ) );
+                }
+
                 $id       = (int) $request['id'];
                 $reassign = isset( $request['reassign'] ) ? absint( $request['reassign'] ) : null;
                 $force    = isset( $request['force'] ) ? (bool) $request['force'] : false;

tests/phpunit/tests/rest-api/rest-users-controller.php

@@ -1642 +1642 @@
                 $request['force'] = true;
                 $response = $this->server->dispatch( $request );
 
+                // Not implemented in multisite.
+                if ( is_multisite() ) {
+                        $this->assertEquals( 501, $response->get_status() );
+                        return;
+                }
+
                 $this->assertEquals( 200, $response->get_status() );
                 $data = $response->get_data();
                 $this->assertTrue( $data['deleted'] );
@@ -1658 +1664 @@
 
                 $request = new WP_REST_Request( 'DELETE', sprintf( '/wp/v2/users/%d', $user_id ) );
                 $response = $this->server->dispatch( $request );
+
+                // Not implemented in multisite.
+                if ( is_multisite() ) {
+                        $this->assertEquals( 501, $response->get_status() );
+                        return;
+                }
+
                 $this->assertErrorResponse( 'rest_trash_not_supported', $response, 501 );
 
                 $request->set_param( 'force', 'false' );
@@ -1680 +1693 @@
                 $request['force'] = true;
                 $response = $this->server->dispatch( $request );
 
+                // Not implemented in multisite.
+                if ( is_multisite() ) {
+                        $this->assertEquals( 501, $response->get_status() );
+                        return;
+                }
+
                 $this->assertEquals( 200, $response->get_status() );
                 $data = $response->get_data();
                 $this->assertTrue( $data['deleted'] );
@@ -1695 +1714 @@
 
                 $request = new WP_REST_Request( 'DELETE', '/wp/v2/users/me' );
                 $response = $this->server->dispatch( $request );
+
+                // Not implemented in multisite.
+                if ( is_multisite() ) {
+                        $this->assertEquals( 501, $response->get_status() );
+                        return;
+                }
+
                 $this->assertErrorResponse( 'rest_trash_not_supported', $response, 501 );
 
                 $request->set_param( 'force', 'false' );
@@ -1733 +1759 @@
                 $request['force'] = true;
                 $response = $this->server->dispatch( $request );
 
+                // Not implemented in multisite.
+                if ( is_multisite() ) {
+                        $this->assertEquals( 501, $response->get_status() );
+                        return;
+                }
+
                 $this->assertErrorResponse( 'rest_user_invalid_id', $response, 404 );
         }
 
@@ -1757 +1789 @@
                 $request->set_param( 'reassign', $reassign_id );
                 $response = $this->server->dispatch( $request );
 
+                // Not implemented in multisite.
+                if ( is_multisite() ) {
+                        $this->assertEquals( 501, $response->get_status() );
+                        return;
+                }
+
                 $this->assertEquals( 200, $response->get_status() );
 
                 // Check that the post has been updated correctly
@@ -1775 +1813 @@
                 $request->set_param( 'reassign', 100 );
                 $response = $this->server->dispatch( $request );
 
+                // Not implemented in multisite.
+                if ( is_multisite() ) {
+                        $this->assertEquals( 501, $response->get_status() );
+                        return;
+                }
+
                 $this->assertErrorResponse( 'rest_user_invalid_reassign', $response, 400 );
         }