Make WordPress Core

Changeset 60929


Ignore:
Timestamp:
10/13/2025 11:10:18 PM (3 months ago)
Author:
SergeyBiryukov
Message:

Coding Standards: Revert the renaming of $s variable in wp-admin/plugins.php.

The $s global needs to remain for backward compatibility.

Follow-up to [60924].

Props westonruter, mukesh27.
See #63168.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-admin/plugins.php

    r60924 r60929  
    1919$action = $wp_list_table->current_action();
    2020
    21 $plugin      = isset( $_REQUEST['plugin'] ) ? wp_unslash( $_REQUEST['plugin'] ) : '';
    22 $search_term = isset( $_REQUEST['s'] ) ? urlencode( wp_unslash( $_REQUEST['s'] ) ) : '';
     21$plugin = isset( $_REQUEST['plugin'] ) ? wp_unslash( $_REQUEST['plugin'] ) : '';
     22$s      = isset( $_REQUEST['s'] ) ? urlencode( wp_unslash( $_REQUEST['s'] ) ) : '';
    2323
    2424// Clean up request URI from temporary args for screen options/paging uri's to work as expected.
     
    5252
    5353            if ( is_multisite() && ! is_network_admin() && is_network_only_plugin( $plugin ) ) {
    54                 wp_redirect( self_admin_url( "plugins.php?plugin_status=$status&paged=$page&s=$search_term" ) );
     54                wp_redirect( self_admin_url( "plugins.php?plugin_status=$status&paged=$page&s=$s" ) );
    5555                exit;
    5656            }
     
    6161            if ( is_wp_error( $result ) ) {
    6262                if ( 'unexpected_output' === $result->get_error_code() ) {
    63                     $redirect = self_admin_url( 'plugins.php?error=true&charsout=' . strlen( $result->get_error_data() ) . '&plugin=' . urlencode( $plugin ) . "&plugin_status=$status&paged=$page&s=$search_term" );
     63                    $redirect = self_admin_url( 'plugins.php?error=true&charsout=' . strlen( $result->get_error_data() ) . '&plugin=' . urlencode( $plugin ) . "&plugin_status=$status&paged=$page&s=$s" );
    6464                    wp_redirect( add_query_arg( '_error_nonce', wp_create_nonce( 'plugin-activation-error_' . $plugin ), $redirect ) );
    6565                    exit;
     
    8686            } else {
    8787                // Overrides the ?error=true one above.
    88                 wp_redirect( self_admin_url( "plugins.php?activate=true&plugin_status=$status&paged=$page&s=$search_term" ) );
     88                wp_redirect( self_admin_url( "plugins.php?activate=true&plugin_status=$status&paged=$page&s=$s" ) );
    8989            }
    9090            exit;
     
    120120
    121121            if ( empty( $plugins ) ) {
    122                 wp_redirect( self_admin_url( "plugins.php?plugin_status=$status&paged=$page&s=$search_term" ) );
     122                wp_redirect( self_admin_url( "plugins.php?plugin_status=$status&paged=$page&s=$s" ) );
    123123                exit;
    124124            }
     
    142142            }
    143143
    144             wp_redirect( self_admin_url( "plugins.php?activate-multi=true&plugin_status=$status&paged=$page&s=$search_term" ) );
     144            wp_redirect( self_admin_url( "plugins.php?activate-multi=true&plugin_status=$status&paged=$page&s=$s" ) );
    145145            exit;
    146146
     
    205205
    206206            if ( ! is_network_admin() && is_plugin_active_for_network( $plugin ) ) {
    207                 wp_redirect( self_admin_url( "plugins.php?plugin_status=$status&paged=$page&s=$search_term" ) );
     207                wp_redirect( self_admin_url( "plugins.php?plugin_status=$status&paged=$page&s=$s" ) );
    208208                exit;
    209209            }
     
    218218
    219219            if ( headers_sent() ) {
    220                 echo "<meta http-equiv='refresh' content='" . esc_attr( "0;url=plugins.php?deactivate=true&plugin_status=$status&paged=$page&s=$search_term" ) . "' />";
    221             } else {
    222                 wp_redirect( self_admin_url( "plugins.php?deactivate=true&plugin_status=$status&paged=$page&s=$search_term" ) );
     220                echo "<meta http-equiv='refresh' content='" . esc_attr( "0;url=plugins.php?deactivate=true&plugin_status=$status&paged=$page&s=$s" ) . "' />";
     221            } else {
     222                wp_redirect( self_admin_url( "plugins.php?deactivate=true&plugin_status=$status&paged=$page&s=$s" ) );
    223223            }
    224224            exit;
     
    247247            }
    248248            if ( empty( $plugins ) ) {
    249                 wp_redirect( self_admin_url( "plugins.php?plugin_status=$status&paged=$page&s=$search_term" ) );
     249                wp_redirect( self_admin_url( "plugins.php?plugin_status=$status&paged=$page&s=$s" ) );
    250250                exit;
    251251            }
     
    264264            }
    265265
    266             wp_redirect( self_admin_url( "plugins.php?deactivate-multi=true&plugin_status=$status&paged=$page&s=$search_term" ) );
     266            wp_redirect( self_admin_url( "plugins.php?deactivate-multi=true&plugin_status=$status&paged=$page&s=$s" ) );
    267267            exit;
    268268
     
    277277            $plugins = isset( $_REQUEST['checked'] ) ? (array) wp_unslash( $_REQUEST['checked'] ) : array();
    278278            if ( empty( $plugins ) ) {
    279                 wp_redirect( self_admin_url( "plugins.php?plugin_status=$status&paged=$page&s=$search_term" ) );
     279                wp_redirect( self_admin_url( "plugins.php?plugin_status=$status&paged=$page&s=$s" ) );
    280280                exit;
    281281            }
     
    283283            $plugins = array_filter( $plugins, 'is_plugin_inactive' ); // Do not allow to delete activated plugins.
    284284            if ( empty( $plugins ) ) {
    285                 wp_redirect( self_admin_url( "plugins.php?error=true&main=true&plugin_status=$status&paged=$page&s=$search_term" ) );
     285                wp_redirect( self_admin_url( "plugins.php?error=true&main=true&plugin_status=$status&paged=$page&s=$s" ) );
    286286                exit;
    287287            }
     
    291291            $invalid_plugin_files = array_filter( $plugins, 'validate_file' );
    292292            if ( $invalid_plugin_files ) {
    293                 wp_redirect( self_admin_url( "plugins.php?plugin_status=$status&paged=$page&s=$search_term" ) );
     293                wp_redirect( self_admin_url( "plugins.php?plugin_status=$status&paged=$page&s=$s" ) );
    294294                exit;
    295295            }
     
    433433            // Cannot use transient/cache, as that could get flushed if any plugin flushes data on uninstall/delete.
    434434            update_option( 'plugins_delete_result_' . $user_ID, $delete_result, false );
    435             wp_redirect( self_admin_url( "plugins.php?deleted=$plugins_to_delete&plugin_status=$status&paged=$page&s=$search_term" ) );
     435            wp_redirect( self_admin_url( "plugins.php?deleted=$plugins_to_delete&plugin_status=$status&paged=$page&s=$s" ) );
    436436            exit;
    437437        case 'clear-recent-list':
     
    454454            check_admin_referer( 'resume-plugin_' . $plugin );
    455455
    456             $result = resume_plugin( $plugin, self_admin_url( "plugins.php?error=resuming&plugin_status=$status&paged=$page&s=$search_term" ) );
     456            $result = resume_plugin( $plugin, self_admin_url( "plugins.php?error=resuming&plugin_status=$status&paged=$page&s=$s" ) );
    457457
    458458            if ( is_wp_error( $result ) ) {
     
    460460            }
    461461
    462             wp_redirect( self_admin_url( "plugins.php?resume=true&plugin_status=$status&paged=$page&s=$search_term" ) );
     462            wp_redirect( self_admin_url( "plugins.php?resume=true&plugin_status=$status&paged=$page&s=$s" ) );
    463463            exit;
    464464        case 'enable-auto-update':
     
    474474            }
    475475
    476             $redirect = self_admin_url( "plugins.php?plugin_status={$status}&paged={$page}&s={$search_term}" );
     476            $redirect = self_admin_url( "plugins.php?plugin_status={$status}&paged={$page}&s={$s}" );
    477477
    478478            if ( 'enable-auto-update' === $action || 'disable-auto-update' === $action ) {
     
    772772}
    773773
    774 if ( strlen( $search_term ) ) {
     774if ( strlen( $s ) ) {
    775775    echo '<span class="subtitle">';
    776776    printf(
    777777        /* translators: %s: Search query. */
    778778        __( 'Search results for: %s' ),
    779         '<strong>' . esc_html( urldecode( $search_term ) ) . '</strong>'
     779        '<strong>' . esc_html( urldecode( $s ) ) . '</strong>'
    780780    );
    781781    echo '</span>';
Note: See TracChangeset for help on using the changeset viewer.