WordPress.org
Get WordPress

Make WordPress Core

Changeset 46268


Ignore:
Timestamp:
09/23/2019 07:49:06 PM (7 years ago)
Author:
SergeyBiryukov
Message:

Site Health: Update PHP extension tests to reflect the current state of the Hosting Teams handbook recommendations.

Props Clorith, bronsonquick, jrf, johnbillion, galbaras, mikeschroder, jorbin.
Fixes #47454.

File:
1 edited

Legend:

Unmodified
Added
Removed

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

    r46267 r46268  
    728728         *
    729729         * @since 5.2.0
     730         * @since 5.3.0 The `$constant` and `$class` parameters were added.
    730731         *
    731732         * @param string $extension Optional. The extension name to test. Default null.
    732733         * @param string $function  Optional. The function name to test. Default null.
     734         * @param string $constant  Optional. The constant name to test for. Default null.
     735         * @param string $class     Optional. The class name to test for. Default null.
    733736         *
    734737         * @return bool Whether or not the extension and function are available.
    735738         */
    736         private function test_php_extension_availability( $extension = null, $function = null ) {
     739        private function test_php_extension_availability( $extension = null, $function = null, $constant = null, $class = null ) {
    737740                // If no extension or function is passed, claim to fail testing, as we have nothing to test against.
    738                 if ( ! $extension && ! $function ) {
     741                if ( ! $extension && ! $function && ! $constant && ! $class ) {
    739742                        return false;
    740743                }
    … …  
    744747                }
    745748                if ( $function && ! function_exists( $function ) ) {
     749                        return false;
     750                }
     751                if ( $constant && ! defined( $constant ) ) {
     752                        return false;
     753                }
     754                if ( $class && ! class_exists( $class ) ) {
    746755                        return false;
    747756                }
    … …  
    789798
    790799                $modules = array(
    791                         'bcmath'    => array(
    792                                 'function' => 'bcadd',
    793                                 'required' => false,
    794                         ),
    795800                        'curl'      => array(
    796801                                'function' => 'curl_version',
    797802                                'required' => false,
    798803                        ),
     804                        'dom'       => array(
     805                                'class'    => 'DOMNode',
     806                                'required' => false,
     807                        ),
    799808                        'exif'      => array(
    800809                                'function' => 'exif_read_data',
    801810                                'required' => false,
    802811                        ),
     812                        'fileinfo'  => array(
     813                                'function' => 'finfo_file',
     814                                'required' => false,
     815                        ),
     816                        'hash'      => array(
     817                                'function' => 'hash',
     818                                'required' => false,
     819                        ),
     820                        'json'      => array(
     821                                'function' => 'json_last_error',
     822                                'required' => true,
     823                        ),
     824                        'mbstring'  => array(
     825                                'function' => 'mb_check_encoding',
     826                                'required' => false,
     827                        ),
     828                        'mysqli'    => array(
     829                                'function' => 'mysqli_connect',
     830                                'required' => false,
     831                        ),
     832                        'libsodium' => array(
     833                                'constant'            => 'SODIUM_LIBRARY_VERSION',
     834                                'required'            => false,
     835                                'php_bundled_version' => '7.2.0',
     836                        ),
     837                        'openssl'   => array(
     838                                'function' => 'openssl_encrypt',
     839                                'required' => false,
     840                        ),
     841                        'pcre'      => array(
     842                                'function' => 'preg_match',
     843                                'required' => false,
     844                        ),
     845                        'imagick'   => array(
     846                                'extension' => 'imagick',
     847                                'required'  => false,
     848                        ),
     849                        'mod_xml'   => array(
     850                                'extension' => 'libxml',
     851                                'required'  => false,
     852                        ),
     853                        'zip'       => array(
     854                                'class'    => 'ZipArchive',
     855                                'required' => false,
     856                        ),
    803857                        'filter'    => array(
    804858                                'function' => 'filter_list',
    805859                                'required' => false,
    806                         ),
    807                         'fileinfo'  => array(
    808                                 'function' => 'finfo_file',
    809                                 'required' => false,
    810                         ),
    811                         'mod_xml'   => array(
    812                                 'extension' => 'libxml',
    813                                 'required'  => false,
    814                         ),
    815                         'mysqli'    => array(
    816                                 'function' => 'mysqli_connect',
    817                                 'required' => false,
    818                         ),
    819                         'libsodium' => array(
    820                                 'function'            => 'sodium_compare',
    821                                 'required'            => false,
    822                                 'php_bundled_version' => '7.2.0',
    823                         ),
    824                         'openssl'   => array(
    825                                 'function' => 'openssl_encrypt',
    826                                 'required' => false,
    827                         ),
    828                         'pcre'      => array(
    829                                 'function' => 'preg_match',
    830                                 'required' => false,
    831                         ),
    832                         'imagick'   => array(
    833                                 'extension' => 'imagick',
    834                                 'required'  => false,
    835860                        ),
    836861                        'gd'        => array(
    … …  
    839864                                'fallback_for' => 'imagick',
    840865                        ),
     866                        'iconv'     => array(
     867                                'function' => 'iconv',
     868                                'required' => false,
     869                        ),
    841870                        'mcrypt'    => array(
    842871                                'extension'    => 'mcrypt',
    … …  
    844873                                'fallback_for' => 'libsodium',
    845874                        ),
     875                        'simplexml' => array(
     876                                'extension'    => 'simplexml',
     877                                'required'     => false,
     878                                'fallback_for' => 'mod_xml',
     879                        ),
    846880                        'xmlreader' => array(
    847881                                'extension'    => 'xmlreader',
    848882                                'required'     => false,
    849                                 'fallback_for' => 'xml',
     883                                'fallback_for' => 'mod_xml',
    850884                        ),
    851885                        'zlib'      => array(
    … …  
    860894                 *
    861895                 * @since 5.2.0
     896                 * @since 5.3.0 The `$constant` and `$class` parameters were added.
    862897                 *
    863898                 * @param array $modules {
    … …  
    870905                 *         string $function     Optional. A function name to test for the existence of.
    871906                 *         string $extension    Optional. An extension to check if is loaded in PHP.
     907                 *         string $constant     Optional. A constant name to check for to verify an extension exists.
     908                 *         string $class        Optional. A class name to check for to verify an extension exists.
    872909                 *         bool   $required     Is this a required feature or not.
    873910                 *         string $fallback_for Optional. The module this module replaces as a fallback.
    … …  
    880917
    881918                foreach ( $modules as $library => $module ) {
    882                         $extension = ( isset( $module['extension'] ) ? $module['extension'] : null );
    883                         $function  = ( isset( $module['function'] ) ? $module['function'] : null );
     919                        $extension  = ( isset( $module['extension'] ) ? $module['extension'] : null );
     920                        $function   = ( isset( $module['function'] ) ? $module['function'] : null );
     921                        $constant   = ( isset( $module['constant'] ) ? $module['constant'] : null );
     922                        $class_name = ( isset( $module['class'] ) ? $module['class'] : null );
    884923
    885924                        // If this module is a fallback for another function, check if that other function passed.
    … …  
    896935                        }
    897936
    898                         if ( ! $this->test_php_extension_availability( $extension, $function ) && ( ! isset( $module['php_bundled_version'] ) || version_compare( PHP_VERSION, $module['php_bundled_version'], '<' ) ) ) {
     937                        if ( ! $this->test_php_extension_availability( $extension, $function, $constant, $class_name ) && ( ! isset( $module['php_bundled_version'] ) || version_compare( PHP_VERSION, $module['php_bundled_version'], '<' ) ) ) {
    899938                                if ( $module['required'] ) {
    900939                                        $result['status'] = 'critical';
Note: See TracChangeset for help on using the changeset viewer.