Ticket #6531: 6531.5a.2.diff

src/wp-admin/includes/file.php

@@ -132 +132 @@
         if ( ! $levels )
                 return false;
 
+        /**
+         * Filters the array of excluded directories and files while scanning the folder.
+         *
+         * @since 4.9
+         *
+         * @param array $exclusions Array of excluded directories and files.
+         */
+        $exclusions = (array) apply_filters( 'list_files_exclusions', array( 'CVS', 'node_modules' ) );
+
         $files = array();
         if ( $dir = @opendir( $folder ) ) {
                 while (($file = readdir( $dir ) ) !== false ) {
                         if ( in_array($file, array('.', '..') ) )
                                 continue;
+
+                        if ( '.' == $file[0] || in_array( $file, $exclusions, true ) )
+                                continue;
+
                         if ( is_dir( $folder . '/' . $file ) ) {
                                 $files2 = list_files( $folder . '/' . $file, $levels - 1);
                                 if ( $files2 )

src/wp-admin/includes/plugin.php

@@ -193 +193 @@
 function get_plugin_files($plugin) {
         $plugin_file = WP_PLUGIN_DIR . '/' . $plugin;
         $dir = dirname($plugin_file);
+
+        $plugin_files = get_transient( 'list_files_cache_' . $dir );
+        if ( ! empty( $plugin_files ) )
+                return $plugin_files;
+
         $plugin_files = array($plugin);
         if ( is_dir($dir) && $dir != WP_PLUGIN_DIR ) {
-                $plugins_dir = @ opendir( $dir );
-                if ( $plugins_dir ) {
-                        while (($file = readdir( $plugins_dir ) ) !== false ) {
-                                if ( substr($file, 0, 1) == '.' )
-                                        continue;
-                                if ( is_dir( $dir . '/' . $file ) ) {
-                                        $plugins_subdir = @ opendir( $dir . '/' . $file );
-                                        if ( $plugins_subdir ) {
-                                                while (($subfile = readdir( $plugins_subdir ) ) !== false ) {
-                                                        if ( substr($subfile, 0, 1) == '.' )
-                                                                continue;
-                                                        $plugin_files[] = plugin_basename("$dir/$file/$subfile");
-                                                }
-                                                @closedir( $plugins_subdir );
-                                        }
-                                } else {
-                                        if ( plugin_basename("$dir/$file") != $plugin )
-                                                $plugin_files[] = plugin_basename("$dir/$file");
-                                }
-                        }
-                        @closedir( $plugins_dir );
-                }
+
+                $list_files = list_files( $dir );
+                $list_files = array_map( 'plugin_basename', $list_files );
+
+                $plugin_files += $list_files;
+                $plugin_files = array_unique( $plugin_files );
+
         }
 
+        set_transient( 'list_files_cache_'. $dir, $plugin_files, MINUTE_IN_SECONDS );
+
         return $plugin_files;
 }