Ticket #20778: 20778.patch

wp-includes/functions.php

@@ -1364 +1364 @@
 
 /**
  * Determines a writable directory for temporary files.
- * Function's preference is to WP_CONTENT_DIR followed by the return value of <code>sys_get_temp_dir()</code>, before finally defaulting to /tmp/
+ * Function's preference is the return value of <code>sys_get_temp_dir()</code>, followed by WP_CONTENT_DIR, before finally defaulting to /tmp/
  *
  * In the event that this function does not find a writable location, It may be overridden by the <code>WP_TEMP_DIR</code> constant in your <code>wp-config.php</code> file.
  *
@@ -1380 +1380 @@
         if ( $temp )
                 return trailingslashit($temp);
 
-        $temp = WP_CONTENT_DIR . '/';
-        if ( is_dir($temp) && @is_writable($temp) )
-                return $temp;
-
         if  ( function_exists('sys_get_temp_dir') ) {
-                $temp = sys_get_temp_dir();
-                if ( @is_writable($temp) )
-                        return trailingslashit($temp);
+                $_temp = sys_get_temp_dir();
+                if ( ( 'WIN' === strtoupper( substr( PHP_OS, 0, 3 ) ) && win_is_writable( $_temp ) ) || @is_writable( $_temp ) ) {
+                        $temp = $_temp;
+                        return trailingslashit( $temp );
+                }
         }
 
         $temp = ini_get('upload_tmp_dir');
-        if ( is_dir($temp) && @is_writable($temp) )
+        if ( is_dir( $temp ) && ( ( 'WIN' === strtoupper( substr( PHP_OS, 0, 3 ) ) && win_is_writable( $temp ) ) || @is_writable( $temp ) ) )
                 return trailingslashit($temp);
 
+        $temp = WP_CONTENT_DIR . '/';
+        if ( is_dir( $temp ) && ( ( 'WIN' === strtoupper( substr( PHP_OS, 0, 3 ) ) && win_is_writable( $temp ) ) || @is_writable( $temp ) ) )
+                return $temp;
+
         $temp = '/tmp/';
         return $temp;
 }