Make WordPress Core

Ticket #43986: 43986v3-2.diff

File 43986v3-2.diff, 6.9 KB (added by afragen, 6 years ago)

No more untested PHP notices.

  • src/wp-admin/includes/class-wp-plugin-install-list-table.php

    diff --git src/wp-admin/includes/class-wp-plugin-install-list-table.php src/wp-admin/includes/class-wp-plugin-install-list-table.php
    index a19729b1d4..607b9389a8 100644
    class WP_Plugin_Install_List_Table extends WP_List_Table { 
    496496                                $author = ' <cite>' . sprintf( __( 'By %s' ), $author ) . '</cite>';
    497497                        }
    498498
     499                        $wp_version = get_bloginfo( 'version' );
     500
     501                        $tested_php     = ! empty( $plugin['requires_php'] );
     502                        $compatible_php = ( ! $tested_php || version_compare( substr( PHP_VERSION, 0, strlen( $plugin['requires_php'] ) ), $plugin['requires_php'], '>=' ) );
     503                        $tested_wp      = ( empty( $plugin['tested'] ) || version_compare( substr( $wp_version, 0, strlen( $plugin['tested'] ) ), $plugin['tested'], '<=' ) );
     504                        $compatible_wp  = ( empty( $plugin['requires'] ) || version_compare( substr( $wp_version, 0, strlen( $plugin['requires'] ) ), $plugin['requires'], '>=' ) );
     505                        $can_install    = ( ! $tested_php || $compatible_php ) && $compatible_wp;
     506
    499507                        $action_links = array();
    500508
    501509                        if ( current_user_can( 'install_plugins' ) || current_user_can( 'update_plugins' ) ) {
    class WP_Plugin_Install_List_Table extends WP_List_Table { 
    504512                                switch ( $status['status'] ) {
    505513                                        case 'install':
    506514                                                if ( $status['url'] ) {
    507                                                         $action_links[] = sprintf(
    508                                                                 '<a class="install-now button" data-slug="%s" href="%s" aria-label="%s" data-name="%s">%s</a>',
    509                                                                 esc_attr( $plugin['slug'] ),
    510                                                                 esc_url( $status['url'] ),
    511                                                                 /* translators: %s: plugin name and version */
    512                                                                 esc_attr( sprintf( __( 'Install %s now' ), $name ) ),
    513                                                                 esc_attr( $name ),
    514                                                                 __( 'Install Now' )
    515                                                         );
     515                                                        if ( $can_install ) {
     516                                                                $action_links[] = sprintf(
     517                                                                        '<a class="install-now button" data-slug="%s" href="%s" aria-label="%s" data-name="%s">%s</a>',
     518                                                                        esc_attr( $plugin['slug'] ),
     519                                                                        esc_url( $status['url'] ),
     520                                                                        /* translators: %s: plugin name and version */
     521                                                                        esc_attr( sprintf( __( 'Install %s now' ), $name ) ),
     522                                                                        esc_attr( $name ),
     523                                                                        __( 'Install Now' )
     524                                                                );
     525                                                        } else {
     526                                                                $action_links[] = sprintf(
     527                                                                        '<button type="button" class="button button-disabled" disabled="disabled">%s</button>',
     528                                                                        _x( 'Cannot Install', 'plugin' )
     529                                                                );
     530                                                        }
    516531                                                }
    517532                                                break;
    518533
    class WP_Plugin_Install_List_Table extends WP_List_Table { 
    665680                                </div>
    666681                                <div class="column-compatibility">
    667682                                        <?php
    668                                         $wp_version = get_bloginfo( 'version' );
    669 
    670                                         if ( ! empty( $plugin['tested'] ) && version_compare( substr( $wp_version, 0, strlen( $plugin['tested'] ) ), $plugin['tested'], '>' ) ) {
     683                                        if ( ! $tested_wp ) {
    671684                                                echo '<span class="compatibility-untested">' . __( 'Untested with your version of WordPress' ) . '</span>';
    672                                         } elseif ( ! empty( $plugin['requires'] ) && version_compare( substr( $wp_version, 0, strlen( $plugin['requires'] ) ), $plugin['requires'], '<' ) ) {
     685                                        } elseif ( ! $compatible_wp ) {
    673686                                                echo '<span class="compatibility-incompatible">' . __( '<strong>Incompatible</strong> with your version of WordPress' ) . '</span>';
    674687                                        } else {
    675688                                                echo '<span class="compatibility-compatible">' . __( '<strong>Compatible</strong> with your version of WordPress' ) . '</span>';
    676689                                        }
     690                                        if ( ! $compatible_php ) {
     691                                                echo '<br><span class="compatibility-incompatible">' .
     692                                                __( '<strong>Incompatible</strong> with your version of PHP' ) . '</span>';
     693                                        } elseif ( $tested_php ) {
     694                                                echo '<br><span class="compatibility-compatible">' . __( '<strong>Compatible</strong> with your version of PHP' ) . '</span>';
     695                                        }
    677696                                        ?>
    678697                                </div>
    679698                        </div>
  • src/wp-admin/includes/plugin-install.php

    diff --git src/wp-admin/includes/plugin-install.php src/wp-admin/includes/plugin-install.php
    index f765fb286e..a22e707e38 100644
    if ( ! empty( $api->contributors ) ) { 
    753753        <?php
    754754        $wp_version = get_bloginfo( 'version' );
    755755
    756         if ( ! empty( $api->tested ) && version_compare( substr( $wp_version, 0, strlen( $api->tested ) ), $api->tested, '>' ) ) {
    757                 echo '<div class="notice notice-warning notice-alt"><p>' . __( '<strong>Warning:</strong> This plugin has <strong>not been tested</strong> with your current version of WordPress.' ) . '</p></div>';
    758         } elseif ( ! empty( $api->requires ) && version_compare( substr( $wp_version, 0, strlen( $api->requires ) ), $api->requires, '<' ) ) {
    759                 echo '<div class="notice notice-warning notice-alt"><p>' . __( '<strong>Warning:</strong> This plugin has <strong>not been marked as compatible</strong> with your version of WordPress.' ) . '</p></div>';
     756        $tested_php     = ! empty( $api->requires_php );
     757        $compatible_php = ( ! $tested_php || version_compare( substr( PHP_VERSION, 0, strlen( $api->requires_php ) ), $api->requires_php, '>=' ) );
     758        $tested_wp      = ( empty( $api->tested ) || version_compare( substr( $wp_version, 0, strlen( $api->tested ) ), $api->tested, '<=' ) );
     759        $compatible_wp  = ( empty( $api->requires ) || version_compare( substr( $wp_version, 0, strlen( $api->requires ) ), $api->requires, '>=' ) );
     760        $can_install    = ( ! $tested_php || $compatible_php ) && $compatible_wp;
     761
     762        if ( ! $tested_wp ) {
     763                echo '<div class="notice notice-warning notice-alt"><p>' . __( '<strong>Warning:</strong> This plugin <strong>has not been tested</strong> with your current version of WordPress.' ) . '</p></div>';
     764        } elseif ( ! $compatible_wp ) {
     765                echo '<div class="notice notice-error notice-alt"><p>' . __( '<strong>Error:</strong> This plugin <strong>has not been marked as compatible</strong> with your version of WordPress.' ) . '</p></div>';
     766        }
     767        if ( ! $compatible_php && $tested_php ) {
     768                echo '<div class="notice notice-error notice-alt"><p>' . sprintf(
     769                        /* translators: Upgrading PHP page URL */
     770                        __( '<strong>Error:</strong> This plugin <strong>requires a newer version of PHP</strong>, so unfortunately you cannot install it. <a href="%s">Click here to learn more about upgrading PHP.</a>' ),
     771                        esc_url( __( 'https://wordpress.org/support/upgrade-php/' ) )
     772                ) . '</p></div>';
    760773        }
    761774
    762775        foreach ( (array) $api->sections as $section_name => $content ) {
    if ( ! empty( $api->contributors ) ) { 
    779792                $status = install_plugin_install_status( $api );
    780793                switch ( $status['status'] ) {
    781794                        case 'install':
    782                                 if ( $status['url'] ) {
     795                                if ( $can_install ) {
    783796                                        echo '<a data-slug="' . esc_attr( $api->slug ) . '" id="plugin_install_from_iframe" class="button button-primary right" href="' . $status['url'] . '" target="_parent">' . __( 'Install Now' ) . '</a>';
     797                                } else {
     798                                        printf(
     799                                                '<button type="button" class="button button-primary button-disabled right" disabled="disabled">%s</button>',
     800                                                _x( 'Cannot Install', 'plugin' )
     801                                        );
    784802                                }
    785803                                break;
    786804                        case 'update_available':