Changeset 38980

Timestamp:

10/27/2016 02:36:49 PM (8 years ago)

Author:

rachelbaker

Message:

REST API: Remove the Location header redirect for the /users/me endpoint.

Props youknowriad jnylen0.
Fixes #38521.

Location:

trunk

Files:

• src/wp-includes/rest-api/endpoints/class-wp-rest-users-controller.php (modified) (1 diff)
• tests/phpunit/tests/rest-api/rest-users-controller.php (modified) (1 diff)

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

@@ -289 +289 @@
         $response = $this->prepare_item_for_response( $user, $request );
         $response = rest_ensure_response( $response );
-        $response->header( 'Location', rest_url( sprintf( '%s/%s/%d', $this->namespace, $this->rest_base, $current_user_id ) ) );
-        $response->set_status( 302 );
 
         return $response;

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

@@ -636 +636 @@
 
         $response = $this->server->dispatch( $request );
-        $this->assertEquals( 302, $response->get_status() );
+        $this->assertEquals( 200, $response->get_status() );
 
         $headers = $response->get_headers();
-        $this->assertArrayHasKey( 'Location', $headers );
-        $this->assertEquals( rest_url( 'wp/v2/users/' . self::$user ), $headers['Location'] );
+        $this->assertArrayNotHasKey( 'Location', $headers );
+
+        $links = $response->get_links();
+        $this->assertEquals( rest_url( 'wp/v2/users/' . self::$user ), $links['self'][0]['href'] );
     }