Changeset 57928 for trunk

Timestamp:

04/04/2024 06:53:39 PM (3 years 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;