Failing Some PHP Compatibility Checks

  • Resolved richardmichael

    (@richardmichael)


    4 years, 8 months ago

    Hello,
    We want to upgrade our website to use PHP 7.2 or higher. We used a plugin to scan for PHP 7.2 compatibility and only 2 out of 31 plugins have warnings. This plugin is one of the two.

    Could you advise on if you are aware of and/or keeping up with PHP version changes? The plugin alleges the following. I have yet to confirm by referencing the PHP changelog directly.

    FILE: /home/r/public_html/wp-content/plugins/woo-discount-rules/v2/App/Helpers/Input.php
-------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 3 ERRORS AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------------------------------------------------
 841 | ERROR | Extension 'mcrypt' is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead
 841 | ERROR | Function mcrypt_create_iv() is deprecated since PHP 7.1 and removed since PHP 7.2; Use random_bytes() or OpenSSL instead
 841 | ERROR | The constant "MCRYPT_DEV_URANDOM" is deprecated since PHP 7.1 and removed since PHP 7.2
-------------------------------------------------------------------------------------------------------------------------------------------------------

FILE: /home/r/public_html/wp-content/plugins/woo-discount-rules/v1/vendor/flycartinc/inputhelper/src/StringHelper.php
-------------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 0 ERRORS AND 6 WARNINGS AFFECTING 6 LINES
-------------------------------------------------------------------------------------------------------------------------------------------------------------
 15 | WARNING | INI directive 'mbstring.internal_encoding' is deprecated since PHP 5.6
 16 | WARNING | INI directive 'mbstring.http_input' is deprecated since PHP 5.6
 17 | WARNING | INI directive 'mbstring.http_output' is deprecated since PHP 5.6
 23 | WARNING | All previously accepted values for the $type parameter of iconv_set_encoding() have been deprecated since PHP 5.6. Found 'internal_encoding'
 24 | WARNING | All previously accepted values for the $type parameter of iconv_set_encoding() have been deprecated since PHP 5.6. Found 'input_encoding'
 25 | WARNING | All previously accepted values for the $type parameter of iconv_set_encoding() have been deprecated since PHP 5.6. Found 'output_encoding'
-------------------------------------------------------------------------------------------------------------------------------------------------------------
Viewing 4 replies - 1 through 4 (of 4 total)
  • Plugin Author flycart

    (@flycart)

    Hi

    That was an Input sanitisation library which supports both PHP 5.x versions and the 7.x versions. That is, it has a if else conditional statement to run the sanitisation based on the PHP versions.

    So please do not worry. Our plugin is fully compatible with the 7.x versions.
    If you are in doubt, you can check the file in question and you can see the version checks

    Thanks

    Thread Starter richardmichael

    (@richardmichael)

    Thank you for the quick information. Great, I can pass that along to the plugin authors and they will add the information as a footnote. Or in some cases make the test consider that completely.

    Plugin Author flycart

    (@flycart)

    Thank you. In the upcoming versions, we will remove this compatibility and bump up of required PHP version to 5.6 so that this false positives were not reported.

    Thanks

    Plugin Author flycart

    (@flycart)

    Hi

    We are marking the thread as closed as this has been resolved in the latest versions.
    Please feel free to open a new ticket at our support page if you have any further questions.

    Thanks

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘Failing Some PHP Compatibility Checks’ is closed to new replies.