Ticket #30202: 30202.1.diff

src/wp-includes/ms-functions.php

@@ -1681 +1681 @@
  * @since MU
  *
  * @param string $directory
+ * @param string $exclude Optional. Directory to exclude from the total.
  * @return int
  */
-function get_dirsize( $directory ) {
-        $dirsize = get_transient( 'dirsize_cache' );
-        if ( is_array( $dirsize ) && isset( $dirsize[ $directory ][ 'size' ] ) )
+function get_dirsize( $directory, $exclude = null ) {
+
+        // Don't get cache for calls with excludes unless it's the main site
+        if ( ! is_null( $exclude ) || ( is_main_site() && '/sites' === $exclude ) ) {
+                $dirsize = get_transient( 'dirsize_cache' );
+        } else {
+                $dirsize = false;
+        }
+
+        if ( is_array( $dirsize ) && isset( $dirsize[ $directory ][ 'size' ] ) ) {
                 return $dirsize[ $directory ][ 'size' ];
+        }
 
-        if ( false == is_array( $dirsize ) )
+        if ( false == is_array( $dirsize ) ) {
                 $dirsize = array();
+        }
 
-        $dirsize[ $directory ][ 'size' ] = recurse_dirsize( $directory );
+        $dirsize[ $directory ][ 'size' ] = recurse_dirsize( $directory, $exclude );
 
-        set_transient( 'dirsize_cache', $dirsize, HOUR_IN_SECONDS );
+        // Don't cache calls with excludes unless it's the main site
+        if ( ! is_null( $exclude ) || ( is_main_site() && '/sites' === $exclude ) ) {
+                set_transient( 'dirsize_cache', $dirsize, HOUR_IN_SECONDS );
+        }
         return $dirsize[ $directory ][ 'size' ];
 }
 
@@ -1706 +1719 @@
  * @since MU
  *
  * @param string $directory
+ * @param string $exclude Optional. Directory to exclude from the total.
  * @return int|false
  */
-function recurse_dirsize( $directory ) {
+function recurse_dirsize( $directory, $exclude = null ) {
         $size = 0;
 
         $directory = untrailingslashit( $directory );
 
-        if ( !file_exists($directory) || !is_dir( $directory ) || !is_readable( $directory ) )
+        if ( ! file_exists( $directory ) || ! is_dir( $directory ) || ! is_readable( $directory ) || $directory === $exclude ) {
                 return false;
+        }
 
-        if ($handle = opendir($directory)) {
-                while(($file = readdir($handle)) !== false) {
+        if ( $handle = opendir( $directory ) ) {
+                while ( ( $file = readdir( $handle ) ) !== false ) {
                         $path = $directory.'/'.$file;
-                        if ($file != '.' && $file != '..') {
-                                if (is_file($path)) {
-                                        $size += filesize($path);
-                                } elseif (is_dir($path)) {
-                                        $handlesize = recurse_dirsize($path);
-                                        if ($handlesize > 0)
+                        if ( $file != '.' && $file != '..' ) {
+                                if ( is_file( $path ) ) {
+                                        $size += filesize( $path );
+                                } elseif ( is_dir( $path ) ) {
+                                        $handlesize = recurse_dirsize( $path, $exclude );
+                                        if ( $handlesize > 0 ) {
                                                 $size += $handlesize;
+                                        }
                                 }
                         }
                 }
-                closedir($handle);
+                closedir( $handle );
         }
         return $size;
 }
@@ -2295 +2311 @@
         $space_used = apply_filters( 'pre_get_space_used', false );
         if ( false === $space_used ) {
                 $upload_dir = wp_upload_dir();
-                $space_used = get_dirsize( $upload_dir['basedir'] ) / 1024 / 1024;
+                $exclude = ( is_main_site() ) ? $upload_dir['basedir'] . '/sites' : null;
+                $space_used = get_dirsize( $upload_dir['basedir'], $exclude ) / 1024 / 1024;
         }
 
         return $space_used;