WordPress.org

Make WordPress Core

Opened 8 years ago

Closed 8 years ago

#21731 closed defect (bug) (duplicate)

ms-default-constants.php takes static wp-content instead should use WP_CONTENT_DIR variable

Reported by: anantshri Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.4.1
Component: Multisite Keywords: needs-patch
Focuses: Cc:

Description

Hi,

refer to the line marked with -> this line is using wp-content as a static string which may not always be the case. not sure if it will affect and be a show stopper. but i think the code should be updated.

function ms_upload_constants(  ) {
	global $wpdb;

	/** @since 3.0.0 */
	// Base uploads dir relative to ABSPATH
	if ( !defined( 'UPLOADBLOGSDIR' ) )
->		define( 'UPLOADBLOGSDIR', 'wp-content/blogs.dir' );

	/** @since 3.0.0 */
	if ( !defined( 'UPLOADS' ) ) {
		// Uploads dir relative to ABSPATH
		define( 'UPLOADS', UPLOADBLOGSDIR . "/{$wpdb->blogid}/files/" );
		if ( 'wp-content/blogs.dir' == UPLOADBLOGSDIR )
			define( 'BLOGUPLOADDIR', WP_CONTENT_DIR . "/blogs.dir/{$wpdb->blogid}/files/" );
	}
}



Change History (1)

#1 @nacin
8 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to duplicate
  • Status changed from new to closed

These essentially go away with #19235.

Note: See TracTickets for help on using tickets.