Changeset 39085

Timestamp:

11/02/2016 03:44:00 AM (8 years ago)

Author:

rmccue

Message:

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

This is a re-commit of [38980], which was reverted in [38990].

Props youknowriad, jnylen0, pento.
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

@@ -344 +344 @@
         $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'] );
     }