Woocommerce Question posted in
The official Woocommerce documentation can be found here.

WooCommerce get_items() fatal error in email setting page

We use this snipped below to unsend the order complete mail for a order IF the order is a Subscription parent. This is working. But I just noticed, that I get a fatal error, when I open the WooCommerce Email settings.

Working solution but with error on WooCommerce eMail setting page:

add_filter( 'woocommerce_email_enabled_customer_completed_order', 'dcwd_conditionally_send_wc_email', 10, 2);

function dcwd_conditionally_send_wc_email( $whether_enabled, $order ) {

    //error_log( 'is_enabled: ' . current_filter() . ' - Order: ' . $object->get_order_number() ); // Debugging info.
    foreach ( $order->get_items() as $item ) {
    // Get an instance of the WC_Product object
    $product = $item->get_product();

    // Get the correct product ID (for variations we take the parent product ID)
    $product_id = $product->get_type();

   //print_r($product_id); die();

   // Check for product categories for this item
   // two encounter both (variable and normal subscriptions) we need to check both => if ($product_id == 'subscription' || $product_id == 'subscription_variation')
   if($product_id == 'subscription_variation') {

       return false;
      }
   }
   return $whether_enabled;
}

This is the error, but I do not understand what the problem is…

Fatal error: Uncaught Error: Call to a member function get_items() on null in /home/hosting7878877/www/staging65678.ch/cbcmembership/wp-content/themes/generatepress_child/functions.php:44 Stack trace: #0 /home/hosting7878877/www/staging65678.ch/cbcmembership/wp-includes/class-wp-hook.php(289): dcwd_conditionally_send_wc_email(true, NULL) #1 /home/hosting7878877/www/staging65678.ch/cbcmembership/wp-includes/plugin.php(206): WP_Hook->apply_filters(true, Array) #2 /home/hosting7878877/www/staging65678.ch/cbcmembership/wp-content/plugins/woocommerce/includes/emails/class-wc-email.php(500): apply_filters('woocommerce_ema...', true, NULL, Object(WC_Email_Customer_Completed_Order)) #3 /home/hosting7878877/www/staging65678.ch/cbcmembership/wp-content/plugins/woocommerce/includes/admin/settings/class-wc-settings-emails.php(318): WC_Email->is_enabled() #4 /home/hosting7878877/www/staging65678.ch/cbcmembership/wp-includes/class-wp-hook.php(287): WC_Settings_Emails->email_notification_setting(Array) #5 /home/hosting7878877/www/staging65678.ch/cbcmembership/wp-includes/class-wp-hook.php(311): WP_Hook->apply_fi in /home/hosting7878877/www/staging65678.ch/cbcmembership/wp-content/themes/generatepress_child/functions.php on line 44
Tags:

2

Answers


  1. 0

    I tested your code and it works for me. You can still resolve the error by checking if the variable is an order object:

    // check if the variable is an order object 
if ( ! is_a( $order, 'WC_Order' ) ) {
    return $whether_enabled;
}

    So:

    add_filter( 'woocommerce_email_enabled_customer_completed_order', 'dcwd_conditionally_send_wc_email', 10, 2);
function dcwd_conditionally_send_wc_email( $whether_enabled, $order ) {

    // check if the variable is an order object 
    if ( ! is_a( $order, 'WC_Order' ) ) {
        return $whether_enabled;
    }

    //error_log( 'is_enabled: ' . current_filter() . ' - Order: ' . $object->get_order_number() ); // Debugging info.
    foreach ( $order->get_items() as $item ) {
        // Get an instance of the WC_Product object
        $product = $item->get_product();

        // Get the correct product ID (for variations we take the parent product ID)
        $product_id = $product->get_type();

        //print_r($product_id); die();

        // Check for product categories for this item
        // two encounter both (variable and normal subscriptions) we need to check both => if ($product_id == 'subscription' || $product_id == 'subscription_variation')
        if ( $product_id == 'subscription_variation' ) {
            return false;
        }
    }
    return $whether_enabled;
}
    Login or Signup to reply.
  2. 0

    Apparently, sometimes WooCommerce passes null in $order. Add this condition in order to bypass it:

    if ( !isset( $order ) || count( $order->get_items() ) == 0 ) {
    return $whether_enabled;
}

    So the final function will be like this:

    add_filter( 'woocommerce_email_enabled_customer_completed_order', 'dcwd_conditionally_send_wc_email', 10, 2);

function dcwd_conditionally_send_wc_email( $whether_enabled, $order ) {
    if ( !isset( $order ) || count( $order->get_items() ) == 0 ) {
        return $whether_enabled;
    }

    //error_log( 'is_enabled: ' . current_filter() . ' - Order: ' . $object->get_order_number() ); // Debugging info.
    foreach ( $order->get_items() as $item ) {
    // Get an instance of the WC_Product object
    $product = $item->get_product();

    // Get the correct product ID (for variations we take the parent product ID)
    $product_id = $product->get_type();

   //print_r($product_id); die();

   // Check for product categories for this item
   // two encounter both (variable and normal subscriptions) we need to check both => if ($product_id == 'subscription' || $product_id == 'subscription_variation')
   if($product_id == 'subscription_variation') {

       return false;
      }
   }

   return $whether_enabled;
}
    Login or Signup to reply.
Please signup or login to give your own answer.

Back To Top
Search