Tidy phpdoc

Author: westonruter

src/wp-includes/class-wp-scripts.php

@@ -997,10 +997,10 @@ private function get_eligible_loading_strategy( $handle ) {
 	 *
 	 * @since 6.3.0
 	 *
-	 * @param string              $handle              The script handle.
-	 * @param string[]|null       $eligible_strategies Optional. The list of strategies to filter. Default null.
-	 * @param array<string, true> $checked             Optional. An array of already checked script handles, used to avoid recursive loops.
-	 * @param array<string, string[]> $stored_results Optional. An array of already computed eligible loading strategies by handle, used to increase performance in large dependency lists.
+	 * @param string                  $handle              The script handle.
+	 * @param string[]|null           $eligible_strategies Optional. The list of strategies to filter. Default null.
+	 * @param array<string, true>     $checked             Optional. An array of already checked script handles, used to avoid recursive loops.
+	 * @param array<string, string[]> $stored_results      Optional. An array of already computed eligible loading strategies by handle, used to increase performance in large dependency lists.
 	 * @return string[] A list of eligible loading strategies that could be used.
 	 */
 	private function filter_eligible_strategies( $handle, $eligible_strategies = null, $checked = array(), array &$stored_results = array() ) {
@@ -1071,10 +1071,9 @@ private function filter_eligible_strategies( $handle, $eligible_strategies = nul
 	 * @see self::filter_eligible_strategies()
 	 * @see WP_Script_Modules::get_highest_fetchpriority_with_dependents()
 	 *
-	 * @param string              $handle  Script module ID.
-	 * @param array<string, true> $checked Optional. An array of already checked script handles, used to avoid recursive loops.
+	 * @param string                $handle         Script module ID.
+	 * @param array<string, true>   $checked        Optional. An array of already checked script handles, used to avoid recursive loops.
 	 * @param array<string, string> $stored_results Optional. An array of already computed max priority by handle, used to increase performance in large dependency lists.
-	 *
 	 * @return string|null Highest fetch priority for the script and its dependents.
 	 */
 	private function get_highest_fetchpriority_with_dependents( string $handle, array $checked = array(), array &$stored_results = array() ): ?string {

tests/phpunit/tests/dependencies/scripts.php

@@ -1436,8 +1436,6 @@ public function test_fetchpriority_bumping_a_to_z() {
 	}
 
 	/**
-	 * @ticket 64194
-	 *
 	 * Tests that `WP_Scripts::get_highest_fetchpriority_with_dependents()` correctly
 	 * reuses cached results (`$stored_results`) for shared dependencies in a diamond-shaped graph.
 	 *
@@ -1450,9 +1448,11 @@ public function test_fetchpriority_bumping_a_to_z() {
 	 *
 	 * This verifies that when multiple dependents share a common dependency (`D`),
 	 * the cached result for `D` is used rather than recalculating it multiple times.
+	 *
+	 * @ticket 64194
+	 *
 	 * @covers WP_Scripts::get_highest_fetchpriority_with_dependents
 	 */
-
 	public function test_highest_fetchpriority_with_dependents_uses_cached_result_for_shared_dependency() {
 		$wp_scripts = new WP_Scripts();
 
@@ -1487,6 +1487,7 @@ public function test_highest_fetchpriority_with_dependents_uses_cached_result_fo
 		if ( PHP_VERSION_ID < 80100 ) {
 			$method->setAccessible( true );
 		}
+
 		// Pass `$stored_results` BY REFERENCE.
 		$result = $method->invokeArgs( $wp_scripts, array( 'd', array(), &$stored_results ) );
 
@@ -1496,6 +1497,7 @@ public function test_highest_fetchpriority_with_dependents_uses_cached_result_fo
 			'Expected "high" indicates that the cached `$stored_results` entry for D was used instead of recalculating.'
 		);
 	}
+
 	/**
 	 * Tests that printing a script without enqueueing has the same output as when it is enqueued.
 	 *
@@ -1594,9 +1596,8 @@ public function test_loading_strategy_with_valid_blocking_registration() {
 		$expected = str_replace( "'", '"', $expected );
 		$this->assertSame( $expected, $output, 'Scripts registered with no strategy assigned, and who have no dependencies, should have no loading strategy attributes printed.' );
 	}
+
 	/**
-	 * @ticket 64194
-	 *
 	 * Tests that `WP_Scripts::filter_eligible_strategies()` correctly reuses cached results
 	 * for shared dependencies in a diamond-shaped dependency graph.
 	 *
@@ -1610,6 +1611,9 @@ public function test_loading_strategy_with_valid_blocking_registration() {
 	 * In this scenario, both B and C depend on D, and A depends on both B and C.
 	 * The goal is to confirm that when `$stored_results` already contains an entry for D,
 	 * the cached value is reused instead of recalculating the strategies for D multiple times.
+	 *
+	 * @ticket 64194
+	 *
 	 * @covers WP_Scripts::filter_eligible_strategies
 	 */
 	public function test_filter_eligible_strategies_uses_cached_result_for_shared_dependency() {
@@ -1653,6 +1657,7 @@ public function test_filter_eligible_strategies_uses_cached_result_for_shared_de
 		if ( PHP_VERSION_ID < 80100 ) {
 			$method->setAccessible( true );
 		}
+
 		// Invoke the method with `$stored_results` passed by reference.
 		$result = $method->invokeArgs( $wp_scripts, array( 'd', null, array(), &$stored_results ) );
 
@@ -1662,6 +1667,7 @@ public function test_filter_eligible_strategies_uses_cached_result_for_shared_de
 			'Expected cached `$stored_results` value for D to be reused instead of recomputed.'
 		);
 	}
+
 	/**
 	 * Tests that scripts registered for the head do indeed end up there.
 	 *