Too few arguments to function MailChimp_Service::handleOrderUpdate()

Hi @hhgk, thanks for reaching out. Can you confirm a few things for us?

• What plugin version are you using?
• Can you enable Remote Diagnostics on your site? You can do this by going to the Settings tab and checking the box for Remote Diagnostics.

We look forward to hearing back from you.

Thanks!

Hi,

The plugin version is 3.1.

I’m sorry but this is a live site. I can not reactivate your plugin before the problem is solved. We also have a DEV version, but at this version the automatic subscription payment system is deactivated.

It would be great if you could find an other way to correct this problem. At least prevent your plugin from interfering with automatic payments.

Regards,

Holger

@hhgk by chance are you using PHP FPM on this machine? You might need to restart PHP to get the new code changes to take place. The second argument of that function is NULLABLE so it shouldn’t be erroring out like that.

We did actually see that the same thing happens on “order created” which we have a patch currently in progress that will be released in 3.2 and that should be coming up soon as well that you could wait for as well.

If you need to patch this immediately, I can assist personally with a code snippet but would require a plugin edit ( short term solution ).

The function in that file class-mailchimp-woocommerce-service.php on line 512, you should see this now:

/**
 * @param $order_id
 * @param $order
 */
public function handleOrderUpdate($order_id, $order = null) {
	if (empty($order)) $order = MailChimp_WooCommerce_HPOS::get_order($order_id);
	mailchimp_log('handleOrderUpdate', 'order_status');
	$this->handleOrderSaved($order_id, $order, true);
}

The same thing needs to be done inside the order creation function which is scheduled to be released in 3.2

/**
 * @param $order_id
 * @param $order
 */
public function handleOrderCreate($order_id, $order = null) {
	if (empty($order)) $order = MailChimp_WooCommerce_HPOS::get_order($order_id);
    $this->handleOrderSaved($order_id, $order, false);
}

Sorry for the issues, we’re staying on top of things and will make sure this issue is patched asap.

@ryanhungate — is this an issue only for WooCommerce Subscriptions-related orders, or is this an issue for all WooCommerce orders?

I use WooCommerce, but not WooCommerce Subscriptions, so just trying to figure out if I should be proactively editing plugin code, too, or not. ??

@hastibe thanks for reaching out. This seems to be an edge case, but we’re going to push out an update that should work for anyone else that might be firing that same hook without the proper arguments.

It’s still in the QA process but shouldn’t be too long before that gets released. If you find yourself in a spot where it needs to be fixed right away please let us know and we can assist with that.

Hi @hhgk, thanks for your patience. This issue should be resolved in the latest version of the plugin (v3.2). Don’t hesitate to reach back out if you need any further assistance.

We appreciate you using the plugin. If you’re happy with this integration, please consider leaving a?5-star review. Your feedback helps other WooCommerce customers find this integration and helps support future development.