I would like the Email Subject line for the admin email to change based on the product category. I've looked at ALL similar stackoverflow questions and NONE of them work for woocommerece 3.8.0 (see this and this).

What I have is this

function custom_admin_email_subject( $subject, $order ) {
    global $woocommerce;
    foreach($order->get_items() as $item_id => $item ){
        if ( has_term( 'Category 1 Name', 'product_cat' , $item->get_product_id() ) ) { 
            break;
            $subject = sprintf( 'Category 1 Email Subject Line' );
        }
    } 
    return $subject;
}

My code simply returns the default email subject line for new orders (which is set in woocommerce/settings/email). I can't figure out why my function does not recognize category names.

Can anyone tell me what is wrong with my code?

I am placing this code in my child-theme/functions.php file I am running woocommerce 3.8.0 and wordpress 5.3

I would like the Email Subject line for the admin email to change based on the product category. I've looked at ALL similar stackoverflow questions and NONE of them work for woocommerece 3.8.0 (see this and this).

What I have is this

function custom_admin_email_subject( $subject, $order ) {
    global $woocommerce;
    foreach($order->get_items() as $item_id => $item ){
        if ( has_term( 'Category 1 Name', 'product_cat' , $item->get_product_id() ) ) { 
            break;
            $subject = sprintf( 'Category 1 Email Subject Line' );
        }
    } 
    return $subject;
}

My code simply returns the default email subject line for new orders (which is set in woocommerce/settings/email). I can't figure out why my function does not recognize category names.

Can anyone tell me what is wrong with my code?

I am placing this code in my child-theme/functions.php file I am running woocommerce 3.8.0 and wordpress 5.3

• woocommerce-offtopic
• email

Share Improve this question edited Nov 19, 2019 at 22:01 fuxia♦ 107k3939 gold badges255255 silver badges459459 bronze badges asked Nov 19, 2019 at 20:14 david leedavid lee 13144 bronze badges

Add a comment  | 

1 Answer 1

Sorted by: Reset to default 0

The break is in the wrong place. When you put a break; in a loop (foreach), it quits the loop right at that point. You have your break before you set the $subject value so you're quitting the loop before the $subject is set. Set the subject, then break.

Also, your code snippet is missing the add_filter() call that triggers this filter function. This needs to be hooked to woocommerce_email_subject_new_order, the same as in the examples you linked to.

add_filter('woocommerce_email_subject_new_order', 'custom_admin_email_subject', 1, 2);    
function custom_admin_email_subject( $subject, $order ) {
    global $woocommerce;
    foreach($order->get_items() as $item_id => $item ){
        if ( has_term( 'Category 1 Name', 'product_cat' , $item->get_product_id() ) ) { 
            $subject = sprintf( 'Category 1 Email Subject Line' );
            break;
        }
    } 
    return $subject;
}