Changeset 57928 for trunk

Timestamp:

04/04/2024 06:53:39 PM (8 months ago)

Author:

joemcgill

Message:

Themes: Avoid errors in some environments from _get_block_templates_paths.

This adds an is_dir() check in _get_block_templates_paths before trying to run a RecursiveDirectoryIterator to avoid errors being reported in New Relic even thought the errors should be handled by a try/catch block.

Follow-up to [57215].

Props iCaleb, sean212, mukesh27, joemcgill.
Fixes #60915.

File:

• trunk/src/wp-includes/block-template-utils.php (modified) (2 diffs)

trunk/src/wp-includes/block-template-utils.php

@@ -230 +230 @@
     }
     $path_list = array();
-    try {
+    if ( is_dir( $base_directory ) ) {
         $nested_files      = new RecursiveIteratorIterator( new RecursiveDirectoryIterator( $base_directory ) );
         $nested_html_files = new RegexIterator( $nested_files, '/^.+\.html$/i', RecursiveRegexIterator::GET_MATCH );
@@ -236 +236 @@
             $path_list[] = $path;
         }
-    } catch ( Exception $e ) {
-        // Do nothing.
     }
     $template_path_list[ $base_directory ] = $path_list;