Changeset 10597 for trunk/src/bp-members/bp-members-functions.php

Timestamp:

02/16/2016 02:17:57 PM (11 years ago)

Author:

djpaul

Message:

Emails: for user/site registrations emails, set recipient name to their user_login.

For multisite and for BP_SIGNUPS_SKIP_USER_CREATION , when the user account isn't created yet, we can't fetch the user's display name. Instead, use their user_login name in the email greeting.

Fixes #6913

Props imath

File:

• trunk/src/bp-members/bp-members-functions.php (modified) (3 diffs)

trunk/src/bp-members/bp-members-functions.php

@@ -1840 +1840 @@
                  */
                 if ( apply_filters( 'bp_core_signup_send_activation_key', true, $user_id, $user_email, $activation_key, $usermeta ) ) {
-                        bp_core_signup_send_validation_email( $user_id, $user_email, $activation_key );
+                        bp_core_signup_send_validation_email( $user_id, $user_email, $activation_key, $user_login );
                 }
         }
@@ -2200 +2200 @@
  * Send activation email to a newly registered user.
  *
- * @param int    $user_id    ID of the new user.
- * @param string $user_email Email address of the new user.
- * @param string $key        Activation key.
- */
-function bp_core_signup_send_validation_email( $user_id, $user_email, $key ) {
+ * @since  2.5.0 Add the $user_login parameter.
+ *
+ * @param int|bool $user_id    ID of the new user, false if BP_SIGNUPS_SKIP_USER_CREATION is true.
+ * @param string   $user_email Email address of the new user.
+ * @param string   $key        Activation key.
+ * @param string   $user_login Optional. The user login name.
+ */
+function bp_core_signup_send_validation_email( $user_id, $user_email, $key, $user_login = '' ) {
         $args = array(
                 'tokens' => array(
@@ -2214 +2217 @@
                 ),
         );
-        bp_send_email( 'core-user-registration', $user_id, $args );
+
+        if ( $user_id ) {
+                $to = $user_id;
+        } else {
+                $to = array( array( $user_email => $user_login ) );
+        }
+
+        bp_send_email( 'core-user-registration', $to, $args );
 }