Как заменить в админке в статусе заказа гостя на зарегистрованного пользователя?

Question

[ligisayan]

Всем привет! Есть магазин на woocommerce.
Вручную добавляю пользователя после сделанного им заказа.
Все бы хорошо, но в статусе заказа система определяет его как гостя, а хотелось бы, чтобы в этом поле отображался, как известный ей пользователь.

Как этого можно добиться? Вот мой код:
add_action( 'woocommerce_thankyou', 'create_user_callback' );
function create_user_callback($order_id){
   $order = wc_get_order( $order_id );
   $order_detail['customer_first_name'] = get_post_meta( $order_id, '_billing_first_name', true );
   $order_detail['customer_phone']      = get_post_meta( $order_id, '_billing_phone', true );
   $order_detail['customer_email']      = get_post_meta( $order_id, '_billing_email', true );
   $order_detail['customer_address']    = get_post_meta( $order_id, '_billing_address_2', true );
   $customers = get_users( array( 'role' => 'customer' ) );
   foreach( $customers as $customer ) :
	   $customers_email = get_user_meta( $customer->ID, 'billing_phone', true );
	   if($customers_phone!=$order_detail['customer_phone'] && $customer->user_email!=$order_detail['billing_email']) {
		 	$userdata = array(
				'user_pass'          => '123',
				'user_login'         => $order_detail['customer_email'],
				'first_name'         => $order_detail['customer_first_name'],
				'user_email'         => $order_detail['customer_email'],
				'role'               => 'customer',
			);
			$userid = wp_insert_user( $userdata );
			if ( !is_wp_error( $userid ) ) { 
			    add_user_meta( $userid, 'billing_phone', $order_detail['customer_phone'] );
			    add_user_meta( $userid, 'billing_address_2', $order_detail['customer_address'] );
			}
	   }
   endforeach;
}

Answer 1

[ligisayan]

Добавление функции
wc_update_new_customer_past_orders( $userid );
помогло мне связать текущего пользователя со всеми предыдущими аккаунтами.
Почерпнул из этой статьи
А вот эта функция позволяет связать текущий заказ с моим пользователем

update_post_meta($order_id, '_customer_user', get_current_user_id());

Answer 2

[Игорь Воротнёв]

Ну, во-первых - брать всех пользователей, проходиться циклом по всему списку и сравнивать имейл чтобы выловить нужного юзера это конечно ад. Есть же get_user_by( 'email', $email ).

А во-вторых, WooCommerce юзеров по-другому регистрирует, и клиента привязывает, и там свои хуки. При чем здесь woocommerce_thankyou?

Comments

[ligisayan]

Игорь Воротнёв ну, во-первых я в первую очередь юзеров по телефону ищу, а это мета-поле 'billing_phone', но спасибо за подсказку, я описался потому, что получал инфу в
$customers_email, а сравнивал с $customers_phone, а email это уже для подстраховки.
Если есть метод поиска по метаполю другой, то охотно на него заменю, но способ
get_user_by( 'email', $email ) для него не работает точно, проверял.

А какие хуки у регистрации клиента woocommerce? Я кстати, нашел нужную мне функцию, которая все заказы привязывает к текущему юзеру
wc_update_new_customer_past_orders( $userid );

и именно в хуке woocommerce_thankyou она работает. По другому не знаю как

[Игорь Воротнёв]

ligisayan, по метаданным можно искать конкретного юзера через WP_User_Query, принцип тот же что и с WP_Query. Впрочем, если телефонный номер считается более уникальным, чем имейл (и на него опираемся), то я бы в таком случае модифицировал регистрационный процесс - нормализация телефонного номера (к одному стандарту - полный международный формат, только цифры) и использование его в качестве логина. Тогда возвращаемся к get_user_by. В любом случае, нужно этот фрагмент упрощать и оптимизировать, потому что брать всех и перебирать со сравнением - это бутылочное горлышко.

А какие хуки у регистрации клиента woocommerce?

Ну я смотрю что у вас на thankyou происходит wp_insert_user() и это как бы совсем неправильно. WooCommerce умеет регистрировать гостей в процессе оформления заказа, и в этом случае на страницу thankyou уже прилетает созданный пользователь, а заказ уже привязан к нему. Вы же, видимо, не используете родную фичу WC, а пытаетесь делать это самостоятельно.

[ligisayan]

Игорь Воротнёв,
"по метаданным можно искать конкретного юзера через WP_User_Query, "
окей, сенкс
"нормализация телефонного номера (к одному стандарту - полный международный формат, только цифры) и использование его в качестве логина"
А это как? Просить вводить юзера не полный телефон? чревато ошибками, расчет на дурака всегда должен быть

