WordPress.org
Get WordPress

Make WordPress Core

Changeset 60924


Ignore:
Timestamp:
10/12/2025 02:20:59 PM (11 months ago)
Author:
SergeyBiryukov
Message:

Coding Standards: Rename $s to $search_term in wp-admin/plugins.php.

Per the ​Naming Conventions:

Don’t abbreviate variable names unnecessarily; let the code be unambiguous and self-documenting.

Follow-up to [15697].

Props costdev, mukesh27.
See #63168.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/wp-admin/plugins.php

    r59784 r60924  
    1919$action = $wp_list_table->current_action();
    2020
    21 $plugin = isset( $_REQUEST['plugin'] ) ? wp_unslash( $_REQUEST['plugin'] ) : '';
    22 $s      = isset( $_REQUEST['s'] ) ? urlencode( wp_unslash( $_REQUEST['s'] ) ) : '';
     21$plugin      = isset( $_REQUEST['plugin'] ) ? wp_unslash( $_REQUEST['plugin'] ) : '';
     22$search_term = 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=$s" ) );
     54                                wp_redirect( self_admin_url( "plugins.php?plugin_status=$status&paged=$page&s=$search_term" ) );
    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=$s" );
     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" );
    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=$s" ) );
     88                                wp_redirect( self_admin_url( "plugins.php?activate=true&plugin_status=$status&paged=$page&s=$search_term" ) );
    8989                        }
    9090                        exit;
    … …  
    120120
    121121                        if ( empty( $plugins ) ) {
    122                                 wp_redirect( self_admin_url( "plugins.php?plugin_status=$status&paged=$page&s=$s" ) );
     122                                wp_redirect( self_admin_url( "plugins.php?plugin_status=$status&paged=$page&s=$search_term" ) );
    123123                                exit;
    124124                        }
    … …  
    142142                        }
    143143
    144                         wp_redirect( self_admin_url( "plugins.php?activate-multi=true&plugin_status=$status&paged=$page&s=$s" ) );
     144                        wp_redirect( self_admin_url( "plugins.php?activate-multi=true&plugin_status=$status&paged=$page&s=$search_term" ) );
    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=$s" ) );
     207                                wp_redirect( self_admin_url( "plugins.php?plugin_status=$status&paged=$page&s=$search_term" ) );
    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=$s" ) . "' />";
    221                         } else {
    222                                 wp_redirect( self_admin_url( "plugins.php?deactivate=true&plugin_status=$status&paged=$page&s=$s" ) );
     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" ) );
    223223                        }
    224224                        exit;
    … …  
    247247                        }
    248248                        if ( empty( $plugins ) ) {
    249                                 wp_redirect( self_admin_url( "plugins.php?plugin_status=$status&paged=$page&s=$s" ) );
     249                                wp_redirect( self_admin_url( "plugins.php?plugin_status=$status&paged=$page&s=$search_term" ) );
    250250                                exit;
    251251                        }
    … …  
    264264                        }
    265265
    266                         wp_redirect( self_admin_url( "plugins.php?deactivate-multi=true&plugin_status=$status&paged=$page&s=$s" ) );
     266                        wp_redirect( self_admin_url( "plugins.php?deactivate-multi=true&plugin_status=$status&paged=$page&s=$search_term" ) );
    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=$s" ) );
     279                                wp_redirect( self_admin_url( "plugins.php?plugin_status=$status&paged=$page&s=$search_term" ) );
    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=$s" ) );
     285                                wp_redirect( self_admin_url( "plugins.php?error=true&main=true&plugin_status=$status&paged=$page&s=$search_term" ) );
    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=$s" ) );
     293                                wp_redirect( self_admin_url( "plugins.php?plugin_status=$status&paged=$page&s=$search_term" ) );
    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=$s" ) );
     435                        wp_redirect( self_admin_url( "plugins.php?deleted=$plugins_to_delete&plugin_status=$status&paged=$page&s=$search_term" ) );
    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=$s" ) );
     456                        $result = resume_plugin( $plugin, self_admin_url( "plugins.php?error=resuming&plugin_status=$status&paged=$page&s=$search_term" ) );
    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=$s" ) );
     462                        wp_redirect( self_admin_url( "plugins.php?resume=true&plugin_status=$status&paged=$page&s=$search_term" ) );
    463463                        exit;
    464464                case 'enable-auto-update':
    … …  
    474474                        }
    475475
    476                         $redirect = self_admin_url( "plugins.php?plugin_status={$status}&paged={$page}&s={$s}" );
     476                        $redirect = self_admin_url( "plugins.php?plugin_status={$status}&paged={$page}&s={$search_term}" );
    477477
    478478                        if ( 'enable-auto-update' === $action || 'disable-auto-update' === $action ) {
    … …  
    772772}
    773773
    774 if ( strlen( $s ) ) {
     774if ( strlen( $search_term ) ) {
    775775        echo '<span class="subtitle">';
    776776        printf(
    777777                /* translators: %s: Search query. */
    778778                __( 'Search results for: %s' ),
    779                 '<strong>' . esc_html( urldecode( $s ) ) . '</strong>'
     779                '<strong>' . esc_html( urldecode( $search_term ) ) . '</strong>'
    780780        );
    781781        echo '</span>';
Note: See TracChangeset for help on using the changeset viewer.