Changeset 46191

Timestamp:

09/19/2019 03:49:33 PM (4 years ago)

Author:

kadamwhite

Message:

REST API: Revert [45687].

This change may not be needed and further investigation is required before we accept it into a release.

See #46907.

File:

• trunk/src/wp-includes/rest-api/class-wp-rest-server.php (modified) (3 diffs)

trunk/src/wp-includes/rest-api/class-wp-rest-server.php

@@ -104 +104 @@
      *
      * @since 4.4.0
-     * @since 5.3.0 Added the `$request` parameter.
-     *
-     * @param WP_REST_Request $request Full data about the request.
+     *
      * @return WP_Error|null WP_Error indicates unsuccessful login, null indicates successful
      *                       or no authentication provided
      */
-    public function check_authentication( $request ) {
+    public function check_authentication() {
         /**
          * Filters REST authentication errors.
@@ -131 +129 @@
          *
          * @since 4.4.0
-         * @since 5.3.0 Added the `$request` argument.
-         *
-         * @param WP_Error|null|bool $result  WP_Error if authentication error, null if authentication
-         *                                    method wasn't used, true if authentication succeeded.
-         * @param WP_REST_Request    $request Full data about the request.
-         */
-        return apply_filters( 'rest_authentication_errors', null, $request );
+         *
+         * @param WP_Error|null|bool WP_Error if authentication error, null if authentication
+         *                              method wasn't used, true if authentication succeeded.
+         */
+        return apply_filters( 'rest_authentication_errors', null );
     }
 
@@ -328 +324 @@
         }
 
-        $result = $this->check_authentication( $request );
+        $result = $this->check_authentication();
 
         if ( ! is_wp_error( $result ) ) {