PHP errors found

  • Resolved RakeshTailor

    (@rakeshtailor)


    5 years, 6 months ago

    Found these during a routine PHP compatibility scan.

    FOUND 4 ERRORS AFFECTING 3 LINES
    ——————————————————————————————————————————————————-
    200 | ERROR | The constant “MCRYPT_MODE_CBC” is deprecated since PHP 7.1 and removed since PHP 7.2
    201 | ERROR | The constant “MCRYPT_RIJNDAEL_128” is deprecated since PHP 7.1 and removed since PHP 7.2
    203 | ERROR | Function mcrypt_encrypt() is deprecated since PHP 7.1 and removed since PHP 7.2; Use OpenSSL instead
    203 | ERROR | Extension ‘mcrypt’ is deprecated since PHP 7.1 and removed since PHP 7.2; Use openssl (preferred) or pecl/mcrypt once available instead

Viewing 1 replies (of 1 total)
  • Plugin Author chriscct7

    (@chriscct7)

    5 years, 6 months ago

    Hi there,
    This is a good example of why PHP compatibility scanners packaged as a WordPress plugin are in almost all cases worthless.

    None of those are actually included in our plugin’s non-vendor codebase, and the uses of them, by Google in the official Google API, are all protected by version checks for PHP (since they are fallback methods), something a PHP Compatibility plugin would be unable to detect, and they are all located in files that are not ever loaded much less used by our plugin, yet again something a PHP Compatibility plugin would be unable to detect.

    So none of these errors are either accurate or affect any of our users on any PHP version.

    Consider that GADWP has over a million active installs. Hundreds of thousands of these installs are on PHP versions that according to the PHP Compatibility plugin would be completely broken.

    Obviously, this is not the case. GADWP is, and has been, PHP 7.2 compatible for some time now.

    We ourselves develop using PHP 7.2 (as well as test code in previous releases), and through contributions to PHP core, are also testing on the beta releases of the upcoming PHP 7.3.

    -Chris

Viewing 1 replies (of 1 total)
  • The topic ‘PHP errors found’ is closed to new replies.