WordPress.org

Resolved TomCobbley
(@tomcobbley)

2 years, 5 months ago

I upgraded to version 2.6.10 and it broke my site. This is from the error log.

[Tue Feb 22 11:01:19.550738 2022] [proxy_fcgi:error] [pid 15506:tid 140222410798848] [client xxx.xxx.xxx.xxx:33228] AH01071: Got error 'PHP message: PHP Parse error: syntax error, unexpected ')' in /var/www/mysite.com/html/wp-content/plugins/facebook-for-woocommerce/facebook-commerce.php on line 709\nPHP message: PHP Fatal error: Exception thrown without a stack frame in Unknown on line 0\n'

I rolled back to 2.6.9 and it’s OK again.

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

wojsmol
(@wojsmol)

2 years, 5 months ago
Hi
Issue is caused by additional , on line 709 in this file in version 2.6.10 of the plugin. If you wont to update to0 this version then here is a fix
replace
```
 $block_sync = apply_filters( 
 	'facebook_for_woocommerce_block_full_batch_api_sync', 
 	false,
 ); 
```
with
```
 $block_sync = apply_filters( 
 	'facebook_for_woocommerce_block_full_batch_api_sync', 
 	false
 ); 
```
- This reply was modified 2 years, 5 months ago by wojsmol.
Thread Starter TomCobbley
(@tomcobbley)

2 years, 5 months ago

Looks like the last changes were done by Automattic. You’d have thought they would have tested it first.

yourbudweiser
(@yourbudweiser)

2 years, 5 months ago

Thanks @wojsmol, that comma crashed my site. Removing it fixed it.

Rashed Ripon (a11n)
(@rur165)

2 years, 5 months ago

Hi all,

Thanks a lot for reporting also suggesting a temporary solution.

I am unable to reproduce the problem on my site. I am using the latest version of WooCommerce 6.2.0 and WordPress 5.9.1. Please make sure you are also using the latest version of these.

I can see that it has been already reported here: https://github.com/woocommerce/facebook-for-woocommerce/issues/2160

I have mentioned this thread there but feel free to join the discussion there or subscribe to the issue.

Thank you.

Plugin Support Madison a11n
(@madisoncm)

2 years, 5 months ago

Hi there!
Thank you for reporting this @tomcobbley and for providing a fix @wojsmol. There have been a few reports of the same issue filed on the Facebook for WooCommerce GitHub page. You can follow these to see the progress of the issue. In this report the solution provided above was also mentioned: https://github.com/woocommerce/facebook-for-woocommerce/issues/2158
Thanks again for being a part of the open-source community!
Plugin Support Chris M. – a11n
(@csmcneill)

2 years, 5 months ago
Howdy @tomcobbley,

I’m not immediately encountering this error on my own test site running just WooCommerce 6.2.0 and Facebook for WooCommerce 2.6.10.

@wojsmol has logged this error on our public GitHub repository, which is where we identify and fix issues. It appears as though some other merchants (like @yourbudweiser) have had success modifying the facebook-commerce.php file to remove the , in line 709, so this is an option worth considering. As you mentioned, reverting to version 2.6.9 is another option for resolving this issue. That version can be accessed here.

If you’d like to follow along with any progress made on this issue, I would recommend subscribing to the issue on GitHub. It can be found at the following link:

https://github.com/woocommerce/facebook-for-woocommerce/issues/2157

I hope this helps! Let us know if you have any further questions!
- This reply was modified 2 years, 5 months ago by Chris M. - a11n.
wojsmol
(@wojsmol)

2 years, 5 months ago
Hi all
For the record for me issue occurred on following setup:
WordPress 5.9.0 – latest on the time of plugin update – also occurs on WP 5.9.1
WooCommerce: 6.2.0
Facebook for WooCommerce:2.6.10
PHP Version: 7.1.9
- This reply was modified 2 years, 5 months ago by wojsmol.
- This reply was modified 2 years, 5 months ago by wojsmol.
Nagdy
(@nagdy)

2 years, 5 months ago

Hi @wojsmol,

Thanks for sharing your setup information with us. It matches the setup information reported in the original post here https://github.com/woocommerce/facebook-for-woocommerce/issues/2157

If you’d like to follow along with any progress made on this issue, I would recommend subscribing to that issue on GitHub.

wojsmol
(@wojsmol)

2 years, 5 months ago

Hi @nagdy
It’s matches because this is the same Wor5dPress installation. I created the linked issue on Github. We have additional reports in https://wordpress.org/support/topic/critical-bug-in-version-2-6-10/

Plugin Support Brad P. a11n
(@snaacks)

2 years, 5 months ago

Hi @wojsmol

Thank you for clarifying that it is quite literally the same install and that you created that issue, we appreciate you doing so!

We’ll look into any additional reports and continue to update as we find out more.

Dave Page
(@davecpage)

2 years, 5 months ago

Hi all,

I had this too and realised the extra comma is a PHP 7.3 supported thing, and I was on PHP 7.2. So I upgraded the server to PHP 7.4. This solved my issue.

The plugin does state that it is supported on PHP 7.0 or higher, so either that needs updating or the comma needs removing.

Plugin Support B C. a11n
(@battouly)

2 years, 5 months ago

Thanks for sharing what worked for you @davecpage!