PHP 8 Compatibility Changes

Resolved KevinPlusPlus
(@kevinplusplus)

4 years, 7 months ago

Hello,

Since updating from PHP 7.4.x to PHP 8.0.x, there is a warning occurring regularly in the PHP logs that should be addressed for full PHP 8 compatibility.

PHP Warning: Undefined array key “detail” in /mailchimp-for-woocommerce/includes/api/class-mailchimp-api.php on line 1894

The existence of $data[‘detail’] just needs to be checked before referencing it. isset would do the trick. 🙂

Hopefully this can make it into the next release for better PHP 8 compatibility.

Kevin

Viewing 8 replies - 1 through 8 (of 8 total)

Plugin Support Jordan Rich
(@builtbyjordan)

4 years, 7 months ago

@kevinplusplus Hey and thanks for bringing that to our attention. I’ll be sure to get this in front of our development teams eyes. I can’t however, promise this will make it to the next update. However, when we know more , we’ll update this thread.

Thanks again.

Plugin Support Jordan Rich
(@builtbyjordan)

4 years, 7 months ago

Just to chime back in on this, we’re currently testing a solution for this. While I can’t provide an exact date on solution being made publicly available, I can say we’re getting close.

Thread Starter KevinPlusPlus
(@kevinplusplus)

4 years, 7 months ago

Jordan,

Thank you so much, appreciate it!

Kevin

Plugin Support khungate
(@khungate)

4 years, 6 months ago

Hi @kevinplusplus, this has been resolved in the latest release (v2.5.4). Please let us know if we can help out in any other way.

Thread Starter KevinPlusPlus
(@kevinplusplus)

4 years, 6 months ago

Jordan,

Unfortunately, with 2.5.4 we have a new one:

PHP Warning: Undefined array key “detail” in /mailchimp-for-woocommerce/includes/api/class-mailchimp-api.php on line 1898

It seems that the ‘detail’ array element is being referenced when it may not exist:

throw new MailChimp_WooCommerce_Error($data[‘detail’], (int) $data[‘status’]);

Thanks,

Kevin Hock

stevebloomer
(@stevebloomer)

4 years, 2 months ago

Hi @kevinplusplus and @khungate ,

Any progress on the above error on line 1898.

Steven

Plugin Support khungate
(@khungate)

4 years, 2 months ago

@stevebloomer it should be resolved now. Can you grab the latest version of the plugin (v2.6.2) to be sure? If not, please create a new issue in Github: https://github.com/mailchimp/mc-woocommerce/issues.

Thanks!

stevebloomer
(@stevebloomer)

4 years, 2 months ago

Just got it and seems to be fixed, thank you.

Viewing 8 replies - 1 through 8 (of 8 total)

The topic ‘PHP 8 Compatibility Changes’ is closed to new replies.