Plans for PHP 7 compatiblity?

Hi @firemyst,

Thank you so mush for your info about PHP 7.

1. About $HTTP_RAW_POST_DATA
I usually test my plugin with PHP 5.6.10 (sometimes, PHP 5.2.17 :)) on MAMP Pro and notice about this warning. But in xmlrpc.php of WP core, $HTTP_RAW_POST_DATA is assigned as a global value using php://input. So I think it doesn’t matter with PHP 7. How do you think?

2. About hexadecimal numeric strings
I didn’t know about this error until now. Then I should use

filter_var('0x10000', FILTER_VALIDATE_INT, FILTER_FLAG_ALLOW_HEX);

instead of 0x10000. Do you think it’s alright?

c.f. https://php.net/manual/en/filter.filters.validate.php

• This reply was modified 8 years, 1 month ago by tokkonopapa.
• This reply was modified 8 years, 1 month ago by tokkonopapa.

Thanks for your response. And I have to say I absolutely LOOOOOOOOOOOOOOOOOOOOOOOVE your plugin. ??

As for those issues, I’m not a PHP Developer, so I don’t know if what you suggested will work, or is appropriate, or is even really an issue. That’s just feedback from another PHP plugin.

I just wanted to bring it to your attention given that PHP 5.6 is coming to end-of-life and sooner (hopefully) or later everything is going to have to migrate towards 7. We’re being a bit more cautious and not moving our Production WordPress sites until most plugins pass php7 compatibility tests.

Hopefully another developer within these forums will be able to answer your questions?

Wow, thanks for your commenting!

OK, I just started to make my plugin compatible with PHP 7 and I keep this thread open until I finish it.

Thank you for the heads up.

FYI:

I just updated to IP Geo Block version 2.2.9 (love the “must use” timing feature!), reran the PHP 7 compatibility checker, and it comes back with the following:

FILE: /wp-content/plugins/ip-geo-block/classes/class-ip-geo-block-logs.php
--------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
--------------------------------
 344 | ERROR | Global variable '$HTTP_RAW_POST_DATA' is deprecated since PHP 5.6 and removed since PHP 7.0 - use php://input instead.
 348 | ERROR | Global variable '$HTTP_RAW_POST_DATA' is deprecated since PHP 5.6 and removed since PHP 7.0 - use php://input instead.
--------------------------------

FILE: /wp-content/plugins/ip-geo-block/wp-content/ip-geo-api/ip2location/bcmath.php
-------------------
FOUND 2 ERRORS AFFECTING 2 LINES
-------------------
 156 | ERROR | Using 'break' outside of a loop or switch structure is invalid and will throw a fatal error since PHP 7.0
 157 | ERROR | Using 'break' outside of a loop or switch structure is invalid and will throw a fatal error since PHP 7.0
-------------------

Hi FireMyst,

Thank you for your reporting again. I think both will not be a problem. Please refer to the discussion at #11 on Github. I’ve tested this release with PHP7 on my local environment. But I’ll improve the 2nd one in the future.

Thanks again!

Although it was not actually error, but improved some code to restrain this issue in 2.2.9.1.

• This reply was modified 8 years ago by tokkonopapa.