Убедитесь, что переменная является экземпляром объекта класса в WooCommerce.

Question

Я использую функцию, с помощью которой я проверяю, есть ли у пользователя возвращенные товары по их предыдущему заказу, и, если они есть, применяю кредит в качестве отрицательной комиссии за корзину при оформлении заказа. Функция работает для пользователей, которые разместили заказ раньше, но вызывает критическую ошибку на сайте для новых пользователей.

Fatal error: Uncaught Error: Call to a member function get_refunds() on bool in /wp-content/themes/my-theme/refunds.php:20 Stack trace:  
#0 /wp-includes/class-wp-hook.php(287): add_last_order_refund_total_at_checkout(Object(WC_Cart))  
#1 /wp-includes/class-wp-hook.php(311): WP_Hook->apply_filters('', Array)  
#2 /wp-includes/plugin.php(478): WP_Hook->do_action(Array)  
#3 /wp-content/plugins/woocommerce/includes/class-wc-cart.php(1714): do_action('woocommerce_car...', Object(WC_Cart))  
#4 /wp-content/plugins/woocommerce/includes/class-wc-cart-totals.php(270): WC_Cart->calculate_fees()  
#5 /wp-content/plugins/woocommerce/includes/class-wc-cart-totals.php(829): WC_Cart_Totals->get_fees_from_cart()  
#6 /wp-content/plugins/woocommerce/includes/class-wc- in /wp-content/themes/my-theme/refunds.php on line 20

Вот код для проверки любых возмещений предыдущий порядок:

//Check total of refunded items from last order - add as a fee at checkout
function add_last_order_refund_total_at_checkout($cart_object){
    if ( is_admin() && ! defined( 'DOING_AJAX' ) )
        return;

    $user_id = get_current_user_id(); // The current user ID
    $customer = new WC_Customer( $user_id );
    $last_order = $customer->get_last_order();
    $order_id = $last_order;

    // Get the WC_Order Object instance (from the order ID)
    $order = wc_get_order( $order_id );

    // Get the Order refunds (array of refunds)
    $order_refunds = $order->get_refunds();
    $total_to_refund = $order->get_total_refunded()*(-1);//WIP need to check which items have tax
    
    if (!empty($order_refunds)) WC()->cart->add_fee( 'Refund', $total_to_refund  );  
}
add_action( 'woocommerce_cart_calculate_fees', 'add_last_order_refund_total_at_checkout', 10, 1 );
 

Я считаю, что мне нужно сначала проверить, есть ли у пользователя какие-либо предыдущие заказы, иначе get_refund () вызывает ошибку, поскольку у них нет заказов на проверку? Как мне это сделать безопасно?

Answer 1

Если вы посмотрите на WC_Customer get_last_order() метод документацию или исходный код , вы увидите:

/*
 * @return WC_Order|false
 */

, что означает, что get_last_order() метод может возвращать альтернативно :

• объект WC_Order
• или false логическое значение.

Итак, вы можете просто используйте в своем коде:

$last_order = $customer->get_last_order();

if ( ! $last_order )  {
   return; // Exit (Not an order)
}

Чтобы избежать этой ошибки.

Теперь вы можете использовать условные функции is_a() php, чтобы проверить, является ли переменная из определенного c класса объекта, а не что-то еще, например:

$last_order = $customer->get_last_order();

if ( ! is_a( $last_order, 'WC_Order' ) ) {
   return; // Exit (Not an order)
}

// Your other code… 

Или вы можете использовать условные функции method_exists() php для метода WC_Order get_refunds(), чтобы проверить, является ли переменная a из определенного c класса объекта, а не чего-то еще:

$last_order = $customer->get_last_order();

if ( ! method_exists( $last_order, 'get_refunds' ) )  {
   return; // Exit (Not an order)
}

// Your other code…

Три случая работают нормально, избегая ошибки

Answer 2

Попробуйте, добавлено несколько элементов управления, см. Комментарии

// Check total of refunded items from last order - add as a fee at checkout
function add_last_order_refund_total_at_checkout( $cart ) {
    if ( is_admin() && ! defined( 'DOING_AJAX' ) )
        return;

    // The current user ID
    $user_id = get_current_user_id();
    
    // User id exists
    if ( $user_id > 0 ) {
        $customer = new WC_Customer( $user_id );
        
        // Get last order
        $last_order = $customer->get_last_order();

        // True
        if ( $last_order ) {

            // Get the order id
            $order_id = $last_order->get_id();
        
            // Order ID exists
            if ( is_numeric( $order_id ) ) {
                // Get the WC_Order Object instance (from the order ID)
                $order = wc_get_order( $order_id );

                // Get the Order refunds (array of refunds)
                $order_refunds = $order->get_refunds();
            
                // NOT empty
                if ( ! empty( $order_refunds) ) {
                    // WIP need to check which items have tax
                    $total_to_refund = $order->get_total_refunded();
                
                    // Add fee
                    $cart->add_fee( 'Refund', $total_to_refund  );
                }
            }
        }
    }
}
add_action( 'woocommerce_cart_calculate_fees', 'add_last_order_refund_total_at_checkout', 10, 1 );