wpengine / phpcompat

WordPress Plugin: PHP Compatibility Checker

Home Page:https://wordpress.org/plugins/php-compatibility-checker/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Error S2member Framework

Medicareschool opened this issue · comments

Currently running S2member v. 190617. It seems to have no issues, but PHP 7.2 returned errors. False?
Report:
FILE: XXXXXXXXX/public_html/wp-content/plugins/s2member/src/vendor/paragonie/random_compat/lib/random_bytes_mcrypt.php

FOUND 3 ERRORS AFFECTING 1 LINE

58 | ERROR | Function mcrypt_create_iv() is deprecated since PHP 7.1 and removed since PHP 7.2; Use random_bytes() or OpenSSL instead
58 | ERROR | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
58 | ERROR | The constant "MCRYPT_DEV_URANDOM" is deprecated since PHP 7.1 and removed since PHP 7.2

XXXXXXXXXXX/html/wp-content/plugins/s2member/src/vendor/paragonie/random_compat/lib/byte_safe_strings.php

FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES

32 | WARNING | INI directive 'mbstring.func_overload' is deprecated since PHP 7.2
86 | WARNING | INI directive 'mbstring.func_overload' is deprecated since PHP 7.2

This has already been fixed in the latest version of PHPCompatibilityWP. This plugin just needs to upgrade to use a more recent version of the underlying technology.

@Medicareschool If those are the only issues reported, you should be fine.

@Medicareschool I was posting that a bit too fast - sorry.

As the underlying tech is so out of date, a lot of PHP 7.3 issues aren't scanned for with this plugin (yet), so there's no telling based on the above results.

If you want to get a more accurate scan, you can use the underlying technology directly to scan your installation with the up-to-date version: https://github.com/PHPCompatibility/PHPCompatibilityWP

Latest PHP Compatibility Checker also reported the mcrypt errors for PHP 7.3, but according to the plugin author those are only loaded for older PHP versions. So, false positive it is.