"Ну я смотрю что у вас на thankyou происходит wp_insert_user() и это как бы совсем неправильно. "
Почему? Я опирался на статью

"Вы же, видимо, не используете родную фичу WC, а пытаетесь делать это самостоятельно."
Естественно, т.к. woocommerce предлагает для этого еще поставить галку зарегистрироваться и все равно ты новый клиент или повторный, но просто не залогиненый, а это неудобно.

[Игорь Воротнёв]

ligisayan, Хм, пробежал по диагонали эту статью, интересно. Уверен, что это не единственный (и возможно даже не самый разумный) способ, но ковырять и решать столь объемную задачу сейчас, увы, времени и возможности нет (хотя не скрою, она меня заинтересовала). Работает - и хорошо. Для вас на данном этапе это самое главное.

А это как? Просить вводить юзера не полный телефон? чревато ошибками, расчет на дурака всегда должен быть

Нормализация это и есть комплекс мер, которые включают и защиту от дурака, и форсирование определенного формата, и валидацию ошибок, и потом, после того как данные прошли, их последующая обработка, чтобы в БД они всегда хранились в одинаковом формате. Например:
- ввести можно только цифры (пробелы, скобки, +, дефисы и тд - не принимаются полем типа "number")
- длина должна быть 10 (3 цифры код оператора/региона + 7 цифр сам номер) или 12 (+2 цифры международного формата). Это для Украины - у нас 38 код страны, 0XX код оператора и XXX XX XX - сам номер. У США код страны 1 цифра, например. Подробнее по нужным странам можно посмотреть тут.
- далее, если 12 цифр - валидируем что первые 2 соответствуют коду страны
- на бекенде смотрим, если 10 цифр, дописываем код страны
- сохраняем в БД в одинаковом формате: 12 цифр (код страны + код оператора + номер)

Это минимально адекватный вариант. В зависимости от требований учесть другие страны можно логику усложнять - можно выбор страны сделать селектом + геолокация, отдельно код оператора/региона (+валидировать его чтобы он был реальный, все эти коды известны и есть их базы), отдельно сам номер 7 цифр без пробелов и других символов. В общем, это не так сложно, как кажется на первый взгляд.

[Игорь Воротнёв]

ligisayan,

Естественно, т.к. woocommerce предлагает для этого еще поставить галку зарегистрироваться и все равно ты новый клиент или повторный, но просто не залогиненый, а это неудобно.

Вот тут мысль как раз пришла. По идее у WooCommerce есть фильтр перед самим созданием пользователя. Данные, возвращаемые этим фильтром передаются далее в wp_insert_user(). Если в эти данные добавить ID юзера (а его мы можем получить проверив на существование по мылу или телефону), то будет обновлен существующий юзер + далее этот же ID существующего юзера будет возвращен далее в WooCommerce в виде $customer_id. То есть, будет получен нужный результат.

[ligisayan]

Игорь Воротнёв, спасибо. Я до вашего ответа пришел к такой функции

add_action( 'woocommerce_thankyou', 'create_user_callback' );
function create_user_callback($order_id){
   $order = new WC_Order($order_id);
   $order_email = $order->billing_email;
   $email = email_exists( $order_email );  
   $user = username_exists( $order_email );
   	$custom = reset(
	 get_users(
	  array(
	   'role' => 'customer',
	   'meta_key' => 'billing_phone',
	   'meta_value' => $order->billing_phone,
	   'number' => 1
	  )
	 )
	);

   if(!$custom && $user == false && $email == false) {
   		$random_password = wp_generate_password();
	 	$userdata = array(
			'user_pass'          => $random_password,
			'user_login'         => $order_email,
			'first_name'         => $order->billing_first_name,
			'user_email'         => $order_email,
			'role'               => 'customer',
		);
		$userid = wp_insert_user( $userdata );
		if ( !is_wp_error( $userid ) ) { 
		    update_user_meta( $userid, 'billing_phone', $order->billing_phone );
		    update_user_meta( $userid, 'billing_company', '');
		    update_user_meta( $userid, 'billing_full_address', $order->billing_full_address );
		    update_post_meta($order_id, '_customer_user', $userid);
		}
		wc_update_new_customer_past_orders( $userid );
   }
   else {
   	update_post_meta($order_id, '_customer_user', $custom->ID);
   	wc_update_new_customer_past_orders( $custom->ID );
   }
}

У вас предложение использовать фильтр и заменить на это выражение - верно?

$userdata = apply_filters( 'woocommerce_new_customer_data', array(
			'user_pass'          => $random_password,
			'user_login'         => $order_email,
			'first_name'         => $order->billing_first_name,
			'user_email'         => $order_email,
			'role'               => 'customer',
		) );