Skip to content
This repository has been archived by the owner on Feb 23, 2022. It is now read-only.

A non-numeric value encountered in #13

Open
bobsburgers opened this issue Nov 12, 2018 · 5 comments
Open

A non-numeric value encountered in #13

bobsburgers opened this issue Nov 12, 2018 · 5 comments

Comments

@bobsburgers
Copy link

Best I can tell this is an issue with php 7.1. Is there a fix in the works? Should I make one?

PHP message: PHP Warning: A non-numeric value encountered in /var/www/site/www/wp-content/plugins/btcpay-for-woocommerce/class-wc-gateway-bitpay.php on line 885" while reading response header from upstream, client: 141.101.77.153, server: site.net, request: "POST /wc-api/WC_Gateway_Bitpay/ HTTP/1.1", upstream: "fastcgi://unix:/run/phpfpm-site.sock:", host: "site.net"
2018/11/12 16:16:34 [error] 337#337: *83248 FastCGI sent in stderr: "PHP message: PHP Warning: A non-numeric value encountered in /var/www/site/www/wp-content/plugins/btcpay-for-woocommerce/class-wc-gateway-bitpay.php on line 660

@pavlenex
Copy link
Contributor

pavlenex commented Nov 12, 2018

Haven't noticed (running php 7.3 though) this but pr is welcome. Maybe @Kukks knows more.
We're actually looking at completely re-ramping the plugin if you have any experience maybe you can help Kukks out.

@bobsburgers
Copy link
Author

Haven't noticed (running php 7.3 though) this but pr is welcome. Maybe @Kukks knows more.
We're actually looking at completely re-ramping the plugin if you have any experience maybe you can help Kukks out.

Yes, I was looking at some of the other issues. If there are some specific tasks I would be happy to help. Ill keep watch for some I can do.

@pavlenex
Copy link
Contributor

For now, we really want to add our plugin to WordPress repository. There are some things that need to be cleaned. I'm sure @Kukks can tell you more.
For now check this issue and this one for specific things we have to fix in order to be accepted by the WP.

@Kukks
Copy link
Member

Kukks commented Nov 13, 2018

Heya @bobsburgers
I'm almost done with getting the plugin ready for wordpress plugin repo. Once that's done I would love to get rid of some of the obscure tooling in the project( no joke, we have to run grunt to move files around that requires ancient npm versions that needs to run in a docker that needs to be run through a powershell file lol) I am fairly confident we can clear most of the tooling without issue. :)

@NicolasDorier
Copy link
Member

yeah took me quite a lot of time to be able to "compile" the project reliably. Needed to isolate the mess inside a dockerfile.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants