Changeset 6026 for sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/tools/class-filesystem.php

Timestamp:

10/13/2017 11:31:06 PM (8 years ago)

Author:

SergeyBiryukov

Message:

Plugin Directory: When searching for the main plugin file on upload, limit recursion depth to 1 level.

Scanning the deeper levels can grab the wrong data, e.g. if there's a sub-plugin with headers, it may be scanned instead of the main one.

Fixes #2804.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/tools/class-filesystem.php (modified) (2 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/tools/class-filesystem.php

@@ -81 +81 @@
      * @param bool   $recursive Optional. Whether to recurse into subdirectories. Default: false.
      * @param string $pattern   Optional. A regular expression to match files against. Default: null.
+     * @param int    $depth     Optional. Recursion depth. Default: -1 (infinite).
      * @return array All files within the passed directory.
      */
-    public static function list_files( $directory, $recursive = false, $pattern = null ) {
+    public static function list_files( $directory, $recursive = false, $pattern = null, $depth = -1 ) {
         if ( $recursive ) {
             $iterator = new \RecursiveIteratorIterator(
@@ -89 +90 @@
                 \RecursiveIteratorIterator::SELF_FIRST
             );
+
+            if ( $depth > -1 ) {
+                $iterator->setMaxDepth( $depth );
+            }
         } else {
             $iterator = new \DirectoryIterator( $directory );