Help to update your plugin to PHP7

  • Resolved Shane

    (@shanemarsh28)


    9 years, 9 months ago

    Hi Everyone,

    I am a Technical Manager and we use your plugin within our WordPress installation.

    PHP7 is currently being rolled out across many platforms and soon we will also be updating our systems to the new, lighter and faster version of PHP. However, our tests show that your plugin is currently incompatible with PHP7. So that we can help things along and to make the update easier, our analytics tools have identified which lines of code that need to be changed within your plugin that would allow it to run within this new environment.

    You will see that most of the updates are very minor and we would be forever thankful if you would be able to make these changes to your next release.

    The report is as follows:

    FILE: /plugins/nextgen-gallery/products/photocrati_nextgen/modules/ngglegacy/nggallery.php
--------------------------------------------------------------------------------------------------------------------------------
FOUND 4 ERRORS AFFECTING 4 LINES
--------------------------------------------------------------------------------------------------------------------------------
 185 | ERROR | INI directive "safe_mode" is deprecated from PHP 5.3 and forbidden from PHP 5.4.
 187 | ERROR | INI directive "safe_mode" is deprecated from PHP 5.3 and forbidden from PHP 5.4.
 188 | ERROR | INI directive "safe_mode" is deprecated from PHP 5.3 and forbidden from PHP 5.4.
 190 | ERROR | INI directive "safe_mode" is deprecated from PHP 5.3 and forbidden from PHP 5.4.
--------------------------------------------------------------------------------------------------------------------------------

FILE: /plugins/nextgen-gallery/products/photocrati_nextgen/modules/ngglegacy/admin/overview.php
-------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------------------------------
 28 | ERROR | INI directive "safe_mode" is deprecated from PHP 5.3 and forbidden from PHP 5.4.
-------------------------------------------------------------------------------------------------------------------------------------

FILE: /plugins/nextgen-gallery/products/photocrati_nextgen/modules/nextgen_data/package.module.nextgen_data.php
-----------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
-----------------------------------------------------------------------------------------------------------------------------------------------------
 1132 | ERROR | INI directive "safe_mode" is deprecated from PHP 5.3 and forbidden from PHP 5.4.
 3661 | ERROR | INI directive "safe_mode" is deprecated from PHP 5.3 and forbidden from PHP 5.4.
-----------------------------------------------------------------------------------------------------------------------------------------------------

    We would also like to take this opportunity to thank you for your ongoing updates and I hope that you look at the above report and implement it’s changes.

    https://wordpress.org/plugins/nextgen-gallery/

Viewing 3 replies - 1 through 3 (of 3 total)
  • Plugin Contributor photocrati

    (@photocrati)

    9 years, 9 months ago

    @shanemarsh28 – Thanks for bringing your concerns to our attention but in each of these cases we are only testing if safe_mode is set as we are still holding our backward compatibility to PHP 5.2 and although we are looking very seriously at raising the minimum PHP requirement it will only be by baby steps and not likely in the immediate future (but soon all the same).

    Currently we use ini_get( 'safe_mode' ) in our conditionals (as reflected in each of your reported error lines) … perhaps your “sniff” is being too strict in its criteria? Can you provide any additional details about the “tests” you are using?

    Thanks, again!

    – Cais.

    Thread Starter Shane

    (@shanemarsh28)

    9 years, 9 months ago

    Hi Cais,

    Thank you so much for getting back to me so quickly. I am delighted you can confirm all will be OK within PHP 7.

    Yes – the sniffer I am using was develeloped by the team at WP Engine and I agree, it’s exceptionally sensitive and also quite blind in how it works.

    The tool completes a recersive search on the filesystem but it cannot determin which files are called and when. Or if in the case of safemode, is just being queried rather than changed. I do however prefer this tool to be sensitive because if I get the upgrade wrong and something isn’t quite right within our very lage multisite installation, it could have huge impacts to our business. Check, check and double check is how we are proceeding lol 🙂

    The tool can be downloaded from here:
    https://wordpress.org/plugins/php-compatibility-checker/

    Kind regards,
    Shane

    Plugin Contributor photocrati

    (@photocrati)

    9 years, 9 months ago

    @shanemarsh28 – Thanks for the additional details, we are reviewing our code in general for eventual increases in the minimum WordPress requirements. You report was very helpful in that regard.

    – Cais.

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

The topic ‘Help to update your plugin to PHP7’ is closed to new replies.

Tags