Support » Plugin: PHP Compatibility Checker » FALSE POSITIVE: Showing WP Super Cache as not PHP 7 Compatible

  • Blackhawk Cybersecurity

    (@blackhawkcybersec)


    Name: WP Super Cache

    FILE: /XXXXXXXX/wp-content/plugins/wp-super-cache/wp-cache.php
    ——————————————————————————————————
    FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 2 LINES
    ——————————————————————————————————
    202 | WARNING | INI directive ‘safe_mode’ is deprecated from PHP 5.3 and forbidden from PHP 5.4.
    202 | WARNING | INI directive ‘safe_mode’ is deprecated from PHP 5.3 and forbidden from PHP 5.4.
    207 | WARNING | INI directive ‘safe_mode_gid’ is deprecated from PHP 5.3 and forbidden from PHP 5.4.
    ——————————————————————————————————

    This is one of the top plugins for WordPress (and most downloaded). It’s definitely PHP 7 compatible. We run it on a ton of PHP 7 sites. Please fix your detection and/or update your whitelist!!

Viewing 2 replies - 1 through 2 (of 2 total)
  • Plugin Author Jason Stallings

    (@octalmage)

    Hi @blackhawkcybersec,

    These are warnings and they won’t affect compatibility. Were there any errors?

    Thanks!

    Blackhawk Cybersecurity

    (@blackhawkcybersec)

    No but it still counts as a false positive because it doesn’t need to be throwing these at all.

    Most of the times we’re seeing your plugin mention safe mode, it’s because a plugin is doing some kind of compatibility check, not enabling it.

    Why have warnings in the results at all? If your plugin is meant to test for compatibility, then this shouldn’t even be in there. The results don’t have enough info to be helpful for developers, but it has too much info for non-developers. Just kill the warnings if there are no errors.

    Developers aren’t going to use this method, they’ll use something more in-depth like WP_DEBUG with logging. So there’s no need to have anything that’s not going to be a show-stopper.

    Don’t get me wrong, as a developer, I want every little notice and warning, but this isn’t for developers.

Viewing 2 replies - 1 through 2 (of 2 total)
  • You must be logged in to reply to this topic.