dorzki / slack-notifications

Get WordPress notifications to your team's Slack channel.
https://wordpress.org/plugins/dorzki-notifications-to-slack/
GNU General Public License v3.0
41 stars 27 forks source link

Fatal Error in version 2.0 #15

Closed mahype closed 6 years ago

mahype commented 6 years ago

Here it is!

Fatal error: Can't use method return value in write context in /wp-content/plugins/dorzki-notifications-to-slack/core/notifications/woocommerce.php on line 355

mahype commented 6 years ago

Can you please add the Pull Request #16, because the acutal version is occuring massive errors to existing sites which updating to it? :)

DorZuberi commented 6 years ago

Hi @mahype , What is your:

WordPress version WooCommerce version PHP version

mahype commented 6 years ago

WordPress Environment

WC Version: 3.3.1 Log Directory Writable: ✔ WP Version: 4.9.4 WP Multisite: ✔ WP Memory Limit: 256 MB WP Debug Mode: – WP Cron: ✔ Language: de_DE

Server Environment

Server Info: Apache/2.4.27 PHP Version: 7.0.20 PHP Post Max Size: 8 MB PHP Time Limit: 0 PHP Max Input Vars: 1000 cURL Version: 7.55.1 OpenSSL/1.0.2l

SUHOSIN Installed: – MySQL Version: 5.6.19 Max Upload Size: 8 MB Default Timezone is UTC: ✔ fsockopen/cURL: ✔ SoapClient: ✔ DOMDocument: ✔ GZip: ✔ Multibyte String: ✔ Remote Post: ✔ Remote Get: ✔

DorZuberi commented 6 years ago

Thanks, Looking into it.

DorZuberi commented 6 years ago

Hi @mahype, I've deployed an update, would love if you can test it before i'm pushing it to WordPress repo.