Stephen S

Well, after a little digging, I have discovered that my function is NOT running first. I added error logging after each of those and found that the built-in woocommerce action (woocommerce/includes/emails/class-wc-email-customer-completed-order.php) is indeed running before the one in my functions.php, EVEN though the built in one is:

add_action( 'woocommerce_order_status_completed_notification', array( $this, 'trigger' ) );

and mine is:

add_action( 'woocommerce_order_status_completed_notification', 'mysite_woocommerce_order_status_completed', 5 );

So, any idea why that might happen? I am loathe to add a priority to or change the base code…

Thanks for the feedback, I am looking in that direction now, although updata_meta_cache either isn’t working or I am using it incorrectly. For what it is worth, my code looks like this:

function mysite_woocommerce_order_status_completed( $order_id ) {
$wooitems = new WC_Order( $order_id );
$woouser_id = $wooitems->user_id;
$_wooitems =   $wooitems->get_items();
foreach ($_wooitems as $item_id => $product) {
            $randomcode = substr(rtrim(base64_encode(md5(microtime())),"="), 0, -20);
		$signupkey = 'signup_code';
    		wc_add_order_item_meta( $item_id, $signupkey, $randomcode, true );
    		if ($woouser_id) {
    		$signupcodeused = wc_get_order_item_meta($item_id,$signupkey, true);
    		update_user_meta( $woouser_id, 'signup_code', $signupcodeused );
    		}
    		update_meta_cache( 'CHAR', $signupkey ); //NEW, trying several variations, none work so far
		}
        }

same problem here! I reverted back to the previous version which has fixed this for now.