X-Git-Url: https://scripts.mit.edu/gitweb/autoinstalls/wordpress.git/blobdiff_plain/fa11948979fd6a4ea5705dc613b239699a459db3..af50974463450c98503e763a7836a50e260461a9:/wp-signup.php?ds=sidebyside diff --git a/wp-signup.php b/wp-signup.php index 0ecee1e1..a519bcc0 100644 --- a/wp-signup.php +++ b/wp-signup.php @@ -19,7 +19,7 @@ if ( is_array( get_site_option( 'illegal_names' )) && isset( $_GET[ 'new' ] ) && */ function do_signup_header() { /** - * Fires within the
section of the site sign-up screen. + * Fires within the head section of the site sign-up screen. * * @since 3.0.0 */ @@ -87,8 +87,8 @@ do_action( 'before_signup_form' ); * @param string $blog_title The new site title * @param array $errors */ -function show_blog_form($blogname = '', $blog_title = '', $errors = '') { - global $current_site; +function show_blog_form( $blogname = '', $blog_title = '', $errors = '' ) { + $current_site = get_current_site(); // Blog name if ( !is_subdomain_install() ) echo ''; @@ -153,8 +153,6 @@ function show_blog_form($blogname = '', $blog_title = '', $errors = '') { * * @since MU * - * @uses wp_get_current_user() to retrieve the current user - * @uses wpmu_validate_blog_signup() to validate new site signup for the current user * @return array Contains the new site data and error messages. */ function validate_blog_form() { @@ -188,7 +186,7 @@ function show_user_form($user_name = '', $user_email = '', $errors = '') { get_error_message('user_email') ) { ?> -' . $errmsg . '
'; @@ -208,7 +206,6 @@ function show_user_form($user_name = '', $user_email = '', $errors = '') { * * @since MU * - * @uses wpmu_validate_user_signup() to retrieve an array of user data * @return array Contains username, email, and error messages. */ function validate_user_form() { @@ -220,13 +217,11 @@ function validate_user_form() { * * @since MU * - * @uses wp_get_current_user() to get the current user * @param string $blogname The new site name * @param string $blog_title The new blog title * @param array $errors */ -function signup_another_blog($blogname = '', $blog_title = '', $errors = '') { - global $current_site; +function signup_another_blog( $blogname = '', $blog_title = '', $errors = '' ) { $current_user = wp_get_current_user(); if ( ! is_wp_error($errors) ) { @@ -258,7 +253,7 @@ function signup_another_blog($blogname = '', $blog_title = '', $errors = '') { $blog_title = $filtered_results['blog_title']; $errors = $filtered_results['errors']; - echo '' . __( 'There was a problem, please correct the form below and try again.' ) . '
'; @@ -304,19 +299,24 @@ function signup_another_blog($blogname = '', $blog_title = '', $errors = '') { * * @since MU * - * @uses wp_get_current_user() to retrieve the current user - * @uses wpmu_create_blog() to add a new site - * @uses confirm_another_blog_signup() to confirm the user's new site signup - * @return bool True if blog signup was validated, false if error + * @return null|boolean True if blog signup was validated, false if error. + * The function halts all execution if the user is not logged in. */ function validate_another_blog_signup() { global $wpdb, $blogname, $blog_title, $errors, $domain, $path; $current_user = wp_get_current_user(); - if ( !is_user_logged_in() ) + if ( ! is_user_logged_in() ) { die(); + } $result = validate_blog_form(); - extract($result); + + // Extracted values set/overwrite globals. + $domain = $result['domain']; + $path = $result['path']; + $blogname = $result['blogname']; + $blog_title = $result['blog_title']; + $errors = $result['errors']; if ( $errors->get_error_code() ) { signup_another_blog($blogname, $blog_title, $errors); @@ -338,7 +338,7 @@ function validate_another_blog_signup() { * * @param array $blog_meta_defaults An array of default blog meta variables. */ - $meta = apply_filters( 'signup_create_blog_meta', $blog_meta_defaults ); + $meta_defaults = apply_filters( 'signup_create_blog_meta', $blog_meta_defaults ); /** * Filter the new default site meta variables. * @@ -351,7 +351,7 @@ function validate_another_blog_signup() { * @type int $blog_public Whether search engines should be discouraged from indexing the site. 1 for true, 0 for false. * } */ - $meta = apply_filters( 'add_signup_meta', $meta ); + $meta = apply_filters( 'add_signup_meta', $meta_defaults ); wpmu_create_blog( $domain, $path, $blog_title, $current_user->ID, $meta, $wpdb->siteid ); confirm_another_blog_signup($domain, $path, $blog_title, $current_user->user_login, $current_user->user_email, $meta); @@ -389,14 +389,12 @@ function confirm_another_blog_signup( $domain, $path, $blog_title, $user_name, $ * * @since MU * - * @uses apply_filters() filter $filtered_results - * @uses show_user_form() to display the user registration form * @param string $user_name The username * @param string $user_email The user's email * @param array $errors */ -function signup_user($user_name = '', $user_email = '', $errors = '') { - global $current_site, $active_signup; +function signup_user( $user_name = '', $user_email = '', $errors = '' ) { + global $active_signup; if ( !is_wp_error($errors) ) $errors = new WP_Error(); @@ -429,8 +427,8 @@ function signup_user($user_name = '', $user_email = '', $errors = '') { ?> -