Make WordPress Core


Ignore:
Timestamp:
09/25/2019 01:46:55 PM (4 years ago)
Author:
desrosj
Message:

Build/Test Tools: Introduce automated PHP compatibility checking.

This change introduces a new Composer script, compat that will scan the codebase for (detectable) potential PHP compatibility issues using the PHP_CodeSniffer and a custom ruleset based off of the PHPCompayibilityWP ruleset (phpcompat.xml.dist).

The command will be run as a separate job within each Travis build. While many compatibility issues and false positives have already been corrected in this commit and other Trac tickets, there are still some remaining. For that reason, the job is allowed to fail while the remainder of the potential compatibility issues are investigated and addressed. After those are resolved, the job should be set as required to pass to help prevent new compatibility issues from being introduced.

Props desrosj, jrf, all PHPCompatibilityWP and PHPCompatibility contributors.
Fixes #46152.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-admin/includes/class-wp-site-health.php

    r46269 r46290  
    159159            $mysql_server_type = mysqli_get_server_info( $wpdb->dbh );
    160160        } else {
    161             // phpcs:ignore WordPress.DB.RestrictedFunctions.mysql_mysql_get_server_info
     161            // phpcs:ignore WordPress.DB.RestrictedFunctions.mysql_mysql_get_server_info,PHPCompatibility.Extensions.RemovedExtensions.mysql_DeprecatedRemoved
    162162            $mysql_server_type = mysql_get_server_info( $wpdb->dbh );
    163163        }
     
    11721172            $mysql_client_version = mysqli_get_client_info();
    11731173        } else {
    1174             // phpcs:ignore WordPress.DB.RestrictedFunctions.mysql_mysql_get_client_info
     1174            // phpcs:ignore WordPress.DB.RestrictedFunctions.mysql_mysql_get_client_info,PHPCompatibility.Extensions.RemovedExtensions.mysql_DeprecatedRemoved
    11751175            $mysql_client_version = mysql_get_client_info();
    11761176        }
Note: See TracChangeset for help on using the changeset viewer.