PHP7 Compatibility

  • Resolved ashpriom

    (@ashpriom)


    8 years ago

    Hi,

    I want to know if this plugin is planned for an upgrade compatible with PHP7?

    I ran a test with WP Engine plugin https://wpengine.com/blog/php-7-compatibility-checker-plugin/ and it shows 9 warnings:

    FILE: /nas/content/staging/synstudio/wp-content/plugins/popup-maker/includes/admin/upgrades/class-pum-admin-upgrade-routine-5.php
    ———————————————————————————————————————————
    FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
    ———————————————————————————————————————————
    43 | WARNING | INI directive ‘safe_mode’ is deprecated since PHP 5.3 and removed since PHP 5.4.
    ———————————————————————————————————————————

    FILE: /nas/content/staging/synstudio/wp-content/plugins/popup-maker/includes/admin/upgrades/class-pum-admin-upgrade-routine-4.php
    ———————————————————————————————————————————
    FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
    ———————————————————————————————————————————
    48 | WARNING | INI directive ‘safe_mode’ is deprecated since PHP 5.3 and removed since PHP 5.4.
    ———————————————————————————————————————————

    FILE: /nas/content/staging/synstudio/wp-content/plugins/popup-maker/includes/admin/upgrades/class-pum-admin-upgrade-routine-2.php
    ———————————————————————————————————————————
    FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
    ———————————————————————————————————————————
    37 | WARNING | INI directive ‘safe_mode’ is deprecated since PHP 5.3 and removed since PHP 5.4.
    ———————————————————————————————————————————

    FILE: /nas/content/staging/synstudio/wp-content/plugins/popup-maker/includes/admin/upgrades/class-pum-admin-upgrade-routine-6.php
    ———————————————————————————————————————————
    FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
    ———————————————————————————————————————————
    55 | WARNING | INI directive ‘safe_mode’ is deprecated since PHP 5.3 and removed since PHP 5.4.
    ———————————————————————————————————————————

    FILE: /nas/content/staging/synstudio/wp-content/plugins/popup-maker/includes/admin/upgrades/class-pum-admin-upgrade-routine-3.php
    ———————————————————————————————————————————
    FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
    ———————————————————————————————————————————
    48 | WARNING | INI directive ‘safe_mode’ is deprecated since PHP 5.3 and removed since PHP 5.4.
    ———————————————————————————————————————————

    FILE: /nas/content/staging/synstudio/wp-content/plugins/popup-maker/includes/admin/tools/tools-page.php
    ——————————————————————————————————-
    FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
    ——————————————————————————————————-
    312 | WARNING | INI directive ‘safe_mode’ is deprecated since PHP 5.3 and removed since PHP 5.4.
    ——————————————————————————————————-

    FILE: /nas/content/staging/synstudio/wp-content/plugins/popup-maker/includes/class-pum-activator.php
    —————————————————————————————————-
    FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
    —————————————————————————————————-
    51 | WARNING | INI directive ‘safe_mode’ is deprecated since PHP 5.3 and removed since PHP 5.4.
    —————————————————————————————————-

    FILE: /nas/content/staging/synstudio/wp-content/plugins/popup-maker/includes/libs/freemius/includes/fs-essential-functions.php
    —————————————————————————————————————————————
    FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
    —————————————————————————————————————————————
    147 | WARNING | Function name “__fs” is discouraged; PHP has reserved all method names with a double underscore prefix for future use
    —————————————————————————————————————————————

    FILE: /nas/content/staging/synstudio/wp-content/plugins/popup-maker/includes/libs/browser.php
    ———————————————————————————————-
    FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
    ———————————————————————————————-
    225 | WARNING | Use of deprecated PHP4 style class constructor is not supported since PHP 7.
    ———————————————————————————————-

Viewing 3 replies - 1 through 3 (of 3 total)
  • waltmesser

    (@waltmesser)

    @ashpriom

    As far as I’ve been told, the plugin is compatible with PHP7. Difference here being warnings versus errors.

    At any rate, thanks for bringing this to our attention, and I’ll make sure our Lead Dev @danieliser sees this, if he already hasn’t.

    Plugin Author Daniel Iser

    (@danieliser)

    @ashpriom – As fellow WP Engine users ourselves I can tell you it works fine on PHP 7. In fact our site is PHP 7 via WP Engine, my dev environment is PHP 7 and so on.

    The notices are non errors, and in most cases will never effect you. To clarify most are in upgrade routines for really old versions, simply there for backward compatibility, others are in the tools page debug report and won’t trigger errors.

    All in all the report just tells you there are old functions included, but those functions still work in PHP 7 without triggering errors in the case they ever did get run, which they won’t.

    PS removing some of those lines from our code would make users with older versions of PHP run into major issues if they ever had to run those upgrades without it. Since we have so many users with range of PHP versions we can’t do that either, but they won’t fail on PHP 7. Simply there for backward compatibility.

    PS If you run into any issues, the WP Engine support team knows me and our plugin pretty well, but from your report and the one we ran last year before moving to PHP 7 there are no problems in the usage and no log messages, our production site logs stay clean for the most part.

    benchmarkgraphics

    (@benchmarkgraphics)

    Hello, I am getting a similar incompatibility message, with a site hosted by SiteGround and using their PHP Config plugin.
    Thanks

Viewing 3 replies - 1 through 3 (of 3 total)
  • The topic ‘PHP7 Compatibility’ is closed to new replies.

Tags