-
Notifications
You must be signed in to change notification settings - Fork 38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Error S2member Framework #243
Comments
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. |
Thank you. I'm new to all of this. I looked for it but did not find it.
So, PHP v 7.3 should not cause an issue with this plugin?
Thank you,
Michael Lewis
…On Mon, Jul 1, 2019 at 9:13 AM Juliette ***@***.***> wrote:
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.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#243?email_source=notifications&email_token=AMP4VPPKXECAFFNRJIYZHFTP5IGJDA5CNFSM4H4SVEYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODY6IG6Q#issuecomment-507282298>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AMP4VPPZHZWFSWGVH6NPZ5TP5IGJDANCNFSM4H4SVEYA>
.
|
@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. |
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
The text was updated successfully, but these errors were encountered: