Closes #7161: Host Google Fonts - Measure time

inc/Engine/Common/AbstractFileSystem.php

@@ -116,7 +116,7 @@ public function hash_to_path( string $hash ): string {
 		 *
 		 * @param int $levels Number of levels.
 		 */
-		$levels = apply_filters( 'rocket_used_css_dir_level', 3 );
+		$levels = wpm_apply_filters_typed( 'integer', 'rocket_used_css_dir_level', 3 );
 
 		$base   = substr( $hash, 0, $levels );
 		$remain = substr( $hash, $levels );

inc/Engine/Media/Fonts/Frontend/Controller.php

@@ -70,6 +70,9 @@ public function rewrite_fonts( $html ): string {
 			return $html;
 		}
 
+		// For test purposes.
+		$start_time = microtime( true );
+
 		$html_nocomments = $this->hide_comments( $html );
 
 		$v1_fonts = $this->find( '<link(?:\s+(?:(?!href\s*=\s*)[^>])+)?(?:\s+href\s*=\s*([\'"])(?<url>(?:https?:)?\/\/fonts\.googleapis\.com\/css[^\d](?:(?!\1).)+)\1)(?:\s+[^>]*)?>', $html_nocomments );
@@ -80,6 +83,11 @@ public function rewrite_fonts( $html ): string {
 			return $html;
 		}
 
+		// Count fonts - for test purposes.
+		$total_v1    = count( $v1_fonts );
+		$total_v2    = count( $v2_fonts );
+		$total_fonts = $total_v1 + $total_v2;
+
 		foreach ( $v1_fonts as $font ) {
 			$html = $this->replace_font( $font, $html );
 		}
@@ -92,6 +100,13 @@ public function rewrite_fonts( $html ): string {
 			$html = $this->remove_preconnect_and_prefetch( $html );
 		}
 
+		// End time measurement.
+		$end_time = microtime( true );
+
+		// Log the total execution time and number of fonts processed, with breakdown.
+		$duration = $end_time - $start_time;
+		Logger::debug( "Total execution time for Host Google Fonts Feature in seconds -- $duration. Fonts processed: $total_fonts | Total v1: $total_v1 | Total v2: $total_v2", [ 'Host Fonts Locally' ] );
+
 		return $html;
 	}
 

tests/Integration/inc/Engine/Media/Fonts/Frontend/Subscriber/rewriteFonts.php

@@ -45,7 +45,7 @@ public function testShouldReturnAsExpected( $config, $expected ) {
 
 		$this->assertSame(
 			$expected['html'],
-			apply_filters('rocket_buffer', $config['html'])
+			wpm_apply_filters_typed('string', 'rocket_buffer', $config['html'])
 		);
 	}
 

tests/Unit/inc/Engine/Media/Fonts/Frontend/Controller/rewriteFonts.php

@@ -44,7 +44,7 @@ public function testShouldDoExpected( $config, $original, $expected ) {
 			->andReturn( $config['write'] );
 		$this->fonts_filesystem->shouldReceive( 'hash_to_path' )
 			->andReturnUsing( function( $hash ) {
-				$levels = apply_filters( 'rocket_used_css_dir_level', 3 );
+				$levels = wpm_apply_filters_typed('integer', 'rocket_used_css_dir_level', 3 );
 
 				$base   = substr( $hash, 0, $levels );
 				$remain = substr( $hash, $levels );