diff --git a/src/wp-includes/abilities-api.php b/src/wp-includes/abilities-api.php index 835bd535d2487..73ba658f3f10d 100644 --- a/src/wp-includes/abilities-api.php +++ b/src/wp-includes/abilities-api.php @@ -132,8 +132,7 @@ * * Ability names must follow these rules: * - * - Contain 2 to 4 segments separated by forward slashes - * (e.g., `my-plugin/my-ability`, `my-plugin/resource/find`, `my-plugin/resource/sub/find`). + * - Include a namespace prefix (e.g., `my-plugin/my-ability`). * - Use only lowercase alphanumeric characters, dashes, and forward slashes. * - Use descriptive, action-oriented names (e.g., `process-payment`, `generate-report`). * @@ -226,8 +225,9 @@ * @see wp_register_ability_category() * @see wp_unregister_ability() * - * @param string $name The name of the ability. Must be the fully-namespaced - * string identifier, e.g. `my-plugin/my-ability` or `my-plugin/resource/my-ability`. + * @param string $name The name of the ability. Must be a namespaced string containing + * a prefix, e.g., `my-plugin/my-ability`. Can only contain lowercase + * alphanumeric characters, dashes, and forward slashes. * @param array $args { * An associative array of arguments for configuring the ability. * @@ -318,7 +318,7 @@ function wp_register_ability( string $name, array $args ): ?WP_Ability { * @see wp_register_ability() * * @param string $name The name of the ability to unregister, including namespace prefix - * (e.g., 'my-plugin/my-ability' or 'my-plugin/resource/find'). + * (e.g., 'my-plugin/my-ability'). * @return WP_Ability|null The unregistered ability instance on success, `null` on failure. */ function wp_unregister_ability( string $name ): ?WP_Ability { @@ -351,7 +351,7 @@ function wp_unregister_ability( string $name ): ?WP_Ability { * @see wp_get_ability() * * @param string $name The name of the ability to check, including namespace prefix - * (e.g., 'my-plugin/my-ability' or 'my-plugin/resource/find'). + * (e.g., 'my-plugin/my-ability'). * @return bool `true` if the ability is registered, `false` otherwise. */ function wp_has_ability( string $name ): bool { @@ -383,7 +383,7 @@ function wp_has_ability( string $name ): bool { * @see wp_has_ability() * * @param string $name The name of the ability, including namespace prefix - * (e.g., 'my-plugin/my-ability' or 'my-plugin/resource/find'). + * (e.g., 'my-plugin/my-ability'). * @return WP_Ability|null The registered ability instance, or `null` if not registered. */ function wp_get_ability( string $name ): ?WP_Ability { diff --git a/src/wp-includes/abilities-api/class-wp-abilities-registry.php b/src/wp-includes/abilities-api/class-wp-abilities-registry.php index 758dd2c2524df..ecd6dc2785e70 100644 --- a/src/wp-includes/abilities-api/class-wp-abilities-registry.php +++ b/src/wp-includes/abilities-api/class-wp-abilities-registry.php @@ -43,8 +43,9 @@ final class WP_Abilities_Registry { * * @see wp_register_ability() * - * @param string $name The name of the ability. Must be the fully-namespaced - * string identifier, e.g. `my-plugin/my-ability` or `my-plugin/resource/my-ability`. + * @param string $name The name of the ability. The name must be a string containing a namespace + * prefix, i.e. `my-plugin/my-ability`. It can only contain lowercase + * alphanumeric characters, dashes and the forward slash. * @param array $args { * An associative array of arguments for the ability. * @@ -77,11 +78,11 @@ final class WP_Abilities_Registry { * @return WP_Ability|null The registered ability instance on success, null on failure. */ public function register( string $name, array $args ): ?WP_Ability { - if ( ! preg_match( '/^[a-z0-9-]+(?:\/[a-z0-9-]+){1,3}$/', $name ) ) { + if ( ! preg_match( '/^[a-z0-9-]+\/[a-z0-9-]+$/', $name ) ) { _doing_it_wrong( __METHOD__, __( - 'Ability name must contain 2 to 4 segments separated by forward slashes, e.g. "my-plugin/my-ability" or "my-plugin/resource/my-ability". It can only contain lowercase alphanumeric characters, dashes, and forward slashes.' + 'Ability name must be a string containing a namespace prefix, i.e. "my-plugin/my-ability". It can only contain lowercase alphanumeric characters, dashes and the forward slash.' ), '6.9.0' ); diff --git a/src/wp-includes/abilities-api/class-wp-ability.php b/src/wp-includes/abilities-api/class-wp-ability.php index bdcb8c0bd017a..967f1641156b0 100644 --- a/src/wp-includes/abilities-api/class-wp-ability.php +++ b/src/wp-includes/abilities-api/class-wp-ability.php @@ -52,7 +52,7 @@ class WP_Ability { /** * The name of the ability, with its namespace. - * Examples: `my-plugin/my-ability`, `my-plugin/resource/find`. + * Example: `my-plugin/my-ability`. * * @since 6.9.0 * @var string @@ -340,7 +340,7 @@ protected function prepare_properties( array $args ): array { /** * Retrieves the name of the ability, with its namespace. - * Examples: `my-plugin/my-ability`, `my-plugin/resource/find`. + * Example: `my-plugin/my-ability`. * * @since 6.9.0 * diff --git a/tests/phpunit/tests/abilities-api/wpAbilitiesRegistry.php b/tests/phpunit/tests/abilities-api/wpAbilitiesRegistry.php index b9cc58279c118..32479d69e2f8c 100644 --- a/tests/phpunit/tests/abilities-api/wpAbilitiesRegistry.php +++ b/tests/phpunit/tests/abilities-api/wpAbilitiesRegistry.php @@ -136,74 +136,6 @@ public function test_register_invalid_uppercase_characters_in_name() { $this->assertNull( $result ); } - /** - * Should accept ability name with 3 segments (2 slashes). - * - * @ticket 64098 - * - * @covers WP_Abilities_Registry::register - */ - public function test_register_valid_name_with_three_segments() { - $result = $this->registry->register( 'test/sub/add-numbers', self::$test_ability_args ); - $this->assertInstanceOf( WP_Ability::class, $result ); - $this->assertSame( 'test/sub/add-numbers', $result->get_name() ); - } - - /** - * Should accept ability name with 4 segments (3 slashes). - * - * @ticket 64098 - * - * @covers WP_Abilities_Registry::register - */ - public function test_register_valid_name_with_four_segments() { - $result = $this->registry->register( 'test/sub/deep/add-numbers', self::$test_ability_args ); - $this->assertInstanceOf( WP_Ability::class, $result ); - $this->assertSame( 'test/sub/deep/add-numbers', $result->get_name() ); - } - - /** - * Should reject ability name with 5 segments (exceeds maximum of 4). - * - * @ticket 64098 - * - * @covers WP_Abilities_Registry::register - * - * @expectedIncorrectUsage WP_Abilities_Registry::register - */ - public function test_register_invalid_name_with_five_segments() { - $result = $this->registry->register( 'test/a/b/c/too-deep', self::$test_ability_args ); - $this->assertNull( $result ); - } - - /** - * Should reject ability name with empty segments (double slashes). - * - * @ticket 64098 - * - * @covers WP_Abilities_Registry::register - * - * @expectedIncorrectUsage WP_Abilities_Registry::register - */ - public function test_register_invalid_name_with_empty_segment() { - $result = $this->registry->register( 'test//add-numbers', self::$test_ability_args ); - $this->assertNull( $result ); - } - - /** - * Should reject ability name with trailing slash. - * - * @ticket 64098 - * - * @covers WP_Abilities_Registry::register - * - * @expectedIncorrectUsage WP_Abilities_Registry::register - */ - public function test_register_invalid_name_with_trailing_slash() { - $result = $this->registry->register( 'test/add-numbers/', self::$test_ability_args ); - $this->assertNull( $result ); - } - /** * Should reject ability registration without a label. * diff --git a/tests/phpunit/tests/rest-api/wpRestAbilitiesV1RunController.php b/tests/phpunit/tests/rest-api/wpRestAbilitiesV1RunController.php index 0c03d72dab8a5..bccc30c2f2e94 100644 --- a/tests/phpunit/tests/rest-api/wpRestAbilitiesV1RunController.php +++ b/tests/phpunit/tests/rest-api/wpRestAbilitiesV1RunController.php @@ -379,43 +379,6 @@ private function register_test_abilities(): void { ) ); - // Ability with nested namespace (3 segments). - $this->register_test_ability( - 'test/math/add', - array( - 'label' => 'Nested Add', - 'description' => 'Adds numbers with nested namespace', - 'category' => 'math', - 'input_schema' => array( - 'type' => 'object', - 'properties' => array( - 'a' => array( - 'type' => 'number', - 'description' => 'First number', - ), - 'b' => array( - 'type' => 'number', - 'description' => 'Second number', - ), - ), - 'required' => array( 'a', 'b' ), - 'additionalProperties' => false, - ), - 'output_schema' => array( - 'type' => 'number', - ), - 'execute_callback' => static function ( array $input ) { - return $input['a'] + $input['b']; - }, - 'permission_callback' => static function () { - return current_user_can( 'edit_posts' ); - }, - 'meta' => array( - 'show_in_rest' => true, - ), - ) - ); - // Read-only ability for query params testing. $this->register_test_ability( 'test/query-params', @@ -469,31 +432,6 @@ public function test_execute_regular_ability_post(): void { $this->assertEquals( 8, $response->get_data() ); } - /** - * Test executing an ability with a nested namespace (3 segments) via REST. - * - * @ticket 64098 - */ - public function test_execute_nested_namespace_ability(): void { - $request = new WP_REST_Request( 'POST', '/wp-abilities/v1/abilities/test/math/add/run' ); - $request->set_header( 'Content-Type', 'application/json' ); - $request->set_body( - wp_json_encode( - array( - 'input' => array( - 'a' => 10, - 'b' => 7, - ), - ) - ) - ); - - $response = $this->server->dispatch( $request ); - - $this->assertEquals( 200, $response->get_status() ); - $this->assertEquals( 17, $response->get_data() ); - } - /** * Test executing a read-only ability with GET. *