BuddyPress.org

I’m now trying to add firstname, lastname and email address straight to the database tables of my mailing list. I’ve added the queries to the function I had put together here:

function synchro_wp_usermeta($user_id, $password, $meta) {

global $bp, $wpdb;



...



$wpdb->query("INSERT mailingusers SET users_id='$user_id', group_id='1', signup_date= UNIX_TIMESTAMP(), firstname= '$firstname', lastname= '$lastname', email_address= '$user_email'");

$wpdb->query("INSERT mailingcdata SET cdata_id=NULL, user_id='$user_id', cfield_id='1', value='$company'");



}

add_action( 'wpmu_activate_user', 'synchro_wp_usermeta', 10, 3);

This mostly works, except the for the email address. $firstname, $lastname and $user_id are all correctly added to the table, but the email_address field stays empty.

Can anyone spot the problem?

How can I “call” the user’s email address upon ‘wpmu_activate_user’?

Adding a $user_email argument only produces missing argument errors. I’m out of guesses…

$current_user->user_email doesn’t work either:

$wpdb->query("INSERT mailingusers SET users_id='$user_id', group_id='1', signup_date= UNIX_TIMESTAMP(), firstname= '$firstname', lastname= '$lastname', email_address= '$current_user->user_email'");

This works:

function synchro_mailinglist($user_id, $password, $meta) {

global $bp, $wpdb;



$email = $wpdb->get_var("SELECT user_email FROM $wpdb->users WHERE ID='$user_id'");



$fullname = $meta[field_1];

$space = strpos( $fullname, ' ' );



$company = $meta[field_2];



if ( false === $space ) {

$firstname = $fullname;

$lastname = '';

} else {

$firstname = substr( $fullname, 0, $space );

$lastname = trim( substr( $fullname, $space, strlen($fullname) ) );

}



$firstname = nameize($firstname);

$lastname = nameize($lastname);



...



$wpdb->query("INSERT mailingusers SET users_id='$user_id', group_id='1', signup_date= UNIX_TIMESTAMP(), firstname= '$firstname', lastname= '$lastname', email_address = '$email' ");

$wpdb->query("INSERT mailingcdata SET cdata_id=NULL, user_id='$user_id', cfield_id='1', value='$company'");



}

add_action( 'wpmu_activate_user', 'synchro_mailinglist', 10, 3);

I used it as part of this function. This should work other mailinglist scripts as well, just change the table and field names.

I don’t understand why $user_email didn’t work. I had to pull the user_email from the database. Is there a cleaner way?

Thanks Nicola Greco!

I knew there had to be a better solution, something like this, but couldn’t put it together.

So I replaced that database line with this:

$uid = get_userdata($user_id);

$email = $uid->user_email;

Works fine.