Ticket #48530: 48530.diff

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

@@ -706 +706 @@
          * used as the delimiter with preg_match()
          *
          * @since 4.4.0
+         * @since 5.4.0 Add $namespace parameter.
          *
+         * @param string $namespace Optionally, only return routes in the given namespace.
          * @return array `'/path/regex' => array( $callback, $bitmask )` or
          *               `'/path/regex' => array( array( $callback, $bitmask ), ...)`.
          */
-        public function get_routes() {
+        public function get_routes( $namespace = '' ) {
+                $endpoints = $this->endpoints;
+
+                if ( $namespace ) {
+                        $endpoints = wp_list_filter( $endpoints, array( 'namespace' => $namespace ) );
+                }
 
                 /**
                  * Filters the array of available endpoints.
@@ -722 +729 @@
                  *                         `'/path/regex' => array( $callback, $bitmask )` or
                  *                         `'/path/regex' => array( array( $callback, $bitmask ).
                  */
-                $endpoints = apply_filters( 'rest_endpoints', $this->endpoints );
+                $endpoints = apply_filters( 'rest_endpoints', $endpoints );
 
                 // Normalise the endpoints.
                 $defaults = array(
@@ -834 +841 @@
                 $method = $request->get_method();
                 $path   = $request->get_route();
 
-                foreach ( $this->get_routes() as $route => $handlers ) {
+                $routes = array();
+
+                foreach ( $this->get_namespaces() as $namespace ) {
+                        if ( 0 === strpos( ltrim( $path, '/' ), $namespace ) ) {
+                                $routes = $this->get_routes( $namespace );
+                                break;
+                        }
+                }
+
+                if ( ! $routes ) {
+                        $routes = $this->get_routes();
+                }
+
+                foreach ( $routes as $route => $handlers ) {
                         $match = preg_match( '@^' . $route . '$@i', $path, $matches );
 
                         if ( ! $match ) {

tests/phpunit/tests/rest-api/rest-server.php

@@ -1211 +1211 @@
                 $this->assertEquals( '', rest_get_server()->sent_body );
         }
 
+        /**
+         * @ticket 48530
+         */
+        public function test_get_routes_respects_namespace_parameter() {
+                $routes = rest_get_server()->get_routes( 'oembed/1.0' );
+
+                foreach ( $routes as $route => $handlers ) {
+                        $this->assertStringStartsWith( '/oembed/1.0', $route );
+                }
+        }
+
         public function _validate_as_integer_123( $value, $request, $key ) {
                 if ( ! is_int( $value ) ) {
                         return new WP_Error( 'some-error', 'This is not valid!' );