Ticket #6531: 6531.6.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);
+
+        $data = get_plugin_data($plugin_file);
+        $label = isset($data['Version']) ? 'list_files_cache_' . $dir . '-' . $data['Version'] : 'list_files_cache_' . $dir;
+
+        $plugin_files = get_transient( $label );
+        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( $label, $plugin_files, HOUR_IN_SECONDS );
+
         return $plugin_files;
 }
 

src/wp-admin/theme-editor.php

@@ -120 +120 @@
 foreach ( $file_types as $type ) {
         switch ( $type ) {
                 case 'php':
-                        $allowed_files += $theme->get_files( 'php', 1 );
+                        $allowed_files += $theme->get_files( 'php', -1 );
                         $has_templates = ! empty( $allowed_files );
                         break;
                 case 'css':
-                        $style_files = $theme->get_files( 'css' );
+                        $style_files = $theme->get_files( 'css', -1 );
                         $allowed_files['style.css'] = $style_files['style.css'];
                         $allowed_files += $style_files;
                         break;
                 default:
-                        $allowed_files += $theme->get_files( $type );
+                        $allowed_files += $theme->get_files( $type, -1 );
                         break;
         }
 }

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

@@ -976 +976 @@
          *                   being absolute paths.
          */
         public function get_files( $type = null, $depth = 0, $search_parent = false ) {
-                $files = (array) self::scandir( $this->get_stylesheet_directory(), $type, $depth );
+                // get and cache all theme files to start with.
+                $label = 'list_files_cache_' . $this->get('Name') . '-' . $this->get('Version');
+                $all_files = get_transient( $label );
+                if ( empty( $all_files ) ) {
+                        $all_files = (array) self::scandir( $this->get_stylesheet_directory(), null, -1 );
 
-                if ( $search_parent && $this->parent() )
-                        $files += (array) self::scandir( $this->get_template_directory(), $type, $depth );
+                        if ( $search_parent && $this->parent() )
+                                $all_files += (array) self::scandir( $this->get_template_directory(), null, -1 );
 
+                        set_transient( $label, $all_files, HOUR_IN_SECONDS );
+                }
+
+                // Filter $all_files by $type & $depth
+                $files = array();
+                if ( $type ) {
+                        $type = (array) $type;
+                        $_extensions = implode( '|', $type );
+                }
+                foreach ($all_files as $key => $file) {
+                        if ( -1 != $depth && substr_count($key,'/') > $depth ) continue; // Filter by depth.
+                        if ( ! $type || preg_match( '~\.(' . $_extensions . ')$~', $file ) ) { // Filter by type.
+                                $files[ $key ] = $file;
+                        }
+                }
+
                 return $files;
         }