search

cloudinary / cloudinary_magento2

Cloudinary's Magento 2 extension. Upload product images to the cloud, manipulate them to match your graphic design and optimize images for better user experience
MIT License
15 stars 8 forks source link

Magento 2.4.4 PHP 8.1 Deprecated issue #115

Closed hanhpv closed 2 years ago

hanhpv commented 2 years ago

Magento 2.4.4 Extension version: 1.18.0 PHP 8.1

In admin go to Catalog > Products

Deprecated Functionality: strlen(): Passing null to parameter #1 ($string) of type string is deprecated in /var/www/html/vendor/cloudinary/cloudinary_php/src/Cloudinary.php on line 546
Exception in /var/www/html/vendor/magento/framework/App/ErrorHandler.php:61
image
wissam-khalili commented 2 years ago

Hi @hanhpv ,

Thank you for reaching out.

  1. Was it working earlier and something has changed?
  2. If you disable the Cloudinary extension, are you still getting the same message?
  3. Can you please share a staging URL for the Cloudinary implementation?
  4. Do you have any other modules that may affect images or the page with issues?
  5. Do you have any additional 3rd party modules installed on your environment? If so, could you please provide a list of these plugins?

Looking forward to your reply.

You're welcome to raise a support ticket with us directly if you'd like us to use examples from your own cloud. To do so, you can fill out the form at this link.

Thanks, Wissam

hanhpv commented 2 years ago

Hi @wissam-khalili

Thanks for looking into this.

  1. Yes it is working now on live site with Magento 2.4.2, PHP 7.4 (we're upgrading to Magento 2.4.4)
  2. If I set Enable Cloudinary to No in store configuration, it is working fine.
  3. Unfortunately, it is a customer site so I can't share.
  4. No, mostly a default Magento site, with Cloudinary installed.
  5. I do have some other 3rd party modules, but I don't think they relate. As stated in (1), it is still working fine on live site.

Maybe a PHP 8 compatible issue? I notice the latest version of the module requires "cloudinary/cloudinary_php": "^1.20.0", but cloudinary/cloudinary_php is now 2.7.1

wissam-khalili commented 2 years ago

Hi @hanhpv ,

Thank you for the additional information. Indeed, we do not support PHP 8.1 which is now required for Magento 2.4.4. We are planning to do so, but I don't have an ETA for that yet.

Regards, Wissam

momoip commented 2 years ago

Hi @hanhpv, We are working on supporting PHP 8 and will let you know once we have an ETA.

Regards, Mo

wissam-khalili commented 2 years ago

Hi @hanhpv,

I would like to update you that we have released a new Magento version 1.19 which supports PHP 8.

Regards, Wissam