PHP 7.0 compatibility

  • Resolved eehmke

    (@eehmke)


    8 years, 1 month ago

    I upgraded my site to PHP 7.0 and installed the PHP compatibility checker plugin. It reports these errors and warnings:

    FILE: ../wp-content/plugins/order-tracking/html/AdminFooter.php
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
 1 | WARNING | No PHP code was found in this file and short open tags are not allowed by this install of PHP. This file may be using short open tags but PHP does not allow them.

    Can be solved by inserting an empty <?php tag.

    FILE: ../wp-content/plugins/order-tracking/PHPExcel/Classes/PHPExcel/Calculation/Functions.php
FOUND 1 ERROR AFFECTING 1 LINE
 581 | ERROR | Using 'break' outside of a loop or switch structure is invalid and will throw a fatal error since PHP 7.0

    Can be solved by removing that break statement, that (after a return) makes no sense anyway.

    FILE: ../wp-content/plugins/order-tracking/PHPExcel/Classes/PHPExcel/Calculation/Database.php
FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
 67 | WARNING | Method name "PHPExcel_Calculation_Database::__fieldExtract" is discouraged; PHP has reserved all method names with a double underscore prefix for future use
 98 | WARNING | Method name "PHPExcel_Calculation_Database::__filter" is discouraged; PHP has reserved all method names with a double underscore prefix for future use

    Can be solved by removing the __ prefix from the function definitions and all references.

    FILE: ../wp-content/plugins/order-tracking/PHPExcel/Classes/PHPExcel/Shared/JAMA/examples/Stats.php
FOUND 0 ERRORS AND 5 WARNINGS AFFECTING 5 LINES
 1428 | WARNING | Method name "Base::__sumdiff" is discouraged; PHP has reserved all method names with a double underscore prefix for future use
 1460 | WARNING | Method name "Base::__calcVariance" is discouraged; PHP has reserved all method names with a double underscore prefix for future use
 1488 | WARNING | Method name "Base::__calcAbsoluteDeviation" is discouraged; PHP has reserved all method names with a double underscore prefix for future use
 1513 | WARNING | Method name "Base::__sumabsdev" is discouraged; PHP has reserved all method names with a double underscore prefix for future use
 1545 | WARNING | Method name "Base::__format" is discouraged; PHP has reserved all method names with a double underscore prefix for future use

    can be solved by removing the __ prefixes.

    FILE: ../wp-content/plugins/order-tracking/PHPExcel/Classes/PHPExcel/Shared/PCLZip/pclzip.lib.php
FOUND 2 ERRORS AND 1 WARNING AFFECTING 3 LINES
  215 | WARNING | Use of deprecated PHP4 style class constructor is not supported since PHP 7.
 5347 | ERROR   | Function set_magic_quotes_runtime() is deprecated since PHP 5.3 and removed since PHP 7.0
 5378 | ERROR   | Function set_magic_quotes_runtime() is deprecated since PHP 5.3 and removed since PHP 7.0

    Replaced constructor by __construct function; removed deprecated functions.

    FILE: ../wp-content/plugins/order-tracking/PHPExcel/Classes/PHPExcel/CachedObjectStorage/SQLite.php
FOUND 9 ERRORS AFFECTING 9 LINES
  63 | ERROR | Extension 'sqlite' is removed since PHP 5.4
  64 | ERROR | Extension 'sqlite' is removed since PHP 5.4
 108 | ERROR | Extension 'sqlite' is removed since PHP 5.4
 142 | ERROR | Extension 'sqlite' is removed since PHP 5.4
 166 | ERROR | Extension 'sqlite' is removed since PHP 5.4
 211 | ERROR | Extension 'sqlite' is removed since PHP 5.4
 236 | ERROR | Extension 'sqlite' is removed since PHP 5.4
 274 | ERROR | Extension 'sqlite' is removed since PHP 5.4
 276 | ERROR | Extension 'sqlite' is removed since PHP 5.4

    For this problem I don’t have a simple solution. The plugin seems to work fine, however.

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

    (@etoilewebdesign)

    Hi eehmke,

    All of those, other than the first one, our related to PHPExcel. We just use the standard version of PHPExcel in our plugin and don’t make any changes to it. The best place to report the notices for those would be directly to PHPExcel (e.g. via their github page here: https://github.com/PHPOffice/PHPExcel).

    We’ll take a look at that first one in the AdminFooter.php file.

    Thank you

    Thread Starter eehmke

    (@eehmke)

    In the actual version 1.8.1 of PHPExcel most of these errors already seem to have been solved. I would suggest to migrate to the newest version.

    The PHPExcel in the plugin seems to be based on version 1.8.0. I replaced the PHPExcel subdirectory in the plugin by version 1.8.1 of the PHPExcel release. All errors and warnings disappeared, with two exceptions:

    • the PHPExcel/Calculation error is still reported. But even this problem is solved in the latest update (not contained in release 1.8.1)
    • the sqlite problem is still not solved.
    • This reply was modified 8 years, 1 month ago by eehmke.
    Plugin Contributor etoilewebdesign

    (@etoilewebdesign)

    Hi,

    Thanks for letting us know about the latest version of PHPExcel. We’ll look to incorporate the newest version in an upcoming update.

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