kjandrews

Thanks very much, i’ll try to test this on one of the sites tomorrow.

Going to make an educated guess this particular site was running on PHP 7.2

Theme is listed as PHP Version:?5.6 or higher here https://www.ads-software.com/themes/zakra/

Trailing comma RFC was voted in for PHP 7.3

https://wiki.php.net/rfc/trailing-comma-function-calls

and implemented:

Implemented in PHP 7.3 via?https://github.com/php/php-src/commit/b591c329ee3129adbdc35141bb1542d119f7a2a1.

If your theme is listed as PHP 5.6 compatible then you’ll need to remove the trailing comma.

Website has been patched for now as we can’t have the sites offline.

For valid compatible PHP code, this line:

delete_post_meta( $post_id, ‘_zakra_product_gallery_image_id_’ . $id, );

needs changing to

delete_post_meta( $post_id, ‘_zakra_product_gallery_image_id_’ . $id );

4th from the end comma needs removing.

Ahhh thanks very much, can’t believe I missed those ending brackets!

All working ??

We’ve tried uploading a previous version from a backup and still have the issue.

I’ve tried to diagnose by uploading each changed file between both versions back to the older one to try and highlight which change causes the issue and none of the original files make it work again.

However uninstalling the plugin, uploading the backup older version and reactivating has fixed the issue.

Uninstalling then installing from the newest version of the plugin breaks again.

Hi @mihaiimagely

Thanks very much for the fix, can confirm this is now working for me!