WordPress.org

Make WordPress Core

Ticket #43986: 43986v4.2.diff

File 43986v4.2.diff, 9.6 KB (added by afragen, 2 years ago)

oops forgot a commit in the last patch. Shouldn't be functionally different but one less conditional.

  • src/wp-admin/css/list-tables-rtl.css

    diff --git src/wp-admin/css/list-tables-rtl.css src/wp-admin/css/list-tables-rtl.css
    index 468dcbf8a1..5a1b371c4e 100644
    div.action-links, 
    15101510        content: "\f158";
    15111511}
    15121512
     1513.plugin-card .column-compatibility .compatibility-incompatible:before {
     1514        color: #f00;
     1515}
     1516
    15131517.plugin-card .compatibility-compatible:before {
    15141518        content: "\f147";
    15151519}
    15161520
     1521.plugin-card .column-compatibility .compatibility-compatible:before {
     1522        color: #008000;
     1523}
     1524
     1525.plugin-card-incompatible {
     1526        color: #fff;
     1527        background-color:#C62D2D;
     1528        border: 1px solid #B02828;
     1529        padding: 10px;
     1530        margin: -12px -20px;
     1531}
     1532
     1533.plugin-card-incompatible a {
     1534        color: #fff;
     1535        text-decoration: underline;
     1536}
     1537
    15171538.plugin-icon {
    15181539        position: absolute;
    15191540        top: 20px;
  • src/wp-admin/css/list-tables.css

    diff --git src/wp-admin/css/list-tables.css src/wp-admin/css/list-tables.css
    index 8d78983993..55fa10cc06 100644
    div.action-links, 
    15101510        content: "\f158";
    15111511}
    15121512
     1513.plugin-card .column-compatibility .compatibility-incompatible:before {
     1514        color: #f00;
     1515}
     1516
    15131517.plugin-card .compatibility-compatible:before {
    15141518        content: "\f147";
    15151519}
    15161520
     1521.plugin-card .column-compatibility .compatibility-compatible:before {
     1522        color: #008000;
     1523}
     1524
     1525.plugin-card-incompatible {
     1526        color: #fff;
     1527        background-color:#C62D2D;
     1528        border: 1px solid #B02828;
     1529        padding: 10px;
     1530        margin: -12px -20px;
     1531}
     1532
     1533.plugin-card-incompatible a {
     1534        color: #fff;
     1535        text-decoration: underline;
     1536}
     1537
    15171538.plugin-icon {
    15181539        position: absolute;
    15191540        top: 20px;
  • 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..cf3b5e781c 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 { 
    632647                                </div>
    633648                        </div>
    634649                        <div class="plugin-card-bottom">
    635                                 <div class="vers column-rating">
     650                        <?php
     651                        if ( ! $can_install ) {
     652                                echo '<div class="plugin-card-incompatible">';
     653
     654                                // Remove this section for commit.
     655                                echo '<style>.plugin-card-incompatible {
     656                                        color: #fff;
     657                                        background-color:#C62D2D;
     658                                        border: 1px solid #B02828;
     659                                        padding: 10px;
     660                                        margin: -12px -20px;
     661                                }
     662                                .plugin-card-incompatible a { color: #fff; text-decoration: underline;}
     663                                </style>';
     664                                // End remove for commit.
     665
     666                                echo '<span class="dashicons dashicons-warning"></span>&nbsp;';
     667                                _e( 'This plugin is incompatible and cannot be installed.' );
     668                                echo '&nbsp;';
     669                                printf(
     670                                        '<a href="%s" class="thickbox open-plugin-details-modal" aria-label="%s" data-title="%s">%s</a>',
     671                                        esc_url( $details_link ),
     672                                        /* translators: %s: plugin name and version */
     673                                        esc_attr( sprintf( __( 'More information about %s' ), $name ) ),
     674                                        esc_attr( $name ),
     675                                        __( 'Why?' )
     676                                );
     677                                echo '</div></div></div>';
     678                                continue;
     679                        } ?>
     680                        <div class="vers column-rating">
    636681                                        <?php
    637682                                        wp_star_rating(
    638683                                                array(
    class WP_Plugin_Install_List_Table extends WP_List_Table { 
    641686                                                        'number' => $plugin['num_ratings'],
    642687                                                )
    643688                                        );
    644 ?>
     689                                        ?>
    645690                                        <span class="num-ratings" aria-hidden="true">(<?php echo number_format_i18n( $plugin['num_ratings'] ); ?>)</span>
    646691                                </div>
    647692                                <div class="column-updated">
    class WP_Plugin_Install_List_Table extends WP_List_Table { 
    664709                                        ?>
    665710                                </div>
    666711                                <div class="column-compatibility">
    667                                         <?php
    668                                         $wp_version = get_bloginfo( 'version' );
    669 
    670                                         if ( ! empty( $plugin['tested'] ) && version_compare( substr( $wp_version, 0, strlen( $plugin['tested'] ) ), $plugin['tested'], '>' ) ) {
    671                                                 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'], '<' ) ) {
    673                                                 echo '<span class="compatibility-incompatible">' . __( '<strong>Incompatible</strong> with your version of WordPress' ) . '</span>';
    674                                         } else {
    675                                                 echo '<span class="compatibility-compatible">' . __( '<strong>Compatible</strong> with your version of WordPress' ) . '</span>';
    676                                         }
    677                                         ?>
    678712                                </div>
    679713                        </div>
    680714                </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..da187ed07f 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 ( ! $compatible_php && $tested_php ) {
     763                echo '<div class="notice notice-error notice-alt"><p>' . sprintf(
     764                        /* translators: Upgrading PHP page URL */
     765                        __( '<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>' ),
     766                        esc_url( __( 'https://wordpress.org/support/upgrade-php/' ) )
     767                ) . '</p></div>';
     768        }
     769        if ( ! $tested_wp ) {
     770                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>';
     771        } elseif ( ! $compatible_wp ) {
     772                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>';
    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':