Я пытаюсь создать несколько коротких кодов, которые отображают некоторую информацию, такую как имя и номер заказа для всех заказов, размещенных пользователем.
Это короткие коды. Первый возвращает 0 в качестве значения, я не понимаю, где я не прав. Второй не отображает значения. Может ли кто-нибудь помочь мне, указав на ошибки?
Мне нужно работать с шорткодами, потому что я воссоздаю страницу orders.php.
// Get All Orders numbers of current user
add_shortcode( 'prcsed_order_numb' , 'prcsed_order_1' );
function prcsed_order_1(){
$customer = new WC_Customer( get_current_user_id() );
$order = new WC_Order( $customer_order );
if ( is_a( $order, 'WC_Order' ) ) {
return $order->get_order_number();
}
}
// Get All Orders Name of current user
add_shortcode( 'prcsed_order_name' , 'prcsed_order_2' );
function prcsed_order_2(){
// Get an instance of the WC_Order object
$order = new WC_Order( $customer_order );
foreach ( $customer_orders->orders as $customer_order ) {
return $order = $item->get_name();
}
}
Я настоятельно рекомендую проявлять осторожность при реализации этого решения. Код, который вы опубликовали, содержит ряд проблем и будет обрабатывать конфиденциальные данные клиентов. Ошибка может привести к утечке данных.
В обратном вызове prcsed_order_2()
вы пытаетесь использовать несуществующие переменные. Вы также возвращаетесь в цикле foreach, поэтому он никогда не пройдет первую итерацию цикла.
Пример:
// Give the callback function a clear and descriptive name.
function wpse_get_customer_order_names() {
// Can you be certain the user is logged in at this stage...
// Consider how you might want to validate the user.
// Get all orders for the current user.
$customer_orders = wc_get_orders([
'customer_id' => get_current_user_id(),
]);
// Transform the array of order objects into an array of order names.
$order_names = array_map( function( $order ) {
return $order->name;
}, $customer_orders );
// Return as a string, ready for output,
return implode( ', ', $order_names );
}
add_shortcode( 'prcsed_order_name' , 'wpse_get_customer_order_names' );
Те же принципы можно применить к другим связанным шорткодам, которые касаются данных заказа клиента. В дополнение к проблеме безопасности, поднятой ранее, я бы поставил под сомнение влияние такой реализации на производительность.
Здесь огромное количество ошибок, возврат в цикле FOR просто вернет 1-ю итерацию и переменные, которые, кажется, не инициализированы и т. д.... но для тестирования
Возможно, вы вызываете шорткод для ранней попытки; Я подправил методы только для собственного тестирования
add_action('init', 'xp33221_add_custom_shortcode');
function xp33221_add_custom_shortcode()
{
add_shortcode( 'prcsed_order_numb' , 'prcsed_order_1' );
add_shortcode( 'prcsed_order_name' , 'prcsed_order_2' );
}
function getCustOrder(int $orderNum): ?WC_Order
{
$order = wc_get_order($orderNum);
if ($order instanceof \WC_Order)) {
return $order;
}
return null;
}
function prcsed_order_1(): ?int
{
$order = getCustOrder($customer_order);
return $order instanceof \WC_Order
? $order->get_order_number()
: $order;
}
function prcsed_order_2() null|mixed
{
$order = getCustOrder($customer_order);
return $order instanceof \WC_Order
? $order->get_order_key()
: $order;
}
Спасибо за ответ. Он отлично работает, как и ожидалось, даже если я ищу другой результат. Не могли бы вы дать мне дополнительные указания о том, как принять меры безопасности и производительности? Есть ли какие-либо справочные ссылки или рекомендации?